Message-ID: <1500394761.4585.11.camel@decadent.org.uk>
Subject: Re: [PATCH 3.16 121/178] kvm: arm/arm64: Fix locking for
 kvm_free_stage2_pgd
From: Ben Hutchings <ben@decadent.org.uk>
To: Suzuki K Poulose <Suzuki.Poulose@arm.com>,
        linux-kernel@vger.kernel.org, stable@vger.kernel.org
Cc: akpm@linux-foundation.org, Mark Rutland <mark.rutland@arm.com>,
        Paolo Bonzini <pbonzin@redhat.com>,
        Christoffer Dall <cdall@linaro.org>,
        Marc Zyngier <marc.zyngier@arm.com>,
        Christoffer Dall <christoffer.dall@linaro.org>
Date: Tue, 18 Jul 2017 17:19:21 +0100
In-Reply-To: <952e43a7-0073-b068-5ca8-03ec6bb8a363@arm.com>
References: <lsq.1500213406.207858647@decadent.org.uk>
         <952e43a7-0073-b068-5ca8-03ec6bb8a363@arm.com>
Content-Type: multipart/signed; micalg="pgp-sha512";
        protocol="application/pgp-signature"; boundary="=-WoJR3bn9sOTrMm/e6aNP"
Mime-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2064
Lines: 60


--=-WoJR3bn9sOTrMm/e6aNP
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Mon, 2017-07-17 at 16:12 +0100, Suzuki K Poulose wrote:
> On 16/07/17 14:56, Ben Hutchings wrote:
> > 3.16.46-rc1 review patch.=C2=A0=C2=A0If anyone has any objections, plea=
se let me know.
> >
> > ------------------
> >
> > From: Suzuki K Poulose <suzuki.poulose@arm.com>
> >
[...]
> There is a follow up patch for this one to make sure we check/free the PG=
D under the
> mmu_lock.
>=20
> commit 6c0d706b563af73 ("kvm: arm/arm64: Fix race in resetting stage2 PGD=
")
>=20
> So unless you want to fold that in, this looks alright. Looks like we mis=
sed a Cc:
> stable for that. I will send it to stable soon.

That does look like a fix suitable for stable.  So far as I can see,
though, "kvm: arm/arm64: Fix locking for kvm_free_stage2_pgd" does not
introduce a regression - right?  So I don't have to apply the two at
the same time.

Ben.

--=20
Ben Hutchings
Quantity is no substitute for quality, but it's the only one we've got.


--=-WoJR3bn9sOTrMm/e6aNP
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEErCspvTSmr92z9o8157/I7JWGEQkFAlluNQkACgkQ57/I7JWG
EQnjcw/+IwLxuk8BU68S6rk5GXj7FV0OCjGPTvtLLfH1DRWIea7nu1W3t18KzKxl
RMi/po9iYzKeLdcntbtUd253aCaBh/l7zjZJ/fYV5KjLAii6o0jK7DvdsD/a0Uow
OPsuoJ/zjvNWZEg1PLBc/Bcoc4T2zZm5eW6SvluhrXRS6S0/9rEaZ8akguPVAZgz
2xb1LBiKkuyM3EPemANGkjvEM+YUtrVJH43brlUKTeMXwbDSQs0MF/LfKKR1THkr
icC5LVlJwL/R4DLYN3dTDAzf9fnB76bOOIx4JElL1XvvvMofWFfyu8DSc1QaaViE
LJ7QpKM51h3Eg28UxqlQxjngPQ9amk7e7N+9INcr/XuQLj9l420qkWWSnbZpwvm1
P8dNWbQH4IVPDQ+XejtU49V/KoI8/KScCGL9qyifzIHJwcaR7uk3AGHIhmj5hE9y
89tvgGkLxlifyTkP7Y7YyW7p0dhJuubQvt5ByqZXjedKX1BjwuKaFdhLrPRQVv6A
VR4FlsJduYrh9Rk5QGmTUtpmHkGzI41Z8gdY5Fd+ejUHL3oSQce1+GR17YDwhP6b
HlSqPUXpAxOgjGWmnkGVdrYSAuOSFTOeG5Zz4rSCnJBqepzHmLz6r8F+XTMjw99E
B/v+KlbPJPdl58ISFV/DS/ThZECBRO6n63lroXx+cSCHzExUNYY=
=tnkB
-----END PGP SIGNATURE-----

--=-WoJR3bn9sOTrMm/e6aNP--