]> git.ipfire.org Git - thirdparty/linux.git/commit
KVM: SVM: Drop the module param to control SEV-ES DebugSwap
authorSean Christopherson <seanjc@google.com>
Fri, 9 Jan 2026 03:31:00 +0000 (19:31 -0800)
committerSean Christopherson <seanjc@google.com>
Wed, 14 Jan 2026 01:35:55 +0000 (17:35 -0800)
commit9587dd7a7ebd7be3c36815a4c4f90f7e2cedbe03
tree048287a03413c165902d96c23440ce3a8a52235d
parentca2eccb953fd33ef38701e33e660b21f7e84aa14
KVM: SVM: Drop the module param to control SEV-ES DebugSwap

Rip out the DebugSwap module param, as the sequence of events that led to
its inclusion was one big mistake, the param no longer serves any purpose.

Commit d1f85fbe836e ("KVM: SEV: Enable data breakpoints in SEV-ES") goofed
by not adding a way for the userspace VMM to control the feature.
Functionally, that was fine, but it broke attestation signatures because
SEV_FEATURES are included in the signature.

Commit 5abf6dceb066 ("SEV: disable SEV-ES DebugSwap by default") fixed that
issue, but the underlying flaw of userspace not having a way to control
SEV_FEATURES was still there.

That flaw was addressed by commit 4f5defae7089 ("KVM: SEV: introduce
KVM_SEV_INIT2 operation"), and so then 4dd5ecacb9a4 ("KVM: SEV: allow
SEV-ES DebugSwap again") re-enabled DebugSwap by default.

Now that the dust has settled, the module param doesn't serve any
meaningful purpose.

Cc: Tom Lendacky <thomas.lendacky@amd.com>
Reviewed-by: Tom Lendacky <thomas.lendacky@amd.com>
Link: https://patch.msgid.link/20260109033101.1005769-2-seanjc@google.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
arch/x86/kvm/svm/sev.c