[xen staging] x86/cpu: Clean up use of LCAP_* constants

[patchbot@xxxxxxx]

commit f962ef76bcab26c65b52c667896530c62b808799
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Fri Jan 23 15:41:35 2026 +0000
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Sat Apr 4 12:20:11 2026 +0200

    x86/cpu: Clean up use of LCAP_* constants
    
    Now that the LCAP_* constants are single bits, we can simplify the 
expressions
    using them.
    
    No functional change.
    
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
---
 xen/arch/x86/cpu/amd.c | 12 ++++++------
 xen/arch/x86/domain.c  | 10 +++++-----
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/xen/arch/x86/cpu/amd.c b/xen/arch/x86/cpu/amd.c
index 2b7f60aa30..45b55b7a8c 100644
--- a/xen/arch/x86/cpu/amd.c
+++ b/xen/arch/x86/cpu/amd.c
@@ -170,7 +170,7 @@ static void cf_check amd_ctxt_switch_masking(const struct 
vcpu *next)
                (nextd && is_pv_domain(nextd) && nextd->arch.pv.cpuidmasks)
                ? nextd->arch.pv.cpuidmasks : &cpuidmask_defaults;
 
-       if ((levelling_caps & LCAP_1cd) == LCAP_1cd) {
+       if (levelling_caps & LCAP_1cd) {
                uint64_t val = masks->_1cd;
 
                /*
@@ -191,7 +191,7 @@ static void cf_check amd_ctxt_switch_masking(const struct 
vcpu *next)
 #define LAZY(cap, msr, field)                                          \
        ({                                                              \
                if (unlikely(these_masks->field != masks->field) &&     \
-                   ((levelling_caps & cap) == cap))                    \
+                   (levelling_caps & cap))                             \
                {                                                       \
                        wrmsr_amd(msr, masks->field);                   \
                        these_masks->field = masks->field;              \
@@ -247,7 +247,7 @@ static void __init noinline amd_init_levelling(void)
 
        probe_masking_msrs();
 
-       if ((levelling_caps & LCAP_1cd) == LCAP_1cd) {
+       if (levelling_caps & LCAP_1cd) {
                uint32_t ecx, edx, tmp;
 
                cpuid(0x00000001, &tmp, &tmp, &ecx, &edx);
@@ -260,7 +260,7 @@ static void __init noinline amd_init_levelling(void)
                cpuidmask_defaults._1cd = ((uint64_t)ecx << 32) | edx;
        }
 
-       if ((levelling_caps & LCAP_e1cd) == LCAP_e1cd) {
+       if (levelling_caps & LCAP_e1cd) {
                uint32_t ecx, edx, tmp;
 
                cpuid(0x80000001, &tmp, &tmp, &ecx, &edx);
@@ -271,7 +271,7 @@ static void __init noinline amd_init_levelling(void)
                cpuidmask_defaults.e1cd = ((uint64_t)ecx << 32) | edx;
        }
 
-       if ((levelling_caps & LCAP_7ab0) == LCAP_7ab0) {
+       if (levelling_caps & LCAP_7ab0) {
                uint32_t eax, ebx, tmp;
 
                cpuid(0x00000007, &eax, &ebx, &tmp, &tmp);
@@ -279,7 +279,7 @@ static void __init noinline amd_init_levelling(void)
                cpuidmask_defaults._7ab0 &= ((uint64_t)eax << 32) | ebx;
        }
 
-       if ((levelling_caps & LCAP_6c) == LCAP_6c) {
+       if (levelling_caps & LCAP_6c) {
                uint32_t ecx = cpuid_ecx(6);
 
                cpuidmask_defaults._6c &= (~0ULL << 32) | ecx;
diff --git a/xen/arch/x86/domain.c b/xen/arch/x86/domain.c
index c9ae8ee6e3..1d458f1372 100644
--- a/xen/arch/x86/domain.c
+++ b/xen/arch/x86/domain.c
@@ -303,7 +303,7 @@ void domain_cpu_policy_changed(struct domain *d)
 
     if ( is_pv_domain(d) )
     {
-        if ( ((levelling_caps & LCAP_1cd) == LCAP_1cd) )
+        if ( levelling_caps & LCAP_1cd )
         {
             uint64_t mask = cpuidmask_defaults._1cd;
             uint32_t ecx = p->basic._1c;
@@ -368,7 +368,7 @@ void domain_cpu_policy_changed(struct domain *d)
             d->arch.pv.cpuidmasks->_1cd = mask;
         }
 
-        if ( ((levelling_caps & LCAP_6c) == LCAP_6c) )
+        if ( levelling_caps & LCAP_6c )
         {
             uint64_t mask = cpuidmask_defaults._6c;
 
@@ -378,7 +378,7 @@ void domain_cpu_policy_changed(struct domain *d)
             d->arch.pv.cpuidmasks->_6c = mask;
         }
 
-        if ( ((levelling_caps & LCAP_7ab0) == LCAP_7ab0) )
+        if ( levelling_caps & LCAP_7ab0 )
         {
             uint64_t mask = cpuidmask_defaults._7ab0;
 
@@ -395,7 +395,7 @@ void domain_cpu_policy_changed(struct domain *d)
             d->arch.pv.cpuidmasks->_7ab0 = mask;
         }
 
-        if ( ((levelling_caps & LCAP_Da1) == LCAP_Da1) )
+        if ( levelling_caps & LCAP_Da1 )
         {
             uint64_t mask = cpuidmask_defaults.Da1;
             uint32_t eax = p->xstate.Da1;
@@ -406,7 +406,7 @@ void domain_cpu_policy_changed(struct domain *d)
             d->arch.pv.cpuidmasks->Da1 = mask;
         }
 
-        if ( ((levelling_caps & LCAP_e1cd) == LCAP_e1cd) )
+        if ( levelling_caps & LCAP_e1cd )
         {
             uint64_t mask = cpuidmask_defaults.e1cd;
             uint32_t ecx = p->extd.e1c;
--
generated by git-patchbot for /home/xen/git/xen.git#staging