summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authoraliguori <aliguori@c046a42c-6fe2-441c-8c8c-71466251a162>2009-02-27 19:53:57 +0000
committeraliguori <aliguori@c046a42c-6fe2-441c-8c8c-71466251a162>2009-02-27 19:53:57 +0000
commit31c05501c76b917bef4ae477f093e27dc9ef1b3b (patch)
tree6a3388776c3d346994e61c587ce6c5d24ea41aa8
parent9a40611cd4e9847caa8a0c80113baf6ad54b4671 (diff)
downloadqemu-31c05501c76b917bef4ae477f093e27dc9ef1b3b.tar.gz
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 <kressb@moose.net> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@6646 c046a42c-6fe2-441c-8c8c-71466251a162
-rw-r--r--hw/cirrus_vga.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/hw/cirrus_vga.c b/hw/cirrus_vga.c
index 0d1ea3a51c..8aade84846 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)