From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:46920)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <thuth@redhat.com>) id 1afUeL-0001Tg-9O
	for qemu-devel@nongnu.org; Mon, 14 Mar 2016 11:43:20 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <thuth@redhat.com>) id 1afUeI-00028h-Iw
	for qemu-devel@nongnu.org; Mon, 14 Mar 2016 11:43:17 -0400
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>
	<56E6D076.9030409@fr.ibm.com>
From: Thomas Huth <thuth@redhat.com>
Message-ID: <56E6DC0E.20406@redhat.com>
Date: Mon, 14 Mar 2016 16:43:10 +0100
MIME-Version: 1.0
In-Reply-To: <56E6D076.9030409@fr.ibm.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: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: =?UTF-8?Q?C=c3=a9dric_Le_Goater?= <clg@fr.ibm.com>, qemu-ppc@nongnu.org
Cc: qemu-devel@nongnu.org

On 14.03.2016 15:53, C=C3=A9dric Le Goater wrote:
> Hello Thomas,
>=20
> 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 chur=
n
>> 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 t=
o
>> be synchronized for migration, since the guest can set this register v=
ia
>> the H_SET_MODE hypercall).
>=20
> So shall I just remove the comment :
>=20
>   "We don't give them a KVM reg number yet as no current KVM version
>    supports HV mode."
>=20
> from patch "Add a bunch of hypervisor SPRs to Book3s" after squashing i=
n=20
> "Add KVM numbers to some P8 SPRs" ?

Either that, or change it into something like:

"We don't give them a KVM reg number to most of the registers yet as no
current KVM version supports HV mode. For DAWR and DAWRX, the KVM reg
number is needed since this register can be set by the guest via the
H_SET_MODE hypercall."

?

 Thomas