Define the VMCS field offset for the extended instruction information. Then, support 5-bit register indices retrieval from VMCS fields when APX feature is enumerated. The presence of the extended instruction information field is indicated by APX enumeration, regardless of the XCR0.APX bit setting. With APX enumerated, the previously reserved bit in the exit qualification can be referenced safely now. However, there is no guarantee that older implementations always zeroed this bit. Link: https://lore.kernel.org/7bb14722-c036-4835-8ed9-046b4e67909e@redhat.com Suggested-by: Paolo Bonzini Signed-off-by: Chang S. Bae --- V1 -> V2: * Switch the change order; putting this ahead of nVMX changes (Chao) * Subsequently, define the field offset here. --- arch/x86/include/asm/vmx.h | 2 ++ arch/x86/kvm/vmx/vmx.h | 23 ++++++++++++++++++++--- 2 files changed, 22 insertions(+), 3 deletions(-) diff --git a/arch/x86/include/asm/vmx.h b/arch/x86/include/asm/vmx.h index c85c50019523..6170251306db 100644 --- a/arch/x86/include/asm/vmx.h +++ b/arch/x86/include/asm/vmx.h @@ -264,6 +264,8 @@ enum vmcs_field { PID_POINTER_TABLE_HIGH = 0x00002043, GUEST_PHYSICAL_ADDRESS = 0x00002400, GUEST_PHYSICAL_ADDRESS_HIGH = 0x00002401, + EXTENDED_INSTRUCTION_INFO = 0x00002406, + EXTENDED_INSTRUCTION_INFO_HIGH = 0x00002407, VMCS_LINK_POINTER = 0x00002800, VMCS_LINK_POINTER_HIGH = 0x00002801, GUEST_IA32_DEBUGCTL = 0x00002802, diff --git a/arch/x86/kvm/vmx/vmx.h b/arch/x86/kvm/vmx/vmx.h index 8d3e0aff2e13..a24d87aa4f79 100644 --- a/arch/x86/kvm/vmx/vmx.h +++ b/arch/x86/kvm/vmx/vmx.h @@ -372,12 +372,26 @@ struct vmx_insn_info { union insn_info info; }; +/* + * The APX enumeration guarantees the presence of the extended fields. + * The host CPUID bit alone is sufficient to rely on it. + */ +static inline bool vmx_insn_info_extended(void) +{ + return static_cpu_has(X86_FEATURE_APX); +} + static inline struct vmx_insn_info vmx_get_insn_info(void) { struct vmx_insn_info insn; - insn.extended = false; - insn.info.word = vmcs_read32(VMX_INSTRUCTION_INFO); + if (vmx_insn_info_extended()) { + insn.extended = true; + insn.info.dword = vmcs_read64(EXTENDED_INSTRUCTION_INFO); + } else { + insn.extended = false; + insn.info.word = vmcs_read32(VMX_INSTRUCTION_INFO); + } return insn; } @@ -413,7 +427,10 @@ static __always_inline unsigned long vmx_get_exit_qual(struct kvm_vcpu *vcpu) static inline int vmx_get_exit_qual_gpr(struct kvm_vcpu *vcpu) { - return (vmx_get_exit_qual(vcpu) >> 8) & 0xf; + if (vmx_insn_info_extended()) + return (vmx_get_exit_qual(vcpu) >> 8) & 0x1f; + else + return (vmx_get_exit_qual(vcpu) >> 8) & 0xf; } static __always_inline u32 vmx_get_intr_info(struct kvm_vcpu *vcpu) -- 2.51.0