Re: [Qemu-devel] [PATCH RFC 0/4] Mac OS 9 compatibility improvements

[Mark Cave-Ayland]

On 22/08/15 16:32, Cormac O'Brien wrote:

> This patchset greatly improves QEMU's ability to run Mac OS 9 on the mac99
> machine, allowing progress past the bootloader and partial progress through
> the load screen.
> 
> Signed-off-by: Cormac O'Brien <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
> 
> Cormac O'Brien (1):
>   PPC: fix CUDA packet header size
> 
>  hw/input/adb.c          |  2 +-
>  hw/misc/macio/cuda.c    | 54 
> ++++++++++++++++++++++++++++---------------------
>  hw/ppc/mac_newworld.c   |  3 ++-
>  target-ppc/mem_helper.c |  5 +++--
>  target-ppc/translate.c  |  2 +-
>  5 files changed, 38 insertions(+), 28 deletions(-)

Hi Cormac,

I can confirm here that in combination with an updated OpenBIOS then the
patchset gets qemu-system-ppc -M mac99 to the OS 9 loading screen, which
is great progress!

When booting OS 9 with this patchset I do see the following output on
the console:

Trying to write invalid spr 0 (0x000) at 00f113c0
Trying to read invalid spr 0 (0x000) at 00f113c8
Trying to write privileged spr 955 (0x3bb) at 00f164b8
Trying to write invalid spr 959 (0x3bf) at 00f16520
Trying to read invalid spr 959 (0x3bf) at 00f16528
Trying to write privileged spr 955 (0x3bb) at 00f164b8
Trying to write invalid spr 959 (0x3bf) at 00f16520
Trying to read invalid spr 959 (0x3bf) at 00f16528

I suspect that this could be similar to commit 1ca448b in that OS 9 is
trying to read/write undefined bits, but someone with more PPC knowledge
should be able to confirm whether or not this is harmless or whether an
additional patch is required.


ATB,

Mark.