qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 0/2] sun4m: Implement Sun CG3 framebuffer for QE


From: Mark Cave-Ayland
Subject: Re: [Qemu-devel] [PATCH 0/2] sun4m: Implement Sun CG3 framebuffer for QEMU
Date: Sat, 26 Oct 2013 09:53:42 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.12) Gecko/20130116 Icedove/10.0.12

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<address@hidden>

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.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]