[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode
From: |
Mark Cave-Ayland |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode |
Date: |
Fri, 3 Jun 2016 15:06:41 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.8.0 |
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 <address@hidden>
>
> Thanks,
>
> C.
ATB,
Mark.
Re: [Qemu-ppc] [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode, Cédric Le Goater, 2016/06/05
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode, Mark Cave-Ayland, 2016/06/05
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode, Cédric Le Goater, 2016/06/06
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode, Cedric Le Goater, 2016/06/06
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode, Mark Cave-Ayland, 2016/06/06
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode, Cédric Le Goater, 2016/06/07