From: "Xin Li (Intel)" <[email protected]>

Setup VM entry/exit FRED controls in the global vmcs_config for proper
FRED VMCS fields management:
  1) load guest FRED state upon VM entry.
  2) save guest FRED state during VM exit.
  3) load host FRED state during VM exit.

Also add FRED control consistency checks to the existing VM entry/exit
consistency check framework.

Signed-off-by: Xin Li (Intel) <[email protected]>
Signed-off-by: Sohil Mehta <[email protected]>
Reviewed-by: Chao Gao <[email protected]>
Reviewed-by: Binbin Wu <[email protected]>
---
v10:
 - No change
---
 arch/x86/include/asm/vmx.h | 4 ++++
 arch/x86/kvm/vmx/vmx.c     | 2 ++
 arch/x86/kvm/vmx/vmx.h     | 7 +++++--
 3 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/arch/x86/include/asm/vmx.h b/arch/x86/include/asm/vmx.h
index 1f7cffc118bf..682f09933612 100644
--- a/arch/x86/include/asm/vmx.h
+++ b/arch/x86/include/asm/vmx.h
@@ -121,6 +121,9 @@ struct vmcs {
 #define VM_EXIT_SAVE_IA32_PERF_GLOBAL_CTRL     0x40000000
 #define VM_EXIT_ACTIVATE_SECONDARY_CONTROLS    0x80000000
 
+#define SECONDARY_VM_EXIT_SAVE_IA32_FRED       BIT_ULL(0)
+#define SECONDARY_VM_EXIT_LOAD_IA32_FRED       BIT_ULL(1)
+
 #define VM_EXIT_ALWAYSON_WITHOUT_TRUE_MSR      0x00036dff
 
 #define VM_ENTRY_LOAD_DEBUG_CONTROLS            0x00000004
@@ -134,6 +137,7 @@ struct vmcs {
 #define VM_ENTRY_PT_CONCEAL_PIP                        0x00020000
 #define VM_ENTRY_LOAD_IA32_RTIT_CTL            0x00040000
 #define VM_ENTRY_LOAD_CET_STATE                 0x00100000
+#define VM_ENTRY_LOAD_IA32_FRED                        0x00800000
 
 #define VM_ENTRY_ALWAYSON_WITHOUT_TRUE_MSR     0x000011ff
 
diff --git a/arch/x86/kvm/vmx/vmx.c b/arch/x86/kvm/vmx/vmx.c
index b4aec4218b7d..062631e2dd37 100644
--- a/arch/x86/kvm/vmx/vmx.c
+++ b/arch/x86/kvm/vmx/vmx.c
@@ -2778,6 +2778,8 @@ static int setup_vmcs_config(struct vmcs_config 
*vmcs_conf,
                u32 entry_control;
                u64 exit_control;
        } const vmcs_entry_exit2_pairs[] = {
+               { VM_ENTRY_LOAD_IA32_FRED,
+                       SECONDARY_VM_EXIT_SAVE_IA32_FRED | 
SECONDARY_VM_EXIT_LOAD_IA32_FRED },
        };
 
        memset(vmcs_conf, 0, sizeof(*vmcs_conf));
diff --git a/arch/x86/kvm/vmx/vmx.h b/arch/x86/kvm/vmx/vmx.h
index 6295f9302cd9..7a22e1b1a273 100644
--- a/arch/x86/kvm/vmx/vmx.h
+++ b/arch/x86/kvm/vmx/vmx.h
@@ -477,7 +477,8 @@ static inline u8 vmx_get_rvi(void)
         VM_ENTRY_LOAD_BNDCFGS |                                        \
         VM_ENTRY_PT_CONCEAL_PIP |                                      \
         VM_ENTRY_LOAD_IA32_RTIT_CTL |                                  \
-        VM_ENTRY_LOAD_CET_STATE)
+        VM_ENTRY_LOAD_CET_STATE |                                      \
+        VM_ENTRY_LOAD_IA32_FRED)
 
 #define __KVM_REQUIRED_VMX_VM_EXIT_CONTROLS                            \
        (VM_EXIT_SAVE_DEBUG_CONTROLS |                                  \
@@ -505,7 +506,9 @@ static inline u8 vmx_get_rvi(void)
               VM_EXIT_ACTIVATE_SECONDARY_CONTROLS)
 
 #define KVM_REQUIRED_VMX_SECONDARY_VM_EXIT_CONTROLS (0)
-#define KVM_OPTIONAL_VMX_SECONDARY_VM_EXIT_CONTROLS (0)
+#define KVM_OPTIONAL_VMX_SECONDARY_VM_EXIT_CONTROLS                    \
+            (SECONDARY_VM_EXIT_SAVE_IA32_FRED |                        \
+             SECONDARY_VM_EXIT_LOAD_IA32_FRED)
 
 #define KVM_REQUIRED_VMX_PIN_BASED_VM_EXEC_CONTROL                     \
        (PIN_BASED_EXT_INTR_MASK |                                      \
-- 
2.43.0


Reply via email to