From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33207) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ae4ti-0002M7-EP for qemu-devel@nongnu.org; Thu, 10 Mar 2016 13:01:19 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ae4te-00043R-8Y for qemu-devel@nongnu.org; Thu, 10 Mar 2016 13:01:18 -0500 References: <1447201710-10229-1-git-send-email-benh@kernel.crashing.org> <1447201710-10229-73-git-send-email-benh@kernel.crashing.org> <56D74D5E.9090307@redhat.com> <56E081E1.5020003@fr.ibm.com> <56E092DE.5040101@redhat.com> From: Thomas Huth Message-ID: <56E1B665.5040909@redhat.com> Date: Thu, 10 Mar 2016 19:01:09 +0100 MIME-Version: 1.0 In-Reply-To: <56E092DE.5040101@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 72/77] ppc: A couple more dummy POWER8 Book4 regs List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?Q?C=c3=a9dric_Le_Goater?= , Benjamin Herrenschmidt , qemu-ppc@nongnu.org Cc: qemu-devel@nongnu.org On 09.03.2016 22:17, Thomas Huth wrote: > On 09.03.2016 21:04, C=C3=A9dric Le Goater wrote: .... >> I have been maintaining a port of Ben's patchset on the latest qemu fo= r other=20 >> parts which should come after pnv is merged so I have a framework to t= est such=20 >> sub-patchsets. I also have time to work on them but clearly not the ex= pertise >> in all areas ! >=20 > That would be great if you could take care of this! >=20 >> What would be nice is to identify the most obvious ones, non controver= sial >> that could be merged after a few iterations. I have a vague idea, the = ones=20 >> Reviewed-by David obviously being good candidates, the definition of n= ew SPRs=20 >> (even the dummy ones ?). >=20 > I really like to see the KVM SPRs patches first - since they are fixing > potential problems with migration of the _current_ KVM machines already= ! > And being bug fixes, maybe these patches could even be included for QEM= U > 2.6 already? (i.e. before the hard freeze at the end of March) >=20 > So my wish-list for a first small patch series looks like this: >=20 > 5b287e66c7513209 ppc: Add macros to register hypervisor mode SPRs > 34f1af75e75e7ba0 ppc: Add dummy CIABR SPR > 48adf38e9cab4663 ppc: A couple more dummy POWER8 Book4 regs > 730a9b4dc9414818 ppc: Add KVM numbers to some P8 SPRs >=20 > There are a couple of other patches touching the SPRs initialization, > but they are not important with regards to migration... so not sure > whether it makes sense to include them now already... FWIW, I just saw today (by doing some more experiments with kvm-unit-tests) that the IAMR register is also not migrated yet ... so it would be nice if you could include the related patches for IAMR, too, and wire the KVM part up with KVM_REG_PPC_IAMR... Thomas