Re: [Qemu-devel] [PATCH 0/6] VGA patches

[Blue Swirl]

On Sat, Feb 4, 2012 at 16:16, Jan Kiszka <address@hidden> wrote:
> On 2012-02-04 14:15, Blue Swirl wrote:
>> On Sun, Jan 29, 2012 at 19:12, Blue Swirl <address@hidden> wrote:
>>> Patch 1 has been sent several times, though the logic has been copied
>>> from flawed implementations before.
>>>
>>> Other patches should be trivial.
>>
>> Pushed. I compared vga.o object files with codiff and objdump
>> disassembler before and after the constant patch and the code was
>> identical.
>>
>>> URL     git://repo.or.cz/qemu/blueswirl.git
>>>        http://repo.or.cz/r/qemu/blueswirl.git
>>>
>>> Blue Swirl (6):
>>>  memory: change dirty getting API to take a size
>
> This causes regressions of WinXP guests. The upper half of the screen is
> not properly updated.

I can't reproduce this, but can you try the following:

diff --git a/hw/vga.c b/hw/vga.c
index d87c4f9..729ab8d 100644
--- a/hw/vga.c
+++ b/hw/vga.c
@@ -1779,7 +1779,7 @@ static void vga_draw_graphic(VGACommonState *s,
int full_update)
         }
         page0 = addr;
         page1 = addr + bwidth - 1;
-        update = memory_region_get_dirty(&s->vram, page0, page1,
+        update = memory_region_get_dirty(&s->vram, page0, bwidth - 1,
                                          DIRTY_MEMORY_VGA);
         /* explicit invalidation for the hardware cursor */
         update |= (s->invalidated_y_table[y >> 5] >> (y & 0x1f)) & 1;