]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
KVM: x86/cpuid: add type suffix to decimal const 48 fix building warning
authorEthan Zhao <haifeng.zhao@linux.intel.com>
Mon, 27 Jan 2025 01:38:37 +0000 (09:38 +0800)
committerSean Christopherson <seanjc@google.com>
Wed, 12 Feb 2025 18:16:29 +0000 (10:16 -0800)
The default type of a decimal constant is determined by the magnitude of
its value. If the value falls within the range of int, its type is int;
otherwise, if it falls within the range of unsigned int, its type is
unsigned int. This results in the constant 48 being of type int. In the
following min call,

g_phys_as = min(g_phys_as, 48);

This leads to a building warning/error (CONFIG_KVM_WERROR=y) caused by
the mismatch between the types of the two arguments to macro min. By
adding the suffix U to explicitly declare the type of the constant, this
issue is fixed.

Signed-off-by: Ethan Zhao <haifeng.zhao@linux.intel.com>
Link: https://lore.kernel.org/r/20250127013837.12983-1-haifeng.zhao@linux.intel.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
arch/x86/kvm/cpuid.c

index 8eb3a88707f213b139e529999337d8163c3b4941..269ffbd5c2e0e0b0972a2842f35bffa5e182df4a 100644 (file)
@@ -1704,7 +1704,7 @@ static inline int __do_cpuid_func(struct kvm_cpuid_array *array, u32 function)
                        phys_as = entry->eax & 0xff;
                        g_phys_as = phys_as;
                        if (kvm_mmu_get_max_tdp_level() < 5)
-                               g_phys_as = min(g_phys_as, 48);
+                               g_phys_as = min(g_phys_as, 48U);
                }
 
                entry->eax = phys_as | (virt_as << 8) | (g_phys_as << 16);