From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55122) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VZze6-0001dz-RN for qemu-devel@nongnu.org; Sat, 26 Oct 2013 04:55:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VZze0-00075V-Kk for qemu-devel@nongnu.org; Sat, 26 Oct 2013 04:54:58 -0400 Received: from p15195424.pureserver.info ([82.165.34.74]:37344) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VZze0-00075N-BH for qemu-devel@nongnu.org; Sat, 26 Oct 2013 04:54:52 -0400 Message-ID: <526B8316.7080105@ilande.co.uk> Date: Sat, 26 Oct 2013 09:53:42 +0100 From: Mark Cave-Ayland MIME-Version: 1.0 References: <1381868394-12656-1-git-send-email-mark.cave-ayland@ilande.co.uk> In-Reply-To: <1381868394-12656-1-git-send-email-mark.cave-ayland@ilande.co.uk> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/2] sun4m: Implement Sun CG3 framebuffer for QEMU List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Blue Swirl , Peter Maydell , aliguori@amazon.com On 15/10/13 21:19, Mark Cave-Ayland wrote: > This patchset follows on from the previous "Add FCode ROM for TCX framebuffer" > patch and provides QEMU with an implementation of the Sun CG3 8-bit > framebuffer. It is based upon Bob Breuer's original work which has been > rebased onto git master, and is now capable of running with an OpenBIOS CG3 > FCode ROM instead of requiring copies of proprietary Sun ROMs. > > Note this patch has been around for a month or so but has been waiting for the > updated OpenBIOS images to be included in QEMU. Therefore while it has not yet > been reviewed, it is a reasonably simple and self-contained patchset > and has been updated to reflect the comments received from the associated > "Add FCode ROM for TCX framebuffer" patch. For this reason, I feel that both > these patches are a candidate for 1.7. > > The motivation behind this patch is that older operating systems such as > Debian Woody and Solaris (running OpenWindows) do not contain drivers for the > TCX framebuffer and as a result currently cannot run in graphical mode. The > screenshots linked below show qemu-system-sparc successfully running both > Debian Woody and the Solaris 8 installer in graphical mode during testing: > > http://www.ilande.co.uk/tmp/debian-woody.png > http://www.ilande.co.uk/tmp/sol8-1.png > http://www.ilande.co.uk/tmp/sol8-2.png > > The CG3 framebuffer is selected by passing -vga cg3 on the command line to > qemu-system-sparc. If either -vga tcx is specified (or the -vga argument is > omitted) then qemu-system-sparc defaults to using the existing TCX > framebuffer to maintain compatibility. > > Signed-off-by: Mark Cave-Ayland Hi Anthony, Do you think it is possible that this (and the associated TCX framebuffer patch also mentioned above) could be applied for 1.7? While I've had no review from Blue (no news is good news?), I feel that the patch itself is in reasonably good shape and has been on my computer in various forms for several months now. Peter reviewed the associated "Add FCode ROM for TCX framebuffer" patch, and I resubmitted a revised version back to the list based upon his comments a while back. I then made the same stylistic changes to the CG3 patch and posted it above just in time for the soft feature freeze for review. Peter has further commented on this patch, but didn't have any objections AFAICT. Both patches are fine according to checkpatch, and if any issues are found then I'm happy to iron them out before the 1.7 release. I realise that SPARC is not one of the more popular architectures these days, however myself, Artyom and Bob still receive personal emails from people who are interested in QEMU's ability to run Solaris under emulation (either to replace aging hardware or provide resilience). On the plus side, given that there are fewer users and the fact that the patch preserves using TCX as the default, then the risk of side-effects from these two patches should be extremely minimal. ATB, Mark.