[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 00/13] Mac OS 9 compatibility improvements (upst
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH 00/13] Mac OS 9 compatibility improvements (upstream rework) |
Date: |
Wed, 11 Nov 2015 13:11:59 +1100 |
User-agent: |
Mutt/1.5.23 (2015-06-09) |
On Wed, Nov 04, 2015 at 11:32:02PM +0000, Mark Cave-Ayland wrote:
> On 04/11/15 03:44, David Gibson wrote:
>
> > On Fri, Oct 30, 2015 at 04:48:12PM +0000, Mark Cave-Ayland wrote:
> >> On 23/10/15 14:56, Mark Cave-Ayland wrote:
> >>
> >>> This is a rework of Cormac O'Brien's GSoC project to try and boot MacOS 9
> >>> under
> >>> QEMU, the original version of which was posted to the qemu-devel list at
> >>> the
> >>> end of August
> >>> (https://lists.gnu.org/archive/html/qemu-devel/2015-08/msg02521.html).
> >>>
> >>> The patchset consisted of some simple patches from Alex and then a large
> >>> set of
> >>> CUDA changes supplied as a single patch which were the result of Cormac
> >>> analysing
> >>> MOL with Alex's help to try and further the boot process.
> >>>
> >>> In their previous form, the patches were unsuitable for applying upstream
> >>> since
> >>> while they furthered MacOS 9 boot, they also caused a couple of major
> >>> regressions
> >>> such as breaking the mouse and causing Darwin/OS X boot to panic on
> >>> startup.
> >>>
> >>> This reworked patchset fixes these regressions, includes some other
> >>> clean-ups
> >>> and more importantly now passes all of my OpenBIOS image boot tests with
> >>> an
> >>> OpenBIOS binary from SVN trunk (separate pull request to be sent shortly).
> >>> Whilst OpenBIOS still needs one additional patch to run the MacOS 9
> >>> bootloader,
> >>> I've uploaded a pre-compiled binary to
> >>> https://www.ilande.co.uk/tmp/openbios-ppc for people interested in
> >>> testing the
> >>> new MacOS 9 functionality.
> >>>
> >>> Apologies for the delay in sending this out on-list, however due to recent
> >>> circumstances I've been without a reliable broadband connection for a
> >>> couple
> >>> of weeks. However given that this is mostly a rework of the previous
> >>> patchset
> >>> and looks good in testing here, I'd definitely like it to be considered
> >>> for
> >>> application during soft freeze.
> >>>
> >>> Signed-off-by: Mark Cave-Ayland <address@hidden>
> >>>
> >>> Alexander Graf (3):
> >>> PPC: Allow Rc bit to be set on mtspr
> >>> PPC: Fix lsxw bounds checks
> >>> PPC: mac99: Always add USB controller
> >>>
> >>> Mark Cave-Ayland (10):
> >>> cuda.c: fix CUDA ADB error packet format
> >>> cuda.c: fix CUDA_PACKET response packet format
> >>> cuda.c: implement simple CUDA_GET_6805_ADDR command
> >>> cuda.c: implement dummy IIC access commands
> >>> cuda.c: fix CUDA SR interrupt clearing
> >>> cuda.c: add defines for CUDA registers
> >>> cuda.c: refactor get_tb() so that the time can be passed in
> >>> cuda.c: rename get_counter() state variable from s to ti for
> >>> consistency
> >>> cuda.c: fix T2 timer and enable its interrupt
> >>> cuda.c: add delay to setting of SR_INT bit
> >>>
> >>> hw/misc/macio/cuda.c | 243
> >>> ++++++++++++++++++++++++++++++-----------------
> >>> hw/ppc/mac.h | 3 +
> >>> hw/ppc/mac_newworld.c | 3 +-
> >>> target-ppc/mem_helper.c | 5 +-
> >>> target-ppc/translate.c | 2 +-
> >>> 5 files changed, 163 insertions(+), 93 deletions(-)
> >>
> >> Ping? Can anyone review this in Alex's absence? In the meantime I've
> >> added it to wiki at http://wiki.qemu.org/Planning/2.5 as it would be
> >> good to get the GSoC work upstream for 2.5.
> >
> > Sorry I've taken a while to get to this. It looks pretty good, though
> > I've sent a handful of comments on individual patches.
> >
> > I gathered from one of your replies that you do intend to do a
> > respin. The current comments all look pretty trivial, so I expect
> > I'll be ok to apply your respin to ppc-next (which I'm looking after
> > in agraf's absence). It would be nice to get a review from someone
> > more familiar with, or better able to test MacOS stuff.
>
> Great. I've sent a further few replies, so if you're happy with the
> answers let me know and I'll send a v2 tomorrow.
Haven't seen v2. Did something sidetrack you, or did I manage to miss
it?
>
> In terms of testing, as I mentioned above it doesn't regress any of my
> working OpenBIOS test-suite images which is a good sign. I'm not sure
> who else would be able to review the MacOS side. Both Segher and Ben. H
> have looked at parts of the original code changes during GSoC, although
> that was from a debugging rather than a code review perspective.
>
>
> ATB,
>
> Mark.
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature