From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33071) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XMB7E-0001bb-GK for qemu-devel@nongnu.org; Tue, 26 Aug 2014 03:24:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XMB7D-0007ey-C6 for qemu-devel@nongnu.org; Tue, 26 Aug 2014 03:24:28 -0400 Date: Tue, 26 Aug 2014 17:24:42 +1000 From: David Gibson Message-ID: <20140826072442.GA9923@voom.redhat.com> References: <20140825134353.2361.52046.stgit@aravindap> <20140825134526.2361.74648.stgit@aravindap> <20140826053952.GP9923@voom.redhat.com> <1409033740.25772.94.camel@pasglop> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="LG5jgFgJbJFFiAfj" Content-Disposition: inline In-Reply-To: <1409033740.25772.94.camel@pasglop> Subject: Re: [Qemu-devel] [PATCH 2/5] target-ppc: Register and handle HCALL to receive updated RTAS region List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Benjamin Herrenschmidt Cc: qemu-ppc@nongnu.org, Aravinda Prasad , aik@au1.ibm.com, qemu-devel@nongnu.org, paulus@samba.org --LG5jgFgJbJFFiAfj Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Aug 26, 2014 at 04:15:40PM +1000, Benjamin Herrenschmidt wrote: > On Tue, 2014-08-26 at 15:39 +1000, David Gibson wrote: > > On Mon, Aug 25, 2014 at 07:15:26PM +0530, Aravinda Prasad wrote: > > > Receive updates from SLOF about the updated rtas-base. > > > A separate patch for SLOF [1] adds functionality to invoke a > > > a private HCALL whenever OS issues instantiate-rtas with > > > a new rtas-base. > > >=20 > > > This is required as qemu needs to know the updated rtas-base > > > as it allocates error reporting structure in RTAS space upon > > > a machine check exception. > >=20 > > This also seems really awkward. Specifically it seems like a rather > > arbitrary and complex division of what qemu's responsible for and what > > SLOF is responsible for. > >=20 > > Instead I'd suggest that we add an H_INSTANTIATE_RTAS hcall, and we > > move the loading of the spapr-rtas blob from normal reset to when that > > hcall is invoked. >=20 > Beware that SLOF needs to call RTAS for its own reasons... >=20 > So it would be instanciated twice. I don't see that that would cause a big problem, or am I missing something? --=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 --LG5jgFgJbJFFiAfj Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJT/DY6AAoJEGw4ysog2bOSvDEQAOcRUUHpJt/4CjWqTZnuCWEq NpxoOQRBs0XWHUGkElsszCfAdPxXtAmorHAmqDeypbNlhLnoZOkpog6PjFyPRFhy A6QEh6Y0YxaQDoi4f1MX+SYPZVb+8+A0ektlxVWjkBGtcaHTh91UFGWAJ9/lZUfy DQBkf+fbsUIwi4m3cODLocebdl9Eh9af0mCDQhnxGTqD6iGmpsNVqe4Dv8I6wQUx tJFjhgh9i6mcbTWp3ENXnkok8aJRCDQ35ZzBrziCQgg+sfh/pl9s6dI2gTjjuML9 fHGH/KxQPniFhoAPswWLSXSw9H3xYIboZEF20x5BIwiIPJSeStx+SAloEkIYDF0M cmT0Q/MRUVWSybH6RRNbHO/lwhOhQTHmE7NvROA3c7TU7NDRPEOn/bPMjfT/e610 oZi0hLGR2x3qUD+3yb+wmbHA/Ng7uxt/aMT2KRybOro9tp0C35nVlYkemFULn6Gi vqPdKKpJDHQ3KF53g+49x0KpDFusxyMr4V7D1JOO7pcw5DVxto0zTCuaGmHXACjE 5LbruS+d9yPGnUU2KbMNq+vJOeoNbdLuS1JgThOPdhoJF0gqZIsg8igcy+i4xPr5 5tAAbo0Pdpi+oYoxy60xTYhjaNi+F4+i9HeoO/714QEXGToXfc4R9uoL9EPQj+F6 jSBuDK8HJq6xpymUMp0a =Ek3l -----END PGP SIGNATURE----- --LG5jgFgJbJFFiAfj--