From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56872) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aVGXC-0000ho-6o for qemu-devel@nongnu.org; Mon, 15 Feb 2016 05:37:42 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aVGX9-0007gt-1D for qemu-devel@nongnu.org; Mon, 15 Feb 2016 05:37:38 -0500 References: <1455194841-4283-1-git-send-email-thuth@redhat.com> <20160212003116.GB2732@voom.fritz.box> From: Thomas Huth Message-ID: <56C1AA5F.3020608@redhat.com> Date: Mon, 15 Feb 2016 11:37:19 +0100 MIME-Version: 1.0 In-Reply-To: <20160212003116.GB2732@voom.fritz.box> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="xDL9x1rkJNthAniHjJ6oDL0k4MNukOdEI" Subject: Re: [Qemu-devel] [PATCH v2 0/4] hw/ppc/spapr: Add "Processor Register Hypervisor Resource Access" H-calls List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Gibson Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --xDL9x1rkJNthAniHjJ6oDL0k4MNukOdEI Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 12.02.2016 01:31, David Gibson wrote: > On Thu, Feb 11, 2016 at 01:47:17PM +0100, Thomas Huth wrote: >> While we were recently debugging a problem with the H_SET_DABR >> call [1], I noticed that some hypercalls from the chapter 14.5.4.3 >> ("Processor Register Hypervisor Resource Access") from the LoPAPR >> spec [2] are still missing in QEMU. >> So here's are some patches that implement these hypercalls. Linux >> apparently does not depend on these hypercalls yet (otherwise somebody= >> would have noticed this earlier), but the hypercalls are rather simple= , >> so I think the implementations are quite straight-forward and easy to >> read. >> >> v2: >> - Don't use set_spr() and set cpu->env.spr[] directly instead >> - Completely reworked the H_PAGE_INIT patch to map the source >> and target pages for higher speed, and to be able to flush now >> the caches if requested. >=20 > I've merged 1-3 into ppc-for-2.6. My only concern with those is > whether we need to be setting some extra flags in the ibm,hypertas > property now that they are implemented. Good point, ... looking at "Table 176. Hypervisor Call Function Table" in LoPAPR, it seems like there are indeed additional function sets that should be signaled with this property. But the current list seems to be in a bad shape, anyway, e.g. "hcall-debug" seems to be missing for the H_LOGICAL_CI_LOAD/STORE function. So I think we best fix this up with a separate patch later (I can submit one once I sorted out the H_PAGE_INIT patch). Thomas --xDL9x1rkJNthAniHjJ6oDL0k4MNukOdEI Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBAgAGBQJWwapqAAoJEC7Z13T+cC21uEwP/31pNviFoamOwau6XnHCTeaQ k0WYU7EX46ClGemicOwR6q3EYATtI96nN65oTCqi3XqXHDgk7eRygcBOAvTny5KP 509uQ5/22ltA7RQtuT0XAlV/04McimLUxOo76Z82aVbrM5igI2dR0zXw4ejZg7mw MF+ci4L8Ep3L1LoNxv+Nb4ZpVG4/TCtfWxR5GQ5Zw+jDxkflv9S/wtObxBzUaxzA KbPW8Xcu3QZ4RXPxVM6sxp+MaeVW/lexs+aOTJ7iswKkbMcKbdNwjmpDb8uqfxq7 1mxAHtmuaNwVuu1QoMLWThjO4IU6wEAdLAmmyBKwdo1j3ZUBPLiMZ52Wj89UxJdr hJJK+fRdMHJxsTN5lfEo70lE9BSnOixe4Q/QyDecpt7aI4A2mCJ5vE8asZ2DKXFF We52SOEo6dA4GPOgbUAIDrNFje7g77knSUiJRvbQLD9mEET9PLas4iwClgG45+IA tpKu3PpDxTmvUlbMRNV2fn5lFycvQQ4THkAGmXes4Ur7jGVhEz+WJSRg0cybin3b qJKXrne9QLA+AU8ItxSJbLQUZJIU+Xv4lBgHdsisTk2O9MI5vH9j/1D7xwTqVAfF sOXjOpr6PV/Qj8Y6dFxAcphgQofUlvhrNyz8YVdn3ZEu+0lJBqJWzeOwl6pAIJf+ jPi1MxryjS7yN5QSeoTJ =Q/BG -----END PGP SIGNATURE----- --xDL9x1rkJNthAniHjJ6oDL0k4MNukOdEI--