Re: [Qemu-devel] [PATCH 00/13] Mac OS 9 compatibility improvements (upstream rework)

[Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>]

On 11/11/15 06:52, David Gibson wrote:

> On Wed, Nov 11, 2015 at 06:29:09AM +0000, Mark Cave-Ayland wrote:
>> On 11/11/15 02:11, David Gibson wrote:
>>
>>> 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 <mark.cave-ayland@ilande.co.uk>
>>>>>>>
>>>>>>> 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?
>>
>> Hi David,
>>
>> I am still waiting for feedback (and Reviewed-by tags) for a few replies
>> to your initial patch review:
>>
>> https://lists.nongnu.org/archive/html/qemu-devel/2015-11/msg00855.html
>> https://lists.nongnu.org/archive/html/qemu-devel/2015-11/msg00856.html
>> https://lists.nongnu.org/archive/html/qemu-devel/2015-11/msg00862.html
> 
> Sorry, I managed to miss these.  The first two replies seem good, I'll
> send a few comments on the third now.
> 
>> In particular would you like me to add the comments about MOL to the
>> commit messages?
> 
> But yes, referencing MOL in the messages would be good so we can see
> where this came from in the future.
> 
>> And I wasn't sure from your comments whether you were
>> still looking for someone else more able to test the MacOS stuff?
> 
> Well, that would be nice, but in the absence of any volunteers, I'll
> take the series anyway (provided I don't spot breakage of other
> platforms, obviously).

Thanks a lot for taking another look at these. I've just arrived in work
now (ahhh timezones!) but will aim to get the updated v2 based upon your
comments out either lunchtime or early evening today at the latest.


Many thanks,

Mark.