diff options
author | Sean Christopherson <seanjc@google.com> | 2023-03-21 18:14:40 -0700 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2023-04-06 13:37:37 -0400 |
commit | 400d2132288edbd6d500f45eab5d85526ca94e46 (patch) | |
tree | 3f490b28e9345a02e36bbafc13f2834ccc1cb305 /rust | |
parent | da3db168fb671f15e393b227f5c312c698ecb6ea (diff) | |
download | linux-stable-400d2132288edbd6d500f45eab5d85526ca94e46.tar.gz linux-stable-400d2132288edbd6d500f45eab5d85526ca94e46.tar.bz2 linux-stable-400d2132288edbd6d500f45eab5d85526ca94e46.zip |
KVM: SVM: Return the local "r" variable from svm_set_msr()
Rename "r" to "ret" and actually return it from svm_set_msr() to reduce
the probability of repeating the mistake of commit 723d5fb0ffe4 ("kvm:
svm: Add IA32_FLUSH_CMD guest support"), which set "r" thinking that it
would be propagated to the caller.
Alternatively, the declaration of "r" could be moved into the handling of
MSR_TSC_AUX, but that risks variable shadowing in the future. A wrapper
for kvm_set_user_return_msr() would allow eliding a local variable, but
that feels like delaying the inevitable.
No functional change intended.
Signed-off-by: Sean Christopherson <seanjc@google.com>
Message-Id: <20230322011440.2195485-7-seanjc@google.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'rust')
0 files changed, 0 insertions, 0 deletions