Re: [PATCH] KVM: VMX: check match table

From: Nick Desaulniers
Date: Tue Oct 03 2017 - 22:54:28 EST


On Tue, Oct 03, 2017 at 11:42:18AM +0200, Paolo Bonzini wrote:
> On 01/10/2017 01:22, Nick Desaulniers wrote:
> > I don't follow (but I also don't know what any of these three letter
> > acryonyms acronyms stand for), does svm depend on vmx or vice-versa?
> Neither, one is Intel (VMX), the other is AMD (SVM).

Oh, neat, did not realize the vendors did not have different names for
their virtualization extensions.

https://rtfmp.com/2016/03/21/what-does-vmx-svm-cpu-flags-mean/

> Would this work for you?

It doesn't apply cleanly on Linus' tree, or the KVM tree master branch,
so I couldn't fully test it. But it does look like it will do the
trick.

> And again, is this only with clang?

Indeed the warning was coming from Clang, but looks like some
additional cleanup was done, which is good.

Reviewed-by: Nick Desaulniers <nick.desaulniers@xxxxxxxxx>