提交 d2477826 编写于 作者: J Joerg Roedel 提交者: Avi Kivity

KVM: SVM: Return correct values in nested_svm_exit_handled_msr

The nested_svm_exit_handled_msr() returned an bool which is
a bug. I worked by accident because the exected integer
return values match with the true and false values. This
patch changes the return value to int and let the function
return the correct values.
Signed-off-by: NJoerg Roedel <joerg.roedel@amd.com>
Signed-off-by: NMarcelo Tosatti <mtosatti@redhat.com>
上级 0fc5c3a5
...@@ -1552,16 +1552,16 @@ static void nested_svm_unmap(struct page *page) ...@@ -1552,16 +1552,16 @@ static void nested_svm_unmap(struct page *page)
kvm_release_page_dirty(page); kvm_release_page_dirty(page);
} }
static bool nested_svm_exit_handled_msr(struct vcpu_svm *svm) static int nested_svm_exit_handled_msr(struct vcpu_svm *svm)
{ {
u32 param = svm->vmcb->control.exit_info_1 & 1; u32 param = svm->vmcb->control.exit_info_1 & 1;
u32 msr = svm->vcpu.arch.regs[VCPU_REGS_RCX]; u32 msr = svm->vcpu.arch.regs[VCPU_REGS_RCX];
bool ret = false;
u32 t0, t1; u32 t0, t1;
int ret;
u8 val; u8 val;
if (!(svm->nested.intercept & (1ULL << INTERCEPT_MSR_PROT))) if (!(svm->nested.intercept & (1ULL << INTERCEPT_MSR_PROT)))
return false; return NESTED_EXIT_HOST;
switch (msr) { switch (msr) {
case 0 ... 0x1fff: case 0 ... 0x1fff:
...@@ -1579,12 +1579,12 @@ static bool nested_svm_exit_handled_msr(struct vcpu_svm *svm) ...@@ -1579,12 +1579,12 @@ static bool nested_svm_exit_handled_msr(struct vcpu_svm *svm)
t0 %= 8; t0 %= 8;
break; break;
default: default:
ret = true; ret = NESTED_EXIT_DONE;
goto out; goto out;
} }
if (!kvm_read_guest(svm->vcpu.kvm, svm->nested.vmcb_msrpm + t1, &val, 1)) if (!kvm_read_guest(svm->vcpu.kvm, svm->nested.vmcb_msrpm + t1, &val, 1))
ret = val & ((1 << param) << t0); ret = val & ((1 << param) << t0) ? NESTED_EXIT_DONE : NESTED_EXIT_HOST;
out: out:
return ret; return ret;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册