From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [STABLE 4.9.y PATCH 0/9] Backport of KVM Speculation Control support Date: Thu, 8 Feb 2018 18:57:38 +0100 Message-ID: <20180208175737.GA22067@kroah.com> References: <1517938181-15317-1-git-send-email-dwmw@amazon.co.uk> <9ea4d52a-7ae6-a179-2f86-1e7b77255387@redhat.com> <1517951146.3677.34.camel@amazon.co.uk> <20180208024959.GA16399@kroah.com> <20180208171436.GA3521@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Cc: "Woodhouse, David" , Jim Mattson , Radim =?utf-8?B?S3LEjW3DocWZ?= , linux-kernel@vger.kernel.org, kvm@vger.kernel.org, KarimAllah Ahmed , stable@vger.kernel.org To: Paolo Bonzini Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On Thu, Feb 08, 2018 at 06:42:03PM +0100, Paolo Bonzini wrote: > On 08/02/2018 18:14, Greg KH wrote: > > On Thu, Feb 08, 2018 at 03:49:59AM +0100, Greg KH wrote: > >> On Tue, Feb 06, 2018 at 09:05:46PM +0000, Woodhouse, David wrote: > >>> > >>> > >>> On Tue, 2018-02-06 at 19:01 +0100, Paolo Bonzini wrote: > >>>> On 06/02/2018 18:29, David Woodhouse wrote: > >>>>> I've put together a linux-4.9.y branch at  > >>>>> http://git.infradead.org/retpoline-stable.git/shortlog/refs/heads/linux-4.9.y > >>>>>   > >>>>> Most of it is fairly straightforward, apart from the IBPB on context  > >>>>> switch for which Tim has already posted a candidate. I wanted some more > >>>>> review on my backports of the KVM bits though, including some extra > >>>>> historical patches I pulled in. > >>>> > >>>> Looks good!  Thanks for the work, > >>>> > >>>> Paolo > >>> > >>> Thanks. In that case, Greg, the full set is lined up in > >>> http://git.infradead.org/retpoline-stable.git/shortlog/refs/heads/linux-4.9.y > >>> or git://git.infradead.org/retpoline-stable linux-4.9.y > >> > >> Many thanks for all of this work. I've now queued up all of these. > > > > There's a problem with the backport of 6342c50ad12e ("KVM: nVMX: > > vmx_complete_nested_posted_interrupt() can't fail") as there is still a > > check in the function that can fail: > > > > vapic_page = kmap(vmx->nested.virtual_apic_page); > > if (!vapic_page) { > > WARN_ON(1); > > return -ENOMEM; > > } > > > > Do we need something else before this patch in order to fix this? I > > guess kmap really can't fail, should I just drop the whole (!vapic_page) > > check? > > Yes, that would be commit 42cf014d38d8822cce63703a467e00f65d000952. > Should David or I respin? No need, I can sneak it into the middle of the series :) I'll do it later tonight and let you know if I have any problems, thanks for pointing out the needed commit. greg k-h