Cmt: [SRU][Xenial][PATCH v2 0/3] Fix corruption on blocked_vcpu_on_cpu list

[Ioanna Alifieraki]

Hi William,

Thanks for the feedback. Even with commit bc22512bb24c(kvm: vmx:
rename vmx_pre/post_block to pi_pre/post_block)
the patches still won't cherry-pick because upstream commit
a0052191624e(kvm: vmx: check apicv is active before using VT-d posted
interrupt) is missing, however the backport
is cleaner and much simpler.
I'm happy to resubmit the patchset with commit bc22512bb24c(kvm: vmx:
rename vmx_pre/post_block to pi_pre/post_block)
included if it is a better option.

On Thu, Jan 7, 2021 at 8:36 AM William Breathitt Gray
<william.gray at canonical.com> wrote:
>
> On Tue, Jan 05, 2021 at 11:54:57PM +0000, Ioanna Alifieraki wrote:
> > BugLink: https://bugs.launchpad.net/bugs/1908428
> >
> > The following 3 patches fix the bug reported in [1].
> > They are backported to apply on 4.4 kernels.
> > Backport is required becasue upstream commit
> > bc22512bb24c(kvm: vmx: rename vmx_pre/post_block to pi_pre/post_block)
> > which renames  vmx_pre/post_block functions to pi_pre/post_block
> > is missing from 4.4.
> > Original patches come from [2] and have been accepted upstream.
> >
> > [1] https://marc.info/?l=kvm&m=149559827906211&w=2
> > [2] https://lore.kernel.org/lkml/20170606105707.23207-1-pbonzini@redhat.com/
> >
> > Paolo Bonzini (3):
> >   KVM: VMX: extract __pi_post_block
> >   KVM: VMX: avoid double list add with VT-d posted interrupts
> >   KVM: VMX: simplify and fix vmx_vcpu_pi_load
> >
> >  arch/x86/kvm/vmx.c | 177 ++++++++++++++++++++++-----------------------
> >  1 file changed, 87 insertions(+), 90 deletions(-)
> >
> > --
> > 2.17.1
>
> Upstream commit
> bc22512bb24c(kvm: vmx: rename vmx_pre/post_block to pi_pre/post_block)
> looks pretty simple. If you pick that up first, then PATCH 1 and PATCH 2
> can be cherry-picks right?
>
> William Breathitt Gray