diff options
| author | Kai Huang <kai.huang@intel.com> | 2026-07-15 21:05:05 +1200 |
|---|---|---|
| committer | Sean Christopherson <seanjc@google.com> | 2026-07-22 13:24:50 -0700 |
| commit | 9498f4d6edd358abff0000f207cf2de1f0b59145 (patch) | |
| tree | ca3c5f45a44ac5811d2c39f54a9b13b80e23cc81 /scripts | |
| parent | 249f9be58c7fd061e3323419c9adfeb938a2c48c (diff) | |
| download | linux-9498f4d6edd358abff0000f207cf2de1f0b59145.tar.gz linux-9498f4d6edd358abff0000f207cf2de1f0b59145.zip | |
KVM: x86: Use KVM_X86_OP() for the .pi_update_irte() hook
Change to using KVM_X86_OP() instead of KVM_X86_OP_OPTIONAL() for the
.pi_update_irte() hook in kvm-x86-ops.h since now both VMX and SVM have
implemented it.
For the Fixes tag:
This hook was introduced for VMX posted-interrupt support. SVM later
added its implementation, but at this point KVM_X86_OP* had not been
introduced yet.
Initially KVM introduced KVM_X86_OP_NULL (and KVM_X86_OP) and used it
for this hook. But this was correct, because the use of KVM_X86_OP_NULL
was "to mark calls that do not follow the [svm|vmx]_func_name naming
convention" and the VMX one was named pi_update_irte(), i.e., did not
follow the convention. See commit 9af5471bdbb2 ("KVM: x86: introduce
definitions to support static calls for kvm_x86_ops").
KVM later removed KVM_X86_OP_NULL (due to "the naming convention is not
in use anymore"), and added KVM_X86_OP_OPTIONAL for the hooks that can
be NULL pointer. It used KVM_X86_OP_OPTIONAL for this hook, but should
use KVM_X86_OP instead. See commit e4fc23bad813 ("KVM: x86: remove
KVM_X86_OP_NULL and mark optional kvm_x86_ops").
Note this hook was named .update_pi_irte() when it was introduced, but
got renamed to .pi_update_irte() at some point between the above two
commits.
Fixes: e4fc23bad813 ("KVM: x86: remove KVM_X86_OP_NULL and mark optional kvm_x86_ops")
Signed-off-by: Kai Huang <kai.huang@intel.com>
Link: https://patch.msgid.link/20260715090505.601174-1-kai.huang@intel.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions
