Commit 683edfd4 authored by Sean Christopherson's avatar Sean Christopherson

KVM: sefltests: Use CPUID_* instead of X86_FEATURE_* for one-off usage

Rename X86_FEATURE_* macros to CPUID_* in various tests to free up the
X86_FEATURE_* names for KVM-Unit-Tests style CPUID automagic where the
function, leaf, register, and bit for the feature is embedded in its
macro value.

No functional change intended.
Signed-off-by: default avatarSean Christopherson <seanjc@google.com>
Link: https://lore.kernel.org/r/20220614200707.3315957-3-seanjc@google.com
parent 4c16fa3e
...@@ -50,6 +50,7 @@ ...@@ -50,6 +50,7 @@
#define CPUID_SMX (1ul << 6) #define CPUID_SMX (1ul << 6)
#define CPUID_PCID (1ul << 17) #define CPUID_PCID (1ul << 17)
#define CPUID_XSAVE (1ul << 26) #define CPUID_XSAVE (1ul << 26)
#define CPUID_OSXSAVE (1ul << 27)
/* CPUID.7.EBX */ /* CPUID.7.EBX */
#define CPUID_FSGSBASE (1ul << 0) #define CPUID_FSGSBASE (1ul << 0)
...@@ -64,6 +65,9 @@ ...@@ -64,6 +65,9 @@
/* CPUID.0x8000_0001.EDX */ /* CPUID.0x8000_0001.EDX */
#define CPUID_GBPAGES (1ul << 26) #define CPUID_GBPAGES (1ul << 26)
/* CPUID.0x8000_000A.EDX */
#define CPUID_NRIPS BIT(3)
/* Page table bitfield declarations */ /* Page table bitfield declarations */
#define PTE_PRESENT_MASK BIT_ULL(0) #define PTE_PRESENT_MASK BIT_ULL(0)
#define PTE_WRITABLE_MASK BIT_ULL(1) #define PTE_WRITABLE_MASK BIT_ULL(1)
......
...@@ -25,9 +25,6 @@ ...@@ -25,9 +25,6 @@
# error This test is 64-bit only # error This test is 64-bit only
#endif #endif
#define X86_FEATURE_XSAVE (1 << 26)
#define X86_FEATURE_OSXSAVE (1 << 27)
#define NUM_TILES 8 #define NUM_TILES 8
#define TILE_SIZE 1024 #define TILE_SIZE 1024
#define XSAVE_SIZE ((NUM_TILES * TILE_SIZE) + PAGE_SIZE) #define XSAVE_SIZE ((NUM_TILES * TILE_SIZE) + PAGE_SIZE)
...@@ -128,9 +125,9 @@ static inline void check_cpuid_xsave(void) ...@@ -128,9 +125,9 @@ static inline void check_cpuid_xsave(void)
eax = 1; eax = 1;
ecx = 0; ecx = 0;
cpuid(&eax, &ebx, &ecx, &edx); cpuid(&eax, &ebx, &ecx, &edx);
if (!(ecx & X86_FEATURE_XSAVE)) if (!(ecx & CPUID_XSAVE))
GUEST_ASSERT(!"cpuid: no CPU xsave support!"); GUEST_ASSERT(!"cpuid: no CPU xsave support!");
if (!(ecx & X86_FEATURE_OSXSAVE)) if (!(ecx & CPUID_OSXSAVE))
GUEST_ASSERT(!"cpuid: no OS xsave support!"); GUEST_ASSERT(!"cpuid: no OS xsave support!");
} }
...@@ -333,7 +330,7 @@ int main(int argc, char *argv[]) ...@@ -333,7 +330,7 @@ int main(int argc, char *argv[])
vm = vm_create_with_one_vcpu(&vcpu, guest_code); vm = vm_create_with_one_vcpu(&vcpu, guest_code);
entry = kvm_get_supported_cpuid_entry(1); entry = kvm_get_supported_cpuid_entry(1);
TEST_REQUIRE(entry->ecx & X86_FEATURE_XSAVE); TEST_REQUIRE(entry->ecx & CPUID_XSAVE);
TEST_REQUIRE(kvm_get_cpuid_max_basic() >= 0xd); TEST_REQUIRE(kvm_get_cpuid_max_basic() >= 0xd);
......
...@@ -19,9 +19,6 @@ ...@@ -19,9 +19,6 @@
#include "kvm_util.h" #include "kvm_util.h"
#include "processor.h" #include "processor.h"
#define X86_FEATURE_XSAVE (1<<26)
#define X86_FEATURE_OSXSAVE (1<<27)
static inline bool cr4_cpuid_is_sync(void) static inline bool cr4_cpuid_is_sync(void)
{ {
int func, subfunc; int func, subfunc;
...@@ -36,7 +33,7 @@ static inline bool cr4_cpuid_is_sync(void) ...@@ -36,7 +33,7 @@ static inline bool cr4_cpuid_is_sync(void)
cr4 = get_cr4(); cr4 = get_cr4();
return (!!(ecx & X86_FEATURE_OSXSAVE)) == (!!(cr4 & X86_CR4_OSXSAVE)); return (!!(ecx & CPUID_OSXSAVE)) == (!!(cr4 & X86_CR4_OSXSAVE));
} }
static void guest_code(void) static void guest_code(void)
...@@ -70,7 +67,7 @@ int main(int argc, char *argv[]) ...@@ -70,7 +67,7 @@ int main(int argc, char *argv[])
struct ucall uc; struct ucall uc;
entry = kvm_get_supported_cpuid_entry(1); entry = kvm_get_supported_cpuid_entry(1);
TEST_REQUIRE(entry->ecx & X86_FEATURE_XSAVE); TEST_REQUIRE(entry->ecx & CPUID_XSAVE);
/* Tell stdout not to buffer its content */ /* Tell stdout not to buffer its content */
setbuf(stdout, NULL); setbuf(stdout, NULL);
......
...@@ -8,7 +8,7 @@ ...@@ -8,7 +8,7 @@
#include "kvm_util.h" #include "kvm_util.h"
#include "processor.h" #include "processor.h"
#define X86_FEATURE_MWAIT (1u << 3) #define CPUID_MWAIT (1u << 3)
enum monitor_mwait_testcases { enum monitor_mwait_testcases {
MWAIT_QUIRK_DISABLED = BIT(0), MWAIT_QUIRK_DISABLED = BIT(0),
...@@ -76,7 +76,7 @@ int main(int argc, char *argv[]) ...@@ -76,7 +76,7 @@ int main(int argc, char *argv[])
cpuid = kvm_get_supported_cpuid(); cpuid = kvm_get_supported_cpuid();
entry = kvm_get_supported_cpuid_index(1, 0); entry = kvm_get_supported_cpuid_index(1, 0);
entry->ecx &= ~X86_FEATURE_MWAIT; entry->ecx &= ~CPUID_MWAIT;
set_cpuid(cpuid, entry); set_cpuid(cpuid, entry);
vm = vm_create_with_one_vcpu(&vcpu, guest_code); vm = vm_create_with_one_vcpu(&vcpu, guest_code);
......
...@@ -19,7 +19,6 @@ ...@@ -19,7 +19,6 @@
#include "test_util.h" #include "test_util.h"
#define INT_NR 0x20 #define INT_NR 0x20
#define X86_FEATURE_NRIPS BIT(3)
static_assert(ATOMIC_INT_LOCK_FREE == 2, "atomic int is not lockless"); static_assert(ATOMIC_INT_LOCK_FREE == 2, "atomic int is not lockless");
...@@ -203,7 +202,7 @@ int main(int argc, char *argv[]) ...@@ -203,7 +202,7 @@ int main(int argc, char *argv[])
nested_svm_check_supported(); nested_svm_check_supported();
cpuid = kvm_get_supported_cpuid_entry(0x8000000a); cpuid = kvm_get_supported_cpuid_entry(0x8000000a);
TEST_ASSERT(cpuid->edx & X86_FEATURE_NRIPS, TEST_ASSERT(cpuid->edx & CPUID_NRIPS,
"KVM with nSVM is supposed to unconditionally advertise nRIP Save\n"); "KVM with nSVM is supposed to unconditionally advertise nRIP Save\n");
atomic_init(&nmi_stage, 0); atomic_init(&nmi_stage, 0);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment