From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1FT13z-0001g6-Dj for qemu-devel@nongnu.org; Mon, 10 Apr 2006 14:24:35 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1FT13y-0001fa-5G for qemu-devel@nongnu.org; Mon, 10 Apr 2006 14:24:34 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1FT13y-0001fX-0S for qemu-devel@nongnu.org; Mon, 10 Apr 2006 14:24:34 -0400 Received: from [216.148.227.151] (helo=rwcrmhc11.comcast.net) by monty-python.gnu.org with esmtp (Exim 4.52) id 1FT18f-0001zS-0h for qemu-devel@nongnu.org; Mon, 10 Apr 2006 14:29:25 -0400 Message-ID: <443AA2DE.3060908@win4lin.com> Date: Mon, 10 Apr 2006 14:24:30 -0400 From: "Leonardo E. Reiter" MIME-Version: 1.0 Subject: Re: [Qemu-devel] Updated BGR vs. RGB vga patch... References: <443A86FC.9010302@win4lin.com> <200604101749.00725.paul@codesourcery.com> <443A8D0C.4030807@win4lin.com> <200604101759.34399.paul@codesourcery.com> In-Reply-To: <200604101759.34399.paul@codesourcery.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit 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 Here's a better patch... I can't seem to validate BGR X servers in 15 or 16-bit mode. This may not make sense (or be prevalent) on little-endian machines anyway (I'm using VNC server on a little endian box to test against.) In any case, 24 and 32-bit works like a charm. I use the same basic logic as the original patch, so it's possible it was always broken for 16-bit... especially given that most Sun X servers run in 24-bit, which is what the originators intended the patch for. Anyway, I didn't spend time figuring out how to query SDL for the ordering. You still have to manually pass in -bgr. The good news is that all the computation is now done at compile-time, with only very few tests done at run-time. I agree with your assessment of compile-time versus run-time options... I was merely suggesting a compromise to fend off the minor performance hit of using the old patch. But the method you suggested is much better and that's how it's implemented now. Thanks, Leo Reiter Paul Brook wrote: > Ok. For the record I also think it's a bad idea to have features conditionally > compiled. Either something is worth including, or we have to ask whether > there's any point having it in qemu at all. > The exception being debug code, which probably isn't useful unless you're > already building qemu from source. > > Paul -- Leonardo E. Reiter Vice President of Product Development, CTO Win4Lin, Inc. Virtual Computing from Desktop to Data Center Main: +1 512 339 7979 Fax: +1 512 532 6501 http://www.win4lin.com