Commit caf22c6d authored by Qiang Liu's avatar Qiang Liu Committed by Sean Christopherson
Browse files

KVM: VMX: Modify the BUILD_BUG_ON_MSG of the 32-bit field in the vmcs_check16 function 



According to the SDM, the meaning of field bit 0 is:
Access type (0 = full; 1 = high); must be full for 16-bit, 32-bit,
and natural-width fields. So there is no 32-bit high field here,
it should be a 32-bit field instead.

Signed-off-by: default avatarQiang Liu <liuq131@chinatelecom.cn>
Link: https://lore.kernel.org/r/20240702064609.52487-1-liuq131@chinatelecom.cn


Signed-off-by: default avatarSean Christopherson <seanjc@google.com>
parent 47ac09b9

arch/x86/kvm/vmx/vmx_ops.h

+1 −1
Original line number Diff line number Diff line
@@ -47,7 +47,7 @@ static __always_inline void vmcs_check16(unsigned long field) 
	BUILD_BUG_ON_MSG(__builtin_constant_p(field) && ((field) & 0x6001) == 0x2001,

			 "16-bit accessor invalid for 64-bit high field");

	BUILD_BUG_ON_MSG(__builtin_constant_p(field) && ((field) & 0x6000) == 0x4000,

			 "16-bit accessor invalid for 32-bit high field");

			 "16-bit accessor invalid for 32-bit field");

	BUILD_BUG_ON_MSG(__builtin_constant_p(field) && ((field) & 0x6000) == 0x6000,

			 "16-bit accessor invalid for natural width field");

}