From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48815) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ahoMx-0001DU-Sm for qemu-devel@nongnu.org; Sun, 20 Mar 2016 21:10:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ahoMw-0000Oc-Mj for qemu-devel@nongnu.org; Sun, 20 Mar 2016 21:10:55 -0400 Date: Mon, 21 Mar 2016 12:00:39 +1100 From: David Gibson Message-ID: <20160321010039.GA23586@voom.redhat.com> References: <1458134034-32500-1-git-send-email-clg@fr.ibm.com> <1458134034-32500-4-git-send-email-clg@fr.ibm.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="T4sUOijqQbZv57TR" Content-Disposition: inline In-Reply-To: <1458134034-32500-4-git-send-email-clg@fr.ibm.com> Subject: Re: [Qemu-devel] [PATCH v2 03/10] ppc: Add a bunch of hypervisor SPRs to Book3s List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?iso-8859-1?Q?C=E9dric?= Le Goater Cc: Thomas Huth , qemu-ppc@nongnu.org, qemu-devel@nongnu.org --T4sUOijqQbZv57TR Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Mar 16, 2016 at 02:13:47PM +0100, C=E9dric Le Goater wrote: > From: Benjamin Herrenschmidt >=20 > 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. >=20 > Signed-off-by: Benjamin Herrenschmidt > [clg: squashed in patch 'ppc: Add KVM numbers to some P8 SPRs' > changed the commit log with a proposal of Thomas Huth > removed all hunks except those related to AMOR and DAWR* ] > Signed-off-by: C=E9dric Le Goater Reviewed-by: David Gibson > --- > target-ppc/translate_init.c | 21 +++++++++++++++++++++ > 1 file changed, 21 insertions(+) >=20 > diff --git a/target-ppc/translate_init.c b/target-ppc/translate_init.c > index 6a11b41206e5..4cb3dd5076c1 100644 > --- a/target-ppc/translate_init.c > +++ b/target-ppc/translate_init.c > @@ -1105,6 +1105,11 @@ static void gen_spr_amr (CPUPPCState *env) > SPR_NOACCESS, SPR_NOACCESS, > &spr_read_generic, &spr_write_generic, > KVM_REG_PPC_UAMOR, 0); > + spr_register_hv(env, SPR_AMOR, "AMOR", > + SPR_NOACCESS, SPR_NOACCESS, > + SPR_NOACCESS, SPR_NOACCESS, > + &spr_read_generic, &spr_write_generic, > + 0); > #endif /* !CONFIG_USER_ONLY */ > } > #endif /* TARGET_PPC64 */ > @@ -7491,6 +7496,20 @@ static void gen_spr_book3s_dbg(CPUPPCState *env) > KVM_REG_PPC_DABRX, 0x00000000); > } > =20 > +static void gen_spr_book3s_207_dbg(CPUPPCState *env) > +{ > + spr_register_kvm_hv(env, SPR_DAWR, "DAWR", > + SPR_NOACCESS, SPR_NOACCESS, > + SPR_NOACCESS, SPR_NOACCESS, > + &spr_read_generic, &spr_write_generic, > + KVM_REG_PPC_DAWR, 0x00000000); > + spr_register_kvm_hv(env, SPR_DAWRX, "DAWRX", > + SPR_NOACCESS, SPR_NOACCESS, > + SPR_NOACCESS, SPR_NOACCESS, > + &spr_read_generic, &spr_write_generic, > + KVM_REG_PPC_DAWRX, 0x00000000); > +} > + > static void gen_spr_970_dbg(CPUPPCState *env) > { > /* Breakpoints */ > @@ -7960,6 +7979,8 @@ static void init_proc_book3s_64(CPUPPCState *env, i= nt version) > } > if (version < BOOK3S_CPU_POWER8) { > gen_spr_book3s_dbg(env); > + } else { > + gen_spr_book3s_207_dbg(env); > } > #if !defined(CONFIG_USER_ONLY) > switch (version) { --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --T4sUOijqQbZv57TR Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJW70e3AAoJEGw4ysog2bOS1eQP/RmHM6g+ZHqMVAaHYXKXE0xh fiPhIG713hUteFIlcIO0Su0aNHUppcJS+Lsb1iwH78cxH7W8w7F6R9IkHLgymreW 8LR1tfmrtRO2ET4q52gnhzkVG2QhTht8mW1mq3heIMWG7fF0yU6incoCVPqams6Q Ioyi3orHpBnsevSV0G7TteOJDVCQhQR6zAfT1WnkwHDiBy/xAUoal1VYKqZYnc5V nxVgW0taiRxKP8SI1E8niQxHjKTcoDMjD5Jm9HTmlxTJBMOWJ3f9nmbgZB8I5HOe qbmhibYp4ck6vF580ov0CLdbTrp+ebBsK8uSpE99icXKjbgkW7dznWhXgprDI0RS RnDohzLmIUMLVC1NxLA9g/D5iWNGinXrWa72EAvsYVhnuHt+LerNtZCBSkilkD+c ccxwTKRvvqFKStt6t7HxVipW8mtXgkYEGrns5AgNABrGspuKTGfXZdHwiz9xhYyc lQaTy2Gobp7wE2ojm0NcKQpqhTRM+TceVDjAa0V1Fbk/2HoImsQDiRT3g+/de4Aw mdRf6kj6lWg59N20yNf1y/EDbMgYgzxJMcz1QQmvUxwy/tG4SXbnUKPyTSHAHQi5 2t6iQ70W1W4j4eHH7ryUKnKrZ3UQVLTxQ+GYEZ1gdslw+hUBySBEwk8LMIWwVIlV psb4/LSEsqsLEXvu8X9I =epOu -----END PGP SIGNATURE----- --T4sUOijqQbZv57TR--