From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48187) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b8pkw-0001jt-9N for qemu-devel@nongnu.org; Fri, 03 Jun 2016 10:07:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b8pkr-0007Gq-8c for qemu-devel@nongnu.org; Fri, 03 Jun 2016 10:07:21 -0400 References: <1464955880-10176-1-git-send-email-clg@kaod.org> <57518BA1.1030400@ilande.co.uk> <57518D7C.50703@kaod.org> From: Cedric Le Goater Message-ID: <57518EF7.3020200@free.fr> Date: Fri, 3 Jun 2016 16:06:47 +0200 MIME-Version: 1.0 In-Reply-To: <57518D7C.50703@kaod.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?Q?C=c3=a9dric_Le_Goater?= , Mark Cave-Ayland , David Gibson Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org On 06/03/2016 04:00 PM, C=C3=A9dric Le Goater wrote: > Hello Mark, >=20 > On 06/03/2016 03:52 PM, Mark Cave-Ayland wrote: >> On 03/06/16 13:11, C=C3=A9dric 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=C3=A9dric 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=C3=A9dric, >> >> 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 on= ly >> outstanding issue is the removal of the tlb_flush() statements which >> causes Darwin, MacOS X and HelenOS 0.60 to panic on boot >=20 > Is that just booting the CDROM or the complete OS ? because I tried tha= t a=20 > couple of time with ppc-for-2.7-20160531 + the three patches above and=20 > did not see the issue again. I reached the device selection prompt.=20 >=20 > I must be doing something wrong.=20 I was using '-cpu 750' for some reason and this is why it succeeded. It f= ails without. We are getting close.=20 C. >> My only request is if it would be possible to move patch 2 "ppc: Bette= r >> figure out if processor has HV mode" to the front of this patchset whi= ch >> will make the remaining patches bisectable for the Mac machines. With = that: >> >> Tested-by: Mark Cave-Ayland >=20 > Thanks, >=20 > C.=20 >=20 >> Does anyone know if Ben has any ideas as to why the MMU tlb_flush >> changes patch is causing such problems? >> >> >> ATB, >> >> Mark. >> >=20