From: Greg KH <gregkh@linuxfoundation.org>
To: Marc Zyngier <marc.zyngier@arm.com>
Cc: Christoffer Dall <christoffer.dall@arm.com>, stable@vger.kernel.org
Subject: Re: WTF: patch "[PATCH] arm/arm64: KVM: Add PSCI version selection API" was seriously submitted to be applied to the 4.16-stable tree?
Date: Mon, 30 Apr 2018 08:25:10 -0700 [thread overview]
Message-ID: <20180430152510.GA1674@kroah.com> (raw)
In-Reply-To: <9d5a4f01-c870-6255-3e5f-72869dc0713c@arm.com>
On Mon, Apr 30, 2018 at 04:12:35PM +0100, Marc Zyngier wrote:
> On 30/04/18 16:09, Greg KH wrote:
> > On Mon, Apr 30, 2018 at 04:37:48PM +0200, Christoffer Dall wrote:
> >> On Sun, Apr 29, 2018 at 02:34:45PM +0200, gregkh@linuxfoundation.org wrote:
> >>> The patch below was submitted to be applied to the 4.16-stable tree.
> >>>
> >>> I fail to see how this patch meets the stable kernel rules as found at
> >>> Documentation/process/stable-kernel-rules.rst.
> >>>
> >>> I could be totally wrong, and if so, please respond to
> >>> <stable@vger.kernel.org> and let me know why this patch should be
> >>> applied. Otherwise, it is now dropped from my patch queues, never to be
> >>> seen again.
> >>
> >> This patch ensures that current userspace drivers of KVM VMs will fail
> >> migration to targets that do not support spectre/meltdown mitigations.
> >> Without this patch, VMs can be migrated to hosts that do not have
> >> mitigation support without any warning to the system admin. We
> >> considered this a real security issue as per the stable kernel rules.
> >>
> >> If you disagree, feel free to drop this patch without further
> >> discussion.
> >>
> >> Thanks,
> >> -Christoffer
> >>
> >>>
> >>> ------------------ original commit in Linus's tree ------------------
> >>>
> >>> From 85bd0ba1ff9875798fad94218b627ea9f768f3c3 Mon Sep 17 00:00:00 2001
> >>> From: Marc Zyngier <marc.zyngier@arm.com>
> >>> Date: Sun, 21 Jan 2018 16:42:56 +0000
> >>> Subject: [PATCH] arm/arm64: KVM: Add PSCI version selection API
> >>>
> >>> Although we've implemented PSCI 0.1, 0.2 and 1.0, we expose either 0.1
> >>> or 1.0 to a guest, defaulting to the latest version of the PSCI
> >>> implementation that is compatible with the requested version. This is
> >>> no different from doing a firmware upgrade on KVM.
> >>>
> >>> But in order to give a chance to hypothetical badly implemented guests
> >>> that would have a fit by discovering something other than PSCI 0.2,
> >>> let's provide a new API that allows userspace to pick one particular
> >>> version of the API.
> >>>
> >>> This is implemented as a new class of "firmware" registers, where
> >>> we expose the PSCI version. This allows the PSCI version to be
> >>> save/restored as part of a guest migration, and also set to
> >>> any supported version if the guest requires it.
> >>>
> >>> Cc: stable@vger.kernel.org #4.16
> >>> Reviewed-by: Christoffer Dall <cdall@kernel.org>
> >>> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
> >
> > Also, it looks like this applies cleanly to 4.14.y, do you want it there
> > as well?
> Yes. It is likely that any backport of the Spectre series will want this
> as well.
It worked for 4.14.y, but does not apply at all to 4.9.y, which does
have the ARM spectre fixes in it. If someone could provide a working
backport for there, I would be glad to apply it.
thanks,
greg k-h
prev parent reply other threads:[~2018-04-30 15:25 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-29 12:34 WTF: patch "[PATCH] arm/arm64: KVM: Add PSCI version selection API" was seriously submitted to be applied to the 4.16-stable tree? gregkh
2018-04-30 14:37 ` Christoffer Dall
2018-04-30 15:08 ` Greg KH
2018-04-30 15:09 ` Greg KH
2018-04-30 15:12 ` Marc Zyngier
2018-04-30 15:25 ` Greg KH [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180430152510.GA1674@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=christoffer.dall@arm.com \
--cc=marc.zyngier@arm.com \
--cc=stable@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).