diff options
| author | Sean Christopherson <seanjc@google.com> | 2024-11-28 04:33:34 +0300 |
|---|---|---|
| committer | Sean Christopherson <seanjc@google.com> | 2024-12-19 01:19:28 +0300 |
| commit | 08833719e77041e331f6193878f1b944744b9068 (patch) | |
| tree | 6210714ce57d65dab613d5ba98b7e0f1d04e9ef9 | |
| parent | bf4dfc3aa875c082dd70b979417f0729a35969a0 (diff) | |
| download | linux-08833719e77041e331f6193878f1b944744b9068.tar.xz | |
KVM: selftests: Assert that vcpu->cpuid is non-NULL when getting CPUID entries
Add a sanity check in __vcpu_get_cpuid_entry() to provide a friendlier
error than a segfault when a test developer tries to use a vCPU CPUID
helper on a barebones vCPU.
Link: https://lore.kernel.org/r/20241128013424.4096668-8-seanjc@google.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
| -rw-r--r-- | tools/testing/selftests/kvm/include/x86/processor.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tools/testing/selftests/kvm/include/x86/processor.h b/tools/testing/selftests/kvm/include/x86/processor.h index 9ec984cf8674..71aa290c583f 100644 --- a/tools/testing/selftests/kvm/include/x86/processor.h +++ b/tools/testing/selftests/kvm/include/x86/processor.h @@ -1014,6 +1014,8 @@ static inline struct kvm_cpuid_entry2 *__vcpu_get_cpuid_entry(struct kvm_vcpu *v uint32_t function, uint32_t index) { + TEST_ASSERT(vcpu->cpuid, "Must do vcpu_init_cpuid() first (or equivalent)"); + return (struct kvm_cpuid_entry2 *)get_cpuid_entry(vcpu->cpuid, function, index); } |