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.

Signed-off-by: Sean Christopherson <sea...@google.com>
---
 tools/testing/selftests/kvm/include/x86_64/processor.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tools/testing/selftests/kvm/include/x86_64/processor.h 
b/tools/testing/selftests/kvm/include/x86_64/processor.h
index 645200e95f89..bdc121ed4ce5 100644
--- a/tools/testing/selftests/kvm/include/x86_64/processor.h
+++ b/tools/testing/selftests/kvm/include/x86_64/processor.h
@@ -1016,6 +1016,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);
 }
-- 
2.47.0.338.g60cca15819-goog


Reply via email to