Re: [PATCH v2 0/3] KVM: SVM: Fix GATag bug for >256 vCPUs

From: Sean Christopherson
Date: Wed Feb 15 2023 - 17:11:26 EST


On Thu, Feb 16, 2023, Suthikulpanit, Suravee wrote:
>
>
> On 2/7/2023 7:21 AM, Sean Christopherson wrote:
> > Fix a bug in KVM's use of the GATag where it unintentionally drops a bit
> > from vCPU IDs greater than 255 and as a result wakes the wrong vCPU.
> >
> > Suravee and/or Alejandro, can you give this proper testing? It's compile
> > tested only at this point. I'll do basic testing before officially
> > applying, but AFAIK I don't have access to x2AVIC hardware, nor do I have
> > a ready-to-go configuration to properly exercise this code.
> >
> > Thanks!
> >
> > Sean Christopherson (2):
> > KVM: SVM: Fix a benign off-by-one bug in AVIC physical table mask
> > KVM: SVM: WARN if GATag generation drops VM or vCPU ID information
> >
> > Suravee Suthikulpanit (1):
> > KVM: SVM: Modify AVIC GATag to support max number of 512 vCPUs
> >
> > arch/x86/include/asm/svm.h | 12 +++++++-----
> > arch/x86/kvm/svm/avic.c | 37 ++++++++++++++++++++++++++++---------
> > 2 files changed, 35 insertions(+), 14 deletions(-)
> >
> >
> > base-commit: 32e69f232db4ca11f26e5961daeff93906ce232f
>
> For the series:
>
> Tested-by: Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>

Thanks much!

Paolo, do you want to grab this directly, or should I throw this in next for a
few days and send a separate pull request? Or do something else entirely?