From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33620) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1afTsd-0005xc-2d for qemu-devel@nongnu.org; Mon, 14 Mar 2016 10:54:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1afTsY-00060Z-NN for qemu-devel@nongnu.org; Mon, 14 Mar 2016 10:53:59 -0400 Received: from e06smtp09.uk.ibm.com ([195.75.94.105]:47131) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1afTsY-00060I-DN for qemu-devel@nongnu.org; Mon, 14 Mar 2016 10:53:54 -0400 Received: from localhost by e06smtp09.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 14 Mar 2016 14:53:51 -0000 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> <56E1B665.5040909@redhat.com> <56E1F4E9.5050307@fr.ibm.com> <56E29820.5000500@redhat.com> <56E2D48B.4070509@fr.ibm.com> <56E2DA46.6080702@redhat.com> From: =?UTF-8?Q?C=c3=a9dric_Le_Goater?= Message-ID: <56E6D076.9030409@fr.ibm.com> Date: Mon, 14 Mar 2016 15:53:42 +0100 MIME-Version: 1.0 In-Reply-To: <56E2DA46.6080702@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit 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: Thomas Huth , qemu-ppc@nongnu.org Cc: qemu-devel@nongnu.org Hello Thomas, On 03/11/2016 03:46 PM, Thomas Huth wrote: > I'd maybe also squash the "Add KVM numbers to some P8 SPRs" patch into > the "Add a bunch of hypervisor SPRs to Book3s" to reduce the code churn > a little bit (note that the patch description for the "Add a bunch of > hypervisor SPRs to Book3s" patch is misleading - DAWR and DAWRX need to > be synchronized for migration, since the guest can set this register via > the H_SET_MODE hypercall). So shall I just remove the comment : "We don't give them a KVM reg number yet as no current KVM version supports HV mode." from patch "Add a bunch of hypervisor SPRs to Book3s" after squashing in "Add KVM numbers to some P8 SPRs" ? Thanks, C.