On 03/06/16 15:00, Cédric Le Goater wrote: > Hello Mark, > > On 06/03/2016 03:52 PM, Mark Cave-Ayland wrote: >> On 03/06/16 13:11, Cédric Le Goater wrote: >> >>> This is follow up to complete the serie "ppc: preparing pnv landing >>> (round 2)" plus a little fix on instruction privileges. >>> >>> Tested on a POWER8 pserie guest and on mac99. >>> >>> Benjamin Herrenschmidt (2): >>> ppc: Fix hreg_store_msr() so that non-HV mode cannot alter MSR:HV >>> ppc: Better figure out if processor has HV mode >>> >>> Cédric Le Goater (1): >>> ppc: fix hrfid, tlbia and slbia privilege >>> >>> target-ppc/cpu.h | 4 ++++ >>> target-ppc/excp_helper.c | 8 ++++++-- >>> target-ppc/helper_regs.h | 4 ++-- >>> target-ppc/translate.c | 10 ++++++---- >>> target-ppc/translate_init.c | 19 +++++++++++++++---- >>> 5 files changed, 33 insertions(+), 12 deletions(-) >> >> Hi Cédric, >> >> I can confirm that this patchset fixes starting up OpenBIOS for both >> g3beige/mac99 in my tests here. With the escc fix also applied, the only >> outstanding issue is the removal of the tlb_flush() statements which >> causes Darwin, MacOS X and HelenOS 0.60 to panic on boot > > Is that just booting the CDROM or the complete OS ? because I tried that a > couple of time with ppc-for-2.7-20160531 + the three patches above and > did not see the issue again. I reached the device selection prompt. > > I must be doing something wrong.
Hmmm not necessarily. It seems that the Darwin crash only manifests itself on -M g3beige (the default) rather than -M mac99, although I can confirm that HelenOS crashes on both -M g3beige and -M mac99: ./qemu-system-ppc -cdrom HelenOS-0.6.0-ppc32.iso -boot d -bios -m 256 -M mac99 >> My only request is if it would be possible to move patch 2 "ppc: Better >> figure out if processor has HV mode" to the front of this patchset which >> will make the remaining patches bisectable for the Mac machines. With that: >> >> Tested-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> > > Thanks, > > C. ATB, Mark.