diff options
| author | Jan Beulich <JBeulich@suse.com> | 2019-05-27 02:45:44 -0600 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2019-06-05 14:14:48 +0200 |
| commit | 5a253552a5108bcabc2eb4e5b2b86262232f17e7 (patch) | |
| tree | 447542558a58e6d45c7f1087c98c0bb42b141f46 /arch/x86/kvm/vmx/nested.c | |
| parent | fb89f4ea7feb1e605f8f405d256c56d8ad69125c (diff) | |
| download | lwn-5a253552a5108bcabc2eb4e5b2b86262232f17e7.tar.gz lwn-5a253552a5108bcabc2eb4e5b2b86262232f17e7.zip | |
x86/kvm/VMX: drop bad asm() clobber from nested_vmx_check_vmentry_hw()
While upstream gcc doesn't detect conflicts on cc (yet), it really
should, and hence "cc" should not be specified for asm()-s also having
"=@cc<cond>" outputs. (It is quite pointless anyway to specify a "cc"
clobber in x86 inline assembly, since the compiler assumes it to be
always clobbered, and has no means [yet] to suppress this behavior.)
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Fixes: bbc0b8239257 ("KVM: nVMX: Capture VM-Fail via CC_{SET,OUT} in nested early checks")
Reviewed-by: Sean Christopherson <sean.j.christopherson@intel.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'arch/x86/kvm/vmx/nested.c')
| -rw-r--r-- | arch/x86/kvm/vmx/nested.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/kvm/vmx/nested.c b/arch/x86/kvm/vmx/nested.c index 1032f068f0b9..0f705c7d590c 100644 --- a/arch/x86/kvm/vmx/nested.c +++ b/arch/x86/kvm/vmx/nested.c @@ -2781,7 +2781,7 @@ static int nested_vmx_check_vmentry_hw(struct kvm_vcpu *vcpu) [launched]"i"(offsetof(struct loaded_vmcs, launched)), [host_state_rsp]"i"(offsetof(struct loaded_vmcs, host_state.rsp)), [wordsize]"i"(sizeof(ulong)) - : "cc", "memory" + : "memory" ); if (vmx->msr_autoload.host.nr) |