From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58334) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ag7RV-00077D-HW for qemu-devel@nongnu.org; Wed, 16 Mar 2016 05:08:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ag7RR-0007Lu-4A for qemu-devel@nongnu.org; Wed, 16 Mar 2016 05:08:37 -0400 Received: from e06smtp13.uk.ibm.com ([195.75.94.109]:35723) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ag7RQ-0007LX-RY for qemu-devel@nongnu.org; Wed, 16 Mar 2016 05:08:33 -0400 Received: from localhost by e06smtp13.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 16 Mar 2016 09:08:30 -0000 References: <1457974600-13828-1-git-send-email-clg@fr.ibm.com> <20160315003942.GE15272@voom.fritz.box> <56E7C3B3.3030809@fr.ibm.com> <20160316011914.GA9032@voom> From: =?UTF-8?Q?C=c3=a9dric_Le_Goater?= Message-ID: <56E92283.7020406@fr.ibm.com> Date: Wed, 16 Mar 2016 10:08:19 +0100 MIME-Version: 1.0 In-Reply-To: <20160316011914.GA9032@voom> Content-Type: text/plain; charset=utf-8 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/16/2016 02:19 AM, David Gibson wrote: > On Tue, Mar 15, 2016 at 09:11:31AM +0100, Cédric Le Goater wrote: >> 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. > > So, I'm ok with including (low risk) patches that aren't directly > relevant to 2.6 if they're prereqs for patches that are relevant to > 2.6. After all, reworking the patches isn't risk free either. Please > mention why these patches are being included in the commit messages > though. Sure. >> 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 ? > > It's not really a question of being too large, it's that I'm nervous > about applying patches which touch the core translation code > (e.g. fixes to HV mode tests) during soft freeze if they're not > addressing a bug that's relevant to 2.6. Could you please take a look at these two patches to see if they are relevant for 2.6 ? From my readings, they seem to be the only ones on the edge. 06/17 ppc: Create cpu_ppc_set_papr() helper 11/17 ppc: Initialize AMOR in PAPR mode but it makes sense to take them if we take : 12/17 ppc: Fix writing to AMR/UAMOR (move hunk to 13) 13/17 ppc: Add POWER8 IAMR register (rework hunk) Thanks for the review, C.