From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44891) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b8pXT-0001f1-Am for qemu-devel@nongnu.org; Fri, 03 Jun 2016 09:53:28 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b8pXQ-0002gK-5a for qemu-devel@nongnu.org; Fri, 03 Jun 2016 09:53:27 -0400 References: <1464955880-10176-1-git-send-email-clg@kaod.org> From: Mark Cave-Ayland Message-ID: <57518BA1.1030400@ilande.co.uk> Date: Fri, 3 Jun 2016 14:52:33 +0100 MIME-Version: 1.0 In-Reply-To: <1464955880-10176-1-git-send-email-clg@kaod.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit 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?= , David Gibson Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org 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 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 Does anyone know if Ben has any ideas as to why the MMU tlb_flush changes patch is causing such problems? ATB, Mark.