Re: [PATCH v2 0/6] KVM/x86: Drop "1" as MSR emulation return value

From: Jürgen Groß

Date: Thu May 28 2026 - 10:10:49 EST


On 28.05.26 15:21, Sean Christopherson wrote:
On Thu, May 28, 2026, Jürgen Groß wrote:
On 28.05.26 15:09, Sean Christopherson wrote:
On Thu, May 28, 2026, Juergen Gross wrote:
Please disregard this series, there is one complication sashiko made me
aware of.

Sashiko beat me to the punch. :-)

See commit 2368048bf5c2 ("KVM: x86: Signal #GP, not -EPERM, on bad WRMSR(MCi_CTL/STATUS)")
for a real world example of how things can and will go wrong.

Yeah, with Sashiko's pointer it was easy to spot.

Question now is whether the already existing cases of -errno passed as return
value are wrong or on purpose.

What are the existing cases?

Have a look at:

kvm_hv_msr_get_crash_data()
kvm_hv_msr_set_crash_data()
svm_get_msr()
svm_set_msr()


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature