From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50164) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1afk4y-0000ie-3K for qemu-devel@nongnu.org; Tue, 15 Mar 2016 04:11:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1afk4u-0001xB-Oe for qemu-devel@nongnu.org; Tue, 15 Mar 2016 04:11:48 -0400 Received: from e06smtp05.uk.ibm.com ([195.75.94.101]:42357) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1afk4u-0001wY-Fw for qemu-devel@nongnu.org; Tue, 15 Mar 2016 04:11:44 -0400 Received: from localhost by e06smtp05.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 15 Mar 2016 08:11:42 -0000 References: <1457974600-13828-1-git-send-email-clg@fr.ibm.com> <20160315003942.GE15272@voom.fritz.box> From: =?UTF-8?Q?C=c3=a9dric_Le_Goater?= Message-ID: <56E7C3B3.3030809@fr.ibm.com> Date: Tue, 15 Mar 2016 09:11:31 +0100 MIME-Version: 1.0 In-Reply-To: <20160315003942.GE15272@voom.fritz.box> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH 00/17] ppc: preparing pnv landing List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Gibson Cc: Thomas Huth , qemu-ppc@nongnu.org, qemu-devel@nongnu.org On 03/15/2016 01:39 AM, David Gibson wrote: > On Mon, Mar 14, 2016 at 05:56:23PM +0100, Cédric Le Goater wrote: >> Hello, >> >> This is a first mini-serie of patches adding support for new ppc SPRs. >> They were taken from Ben's larger patchset adding the ppc powernv >> platform and they should already be useful for the pseries guest >> migration. >> >> Initial patches come from : >> >> https://github.com/ozbenh/qemu/commits/powernv >> >> The changes are mostly due to the rebase on Dave's 2.6 branch: >> >> https://github.com/dgibson/qemu/commits/ppc-for-2.6 >> >> A couple more are bisect and checkpatch fixes and finally some patches >> were merge to reduce the noise. >> >> >> >> The patchset is also available here: >> >> https://github.com/legoater/qemu/commits/for-2.6 >> >> It was quickly tested with a pseries guest using KVM and TCG. > > Hmm.. do these all fix bugs with migration, or only some of them? Probably only some. Initially, Thomas gave a shorter list which I expanded to a larger one because of dependencies between patches and I didn't want to change too much what Ben had sent. You had also reviewed a few. > The relevance is that things to fix migration should go into 2.6, but > preparation work for powernv that doesn't fix bug shouldn't really be > going in now, after the soft freeze and will need to wait for 2.7. OK. I will rework and keep the rest for 2.7. Thomas, thanks for the review. I have identified a few things I need to work on but may be, the patchset is still too large for 2.6 ? Full list is below. Thanks, C. >> Benjamin Herrenschmidt (17): >> ppc: Update SPR definitions >> ppc: Add macros to register hypervisor mode SPRs >> ppc: Add a bunch of hypervisor SPRs to Book3s >> ppc: Add number of threads per core to the processor definition >> ppc: Fix hreg_store_msr() so that non-HV mode cannot alter MSR:HV >> ppc: Create cpu_ppc_set_papr() helper >> ppc: Better figure out if processor has HV mode >> ppc: Add placeholder SPRs for DPDES and DHDES on P8 >> ppc: SPURR & PURR are HV writeable and privileged >> ppc: Add dummy SPR_IC for POWER8 >> ppc: Initialize AMOR in PAPR mode >> ppc: Fix writing to AMR/UAMOR >> ppc: Add POWER8 IAMR register >> ppc: Add dummy write to VTB >> ppc: Add dummy POWER8 MPPR register >> ppc: Add dummy CIABR SPR >> ppc: A couple more dummy POWER8 Book4 regs >> >> hw/ppc/spapr.c | 11 +- >> target-ppc/cpu-qom.h | 1 + >> target-ppc/cpu.h | 68 ++++++- >> target-ppc/excp_helper.c | 8 +- >> target-ppc/helper_regs.h | 4 +- >> target-ppc/translate.c | 30 +-- >> target-ppc/translate_init.c | 461 ++++++++++++++++++++++++++++++++++++++++---- >> 7 files changed, 510 insertions(+), 73 deletions(-) >> >