From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Gibson Subject: Re: [kvm-unit-tests PATCH 11/14] powerpc/ppc64: add rtas_power_off Date: Wed, 5 Aug 2015 10:12:27 +1000 Message-ID: <20150805001227.GD7739@voom.redhat.com> References: <1438612891-3718-1-git-send-email-drjones@redhat.com> <1438612891-3718-12-git-send-email-drjones@redhat.com> <20150804041130.GF3080@voom.redhat.com> <20150804075444.GF13384@hawk.localdomain> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="bjuZg6miEcdLYP6q" Cc: kvm@vger.kernel.org, kvm-ppc@vger.kernel.org, dgibson@redhat.com, agraf@suse.de, thuth@redhat.com, lvivier@redhat.com, pbonzini@redhat.com To: Andrew Jones Return-path: Received: from ozlabs.org ([103.22.144.67]:48962 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753105AbbHEAOL (ORCPT ); Tue, 4 Aug 2015 20:14:11 -0400 Content-Disposition: inline In-Reply-To: <20150804075444.GF13384@hawk.localdomain> Sender: kvm-owner@vger.kernel.org List-ID: --bjuZg6miEcdLYP6q Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Aug 04, 2015 at 09:54:44AM +0200, Andrew Jones wrote: > On Tue, Aug 04, 2015 at 02:11:30PM +1000, David Gibson wrote: > > On Mon, Aug 03, 2015 at 04:41:28PM +0200, Andrew Jones wrote: > > > Add enough RTAS support to support power-off, and apply it to > > > exit(). > > >=20 > > > Signed-off-by: Andrew Jones > > > --- > > > lib/powerpc/asm/rtas.h | 27 ++++++++++++++++ > > > lib/powerpc/io.c | 3 ++ > > > lib/powerpc/rtas.c | 84 +++++++++++++++++++++++++++++++++++++++= ++++++++++ > > > lib/ppc64/asm/rtas.h | 1 + > > > powerpc/Makefile.common | 1 + > > > powerpc/cstart64.S | 9 ++++++ > > > 6 files changed, 125 insertions(+) > > > create mode 100644 lib/powerpc/asm/rtas.h > > > create mode 100644 lib/powerpc/rtas.c > > > create mode 100644 lib/ppc64/asm/rtas.h > > >=20 > > > diff --git a/lib/powerpc/asm/rtas.h b/lib/powerpc/asm/rtas.h > > > new file mode 100644 > > > index 0000000000000..53ce126c69a42 > > > --- /dev/null > > > +++ b/lib/powerpc/asm/rtas.h > > > @@ -0,0 +1,27 @@ > > > +#ifndef _ASMPOWERPC_RTAS_H_ > > > +#define _ASMPOWERPC_RTAS_H_ > > > +/* > > > + * Copyright (C) 2015, Red Hat Inc, Andrew Jones > > > + * > > > + * This work is licensed under the terms of the GNU LGPL, version 2. > > > + */ > > > +#include > > > + > > > +#define RTAS_UNKNOWN_SERVICE (-1) > > > + > > > +struct rtas_args { > > > + u32 token; > > > + u32 nargs; > > > + u32 nret; > > > + u32 args[16]; > > > + u32 *rets; > > > +}; > > > + > > > +extern void rtas_init(void); > > > +extern void enter_rtas(unsigned long args); > > > +extern int rtas_token(const char *service); > > > +extern int rtas_call(int token, int nargs, int nret, int *outputs, .= =2E.); > > > + > > > +extern void rtas_power_off(void); > > > + > > > +#endif /* _ASMPOWERPC_RTAS_H_ */ > > > diff --git a/lib/powerpc/io.c b/lib/powerpc/io.c > > > index a7eaafeca9205..25cdd0ad961ee 100644 > > > --- a/lib/powerpc/io.c > > > +++ b/lib/powerpc/io.c > > > @@ -7,6 +7,7 @@ > > > */ > > > #include > > > #include > > > +#include > > > =20 > > > extern void halt(int code); > > > extern void putchar(int c); > > > @@ -15,6 +16,7 @@ static struct spinlock uart_lock; > > > =20 > > > void io_init(void) > > > { > > > + rtas_init(); > > > } > > > =20 > > > void puts(const char *s) > > > @@ -27,5 +29,6 @@ void puts(const char *s) > > > =20 > > > void exit(int code) > > > { > > > + rtas_power_off(); > > > halt(code); > > > } > > > diff --git a/lib/powerpc/rtas.c b/lib/powerpc/rtas.c > > > new file mode 100644 > > > index 0000000000000..39f7d4428ee77 > > > --- /dev/null > > > +++ b/lib/powerpc/rtas.c > > > @@ -0,0 +1,84 @@ > > > +/* > > > + * powerpc RTAS > > > + * > > > + * Copyright (C) 2015, Red Hat Inc, Andrew Jones > > > + * > > > + * This work is licensed under the terms of the GNU LGPL, version 2. > > > + */ > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > + > > > +static struct spinlock lock; > > > +struct rtas_args rtas_args; > > > +static int rtas_node; > > > + > > > +void rtas_init(void) > > > +{ > > > + if (!dt_available()) { > > > + printf("%s: No device tree!\n", __func__); > > > + abort(); > > > + } > > > + > > > + rtas_node =3D fdt_path_offset(dt_fdt(), "/rtas"); > > > + if (rtas_node < 0) { > > > + printf("%s: /rtas node: %s\n", __func__, > > > + fdt_strerror(rtas_node)); > > > + abort(); > > > + } > > > +} > > > + > > > +int rtas_token(const char *service) > > > +{ > > > + const struct fdt_property *prop; > > > + u32 *token; > > > + > > > + prop =3D fdt_get_property(dt_fdt(), rtas_node, service, NULL); > >=20 > > Oh.. one other thing here. > >=20 > > This is only safe if you never alter the device tree blob you're given > > (or at least, not after rtas_init()). That may well be the case, but > > I don't know your code well enough to be sure. > >=20 > > Otherwise, the rtas node could get moved by other dt changes, meaning > > the offset stored in rtas_node is no longer valid. >=20 > I hadn't planned on modifying the DT, but if you can conceive of a test > they may want to, then we should do this right. That's probably ok then, as long as it's made clear that you can't mess with the DT. Or if tests might want to modify it, you could give them their own copy of it. >I guess doing it right > just means to hunt down rtas_node each time, that's easy. That's right. Not having persistent handles is a bit of a pain, but it's the necessary tradeoff to work with the tree in flattened form without having some complicated pile of "context" state to go with it. (Attempting to remind people that these aren't persistent handles is, incidentally, why so many of the libfdt functions explicitly mention "offset" in their names). --=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 --bjuZg6miEcdLYP6q Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJVwVTrAAoJEGw4ysog2bOSI9AQAL7grgANcdMJuqMjWULZmr3g EqiY1iNGQt++s+xuKIpY4annoZhfv5xmD0nTRvODbZTm9QEIDwLpAtTDx8e+6E7p QU1TADvO7jSPaMzRwIxluPNW9bG6/DALULXH8gR9h6qCNmtZ2EHQkIwNalmR2OT+ SLSzQNT4E62r25pQaIav6HZq2cetjtu5/dqrECgHNiGgZzVIL7wWu9Rkhj19Est2 tEmrtkxdWcj+b/+eTmFfp6cN3r/AvWgM4y5x00mgzJgbTk7oS2JpoEh99jlIuNSJ WnZpTGAOs7Gn/fepYv+75VDlQWW9qdaXZ51bLm+gKimoHAEEoxomkFMt+Qa0dI0g q1GvdOfF6OIuu/MkDNyJ4oCShF0rFL5EIehXZqUyJOrWREX/Wx7L2Z/CcTztiJM9 42f7idqfrac8mAhnyonM3G5kloL4krCtR5kabbUcVnmNtxB49/XICITglInvl3Z8 0eH3bSA1hQ9hufG6XNcBDvQDkjFBDESSv9NQYys9oeJIPCGFZML05726vaI38PYk pzQKMg1lxNVNjaXGatLiuw/N7hBHU7fPRnchSbeLqbIw5HK0RHJgoECojYmJlEXE n1QsL5dbPAUbJtcIsamiR88CX8wUarZE0uUtGNLW9MxwfyLufXpVgqL+bBVnLdGF +ZTXrpfTiVvS7Vkm/lc2 =8KaV -----END PGP SIGNATURE----- --bjuZg6miEcdLYP6q--