From 31c05501c76b917bef4ae477f093e27dc9ef1b3b Mon Sep 17 00:00:00 2001 From: aliguori Date: Fri, 27 Feb 2009 19:53:57 +0000 Subject: Missing cirrus_invalidate_region() in cirrus_do_copy() (Brian Kress) After doing a video to video copy, cirrus_do_copy() in cirrus_vga.c does a conditional call to cirrus_invalidate_region() with an "if (!notify)" test. However at this point the blt has already been done so it seems like this call should be made unconditionally. The patch below fixes some display update problems when dragging windows under both X (fedora 10 guest) and a Windows XP guest. Signed-off-by: Brian Kress Signed-off-by: Anthony Liguori git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@6646 c046a42c-6fe2-441c-8c8c-71466251a162 --- hw/cirrus_vga.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'hw/cirrus_vga.c') diff --git a/hw/cirrus_vga.c b/hw/cirrus_vga.c index 0d1ea3a51..8aade8484 100644 --- a/hw/cirrus_vga.c +++ b/hw/cirrus_vga.c @@ -781,10 +781,9 @@ static void cirrus_do_copy(CirrusVGAState *s, int dst, int src, int w, int h) /* we don't have to notify the display that this portion has changed since qemu_console_copy implies this */ - if (!notify) - cirrus_invalidate_region(s, s->cirrus_blt_dstaddr, - s->cirrus_blt_dstpitch, s->cirrus_blt_width, - s->cirrus_blt_height); + cirrus_invalidate_region(s, s->cirrus_blt_dstaddr, + s->cirrus_blt_dstpitch, s->cirrus_blt_width, + s->cirrus_blt_height); } static int cirrus_bitblt_videotovideo_copy(CirrusVGAState * s) -- cgit v1.2.3