From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1DM7EO-0007zW-GV for qemu-devel@nongnu.org; Thu, 14 Apr 2005 12:30:17 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1DM7EA-0007qA-3V for qemu-devel@nongnu.org; Thu, 14 Apr 2005 12:30:10 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DM7E7-0007nu-DI for qemu-devel@nongnu.org; Thu, 14 Apr 2005 12:29:59 -0400 Received: from [212.227.126.187] (helo=moutng.kundenserver.de) by monty-python.gnu.org with esmtp (Exim 4.34) id 1DM7Cn-0001ki-3Q for qemu-devel@nongnu.org; Thu, 14 Apr 2005 12:28:37 -0400 From: Volker Ruppert Subject: Re: [Qemu-devel] [PATCH] Cirrus bugfixes Date: Thu, 14 Apr 2005 18:29:13 +0200 References: <200504132248.53304.info@vruppert.de> <41e41e7a05041314225d955409@mail.gmail.com> In-Reply-To: <41e41e7a05041314225d955409@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline Message-Id: <200504141829.13439.info@vruppert.de> Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, Hetz Ben Hamo Hi, > I was wondering - is this patch affects only the ISA part of the > Cirrus emulation? No, this patch also affects the PCI version of the Cirrus adapter. I did my= =20 tests mostly with the ISA version to find out the bugs. The Win 3.11 driver= =20 for the CL-GD5430 seems to use more of the currently unimplemented features= =20 than the PCI driver for Win95 does. > Also, forgive me for asking, but why do you use the ISA > implementation? I'm sure there are drivers for both win 3.11 and > win95/98 for the PCI Cirrus.. I know that there are drivers for PCI and I'm using the Qemu PCI system for= =20 Win95/98, but using Win3.11 on ISA was the best way to find out how the=20 bitblt destination write mask should work. Here is the list of changes with some additions: =A0- partial implementation of the destination write mask (bit 7 unhandled) This affects ISA and PCI on Win3.1x,Win9x and maybe others. It fixes the Win95 tooltips and the Borders of selected items on Win3.11 with Calmira II. This is the most important part of the patch. =A0- fixed banked memory access of the ISA version This description is wrong. Banked memory access is possible on PCI, too, but the PCI drivers usually using the LFB. =A0- the bus type bits in sequencer reg 0x17 must be readonly Without this change the bus type bits can be changed by software. This shouldn't happen. The Cirrus driver for Win3.11 probes this register and tries to overwrite these bits (switches from ISA to PCI). =A0- fixed destination write mask in debug message This bugfix made it possible to find the other bugs. First I was wonderi= ng why the value was always 0, but I expected something else. =2D- Bye Volker