From: WaxDragon <waxdragon@gmail.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH]Fix for minor video corruption under Windows
Date: Wed, 10 May 2006 10:27:03 -0400 [thread overview]
Message-ID: <9b0d5f320605100727ga99046br4f89afc8cb256d38@mail.gmail.com> (raw)
In-Reply-To: <7F740D512C7C1046AB53446D3720017307E75ED4@scsmsx402.amr.corp.intel.com>
I tried out this patch (with CVS, after fixing it) with XP SP2 and 2k3
SP1. It did address the corruption when changing bpp or resolution,
but paints the screen white while the UI elements repaint themselves.
Just a little disturbing, but I'll get used to it. Also, I had the
leave the second chunk in, otherwise XP failed to paint much of the
screen upon bootup.
Index: hw/cirrus_vga.c
===================================================================
RCS file: /sources/qemu/qemu/hw/cirrus_vga.c,v
retrieving revision 1.21
diff -u -r1.21 cirrus_vga.c
--- hw/cirrus_vga.c 30 Apr 2006 21:28:36 -0000 1.21
+++ hw/cirrus_vga.c 10 May 2006 14:08:25 -0000
@@ -1181,6 +1181,17 @@
break;
case 0x05: // ???
case 0x07: // Extended Sequencer Mode
+ /* Win2K seems to assume that the VRAM is set to 0xff
+ * whenever VGA/SVGA mode changes
+ */
+ if ((s->sr[0x07] ^ *reg_value) & CIRRUS_SR7_BPP_SVGA)
+ memset(s->vram_ptr, 0xff, s->real_vram_size);
+ *reg_value = s->sr[0x07];
+#ifdef DEBUG_CIRRUS
+ printf("cirrus: handled outport sr_index %02x, sr_value %02x\n",
+ reg_index, reg_value);
+#endif
+ break;
case 0x08: // EEPROM Control
case 0x09: // Scratch Register 0
case 0x0a: // Scratch Register 1
WD
--
ReactOS is a hub, follow the spokes and you'll
immediately find absolutely everything you need
to know about Windows. ReactOS is not just
software, it's people.
kjk_hyperion
next prev parent reply other threads:[~2006-05-10 14:27 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-05-09 23:12 [Qemu-devel] [PATCH]Fix for minor video corruption under Windows Dugger, Donald D
2006-05-10 14:27 ` WaxDragon [this message]
2006-05-11 22:47 ` Fabrice Bellard
-- strict thread matches above, loose matches on Subject: below --
2006-05-12 0:15 Dugger, Donald D
2006-05-10 16:17 Dugger, Donald D
2006-05-09 19:56 Donald D. Dugger
2006-05-09 20:29 ` Leonardo E. Reiter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9b0d5f320605100727ga99046br4f89afc8cb256d38@mail.gmail.com \
--to=waxdragon@gmail.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).