On Fri, Jan 25, 2019 at 7:41 AM Sean Christopherson
<sean.j.christopherson@xxxxxxxxx> wrote:
>
> Unlike the primary vCPU-run flow, the nested early checks code doesn't
> actually want to propagate VM-Fail back 'vmx'. Yay copy+paste.
>
> In additional to eliminating the need to clear vmx->fail before
> returning, using a local boolean also drops a reference to 'vmx' in the
> asm blob. Dropping the reference to 'vmx' will save a register in the
> long run as future patches will shift all pointer references from 'vmx'
> to 'vmx->loaded_vmcs'.
>
> Fixes: 52017608da33 ("KVM: nVMX: add option to perform early consistency checks via H/W")
> Signed-off-by: Sean Christopherson <sean.j.christopherson@xxxxxxxxx>
Reviewed-by: Jim Mattson <jmattson@xxxxxxxxxx>
