From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42555) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bT1lf-00065C-1k for qemu-devel@nongnu.org; Fri, 29 Jul 2016 02:59:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bT1ld-0005Hp-TP for qemu-devel@nongnu.org; Fri, 29 Jul 2016 02:59:35 -0400 Date: Fri, 29 Jul 2016 15:42:22 +1000 From: David Gibson Message-ID: <20160729054222.GS2588@voom.fritz.box> References: <1469571686-7284-1-git-send-email-benh@kernel.crashing.org> <1469571686-7284-28-git-send-email-benh@kernel.crashing.org> <76968d9e-1b45-538e-a400-9069eb72f42e@twiddle.net> <1469758381.5978.241.camel@kernel.crashing.org> <20160729033456.GI2588@voom.fritz.box> <1469767204.5978.244.camel@kernel.crashing.org> <1469768287.5978.251.camel@kernel.crashing.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="wnBGVoaGQwxWUIo6" Content-Disposition: inline In-Reply-To: <1469768287.5978.251.camel@kernel.crashing.org> Subject: Re: [Qemu-devel] [PATCH 28/32] ppc: Avoid double translation for lvx/lvxl/stvx/stvxl List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Benjamin Herrenschmidt Cc: Richard Henderson , qemu-ppc@nongnu.org, qemu-devel@nongnu.org --wnBGVoaGQwxWUIo6 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Jul 29, 2016 at 02:58:07PM +1000, Benjamin Herrenschmidt wrote: > On Fri, 2016-07-29 at 14:40 +1000, Benjamin Herrenschmidt wrote: > > On Fri, 2016-07-29 at 13:34 +1000, David Gibson wrote: > > >=20 > > > =A0 > > > What should I do with this in the short term?=A0=A0Leave it in > > > ppc-for-2.8, or remove it for now pending possible changes? > >=20 > > I think I'm still measuring a performance improvement with this, I'll > > test a bit more and will get back to you. > >=20 > > It will definitely better to do it without a helper once Richard's 128- > > bit stuff is in. >=20 > Ok, after I nice'd myself on top of all the gcc's on that test machine > I confirm that this patch is a loss on qemu-user. It might still be a > win for qemu-system-ppc64 due to the translation but probably not > worth bothering. >=20 > So drop this one. I'll use the 128-bit support when it becomes > available. Done. --=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 --wnBGVoaGQwxWUIo6 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJXmuy+AAoJEGw4ysog2bOSb1YP/0BQn8JVJwKPJM/ISz8PpKu+ oeVibCHFSmJZdPzM3/VwbTfcRUnE/Gx9N9y8F3VD+xl0CumCORqmcfPf3YtpigKH 8PamqY68Di4hH5RFsPwhHNvaPXY8u8CV7/Ye8gfNW3hmfZWL5WtmhvuleLVuKaj6 GAcJWHQP8t68wS1oJCSePUD+YKqT13oFuMsNsVDxNl/8uHIuF+7wQMI5+ph0aTXB kT2zvlh0QnHB1XUW6+hA30rC7zGWMaFnTQGR2XOBGuNw4MAsrF7eJCn3I15rwEd4 yWm9haKNb9J970h4sr04Tj1Yd1GupxlgZyfVp6Ucw473d27Em8G3578E6+4yv7oC ZDXdj5K2zbA0M/Y1ROx+PgoXF/4wjhxA1zB/qwB+uuqLn2HEHSIz/vtb8+evUKuj TFrgTvz3yFdK/kcKkYfOrvUYxOrS24upRSpXZua1aGeL99/rXQvAC1/LNTG6ozwr xSpxNj1H1+h1ap7DxsY43A8x9kY5zin/S6RVfy7Yt9AYKKW8bySpo7g00Q8U6jqg QV37dyWm5YasIFl9ubTV1WvjKTmljiD4ssoIwThDwt897DwEToRIpykisKA2FE1r fyPnOqTM4ZbpC4JHeo4eawIOagUgeI3THeum0qV0xIjwRh+AN4cuIBC9Lygrbp2W abg7YGV2hpfA5PuNzL3p =6CZI -----END PGP SIGNATURE----- --wnBGVoaGQwxWUIo6--