Patchwork [RFC,v5,00/86] Memory API

login
register
mail settings
Submitter Jan Kiszka
Date July 20, 2011, 5:41 p.m.
Message ID <4E27132E.6080504@siemens.com>
Download mbox | patch
Permalink /patch/105994/
State New
Headers show

Comments

Jan Kiszka - July 20, 2011, 5:41 p.m.
On 2011-07-20 18:49, Avi Kivity wrote:
> New in this version:
>   - more mindless conversions; I believe there are no longer any destructive
>     operations in the tree (IO_MEM_UNASSIGNED)
>   - fix memory map generation bug (patch 13)
>   - proper 440FX PAM/SMRAM and PCI holes
> 

This on top fixes standard VGA dirty logging:


Cirrus is still broken (test case: grub), debugging ATM.

Jan

Patch

diff --git a/hw/cirrus_vga.c b/hw/cirrus_vga.c
index e367052..8e8b24c 100644
--- a/hw/cirrus_vga.c
+++ b/hw/cirrus_vga.c
@@ -2944,8 +2944,6 @@  static int pci_cirrus_vga_initfn(PCIDevice *dev)
     memory_region_add_subregion(&s->pci_bar, 0x1000000,
                                 &s->cirrus_linear_bitblt_io);
 
-    vga_dirty_log_start(&s->vga);
-
      /* setup memory space */
      /* memory #0 LFB */
      /* memory #1 memory-mapped I/O */
diff --git a/hw/vga.c b/hw/vga.c
index 5a78b71..8acc545 100644
--- a/hw/vga.c
+++ b/hw/vga.c
@@ -2195,6 +2195,7 @@  void vga_common_init(VGACommonState *s, int vga_ram_size)
         s->update_retrace_info = vga_precise_update_retrace_info;
         break;
     }
+    vga_dirty_log_start(s);
 }
 
 /* used by both ISA and PCI */