added FS_BASE and GS_BASE MSRs to passthrough set.

I think these MSRs are probably hardwired directly to the GDT entries, so there is no need to virtualize them.
This is good because the hardware does not provide any virtualization support for these two MSRs at all.

Of course I might be wrong about this...

diff --git a/palacios/src/palacios/svm.c b/palacios/src/palacios/svm.c
index cbb680b..95d9e6c 100644 (file)
--- a/palacios/src/palacios/svm.c
+++ b/palacios/src/palacios/svm.c
@@ -326,6 +326,10 @@ static void Init_VMCB_BIOS(vmcb_t * vmcb, struct guest_info * core) {
        v3_hook_msr(core->vm_info, SYSENTER_ESP_MSR, NULL, NULL, NULL);
        v3_hook_msr(core->vm_info, SYSENTER_EIP_MSR, NULL, NULL, NULL);
 
+
+       v3_hook_msr(core->vm_info, FS_BASE_MSR, NULL, NULL, NULL);
+       v3_hook_msr(core->vm_info, GS_BASE_MSR, NULL, NULL, NULL);
+
        // Passthrough read operations are ok.
        v3_hook_msr(core->vm_info, INT_PENDING_AMD_MSR, NULL, v3_msr_unhandled_write, NULL);
     }

diff --git a/palacios/src/palacios/vmx.c b/palacios/src/palacios/vmx.c
index 49be909..bc45c9b 100644 (file)
--- a/palacios/src/palacios/vmx.c
+++ b/palacios/src/palacios/vmx.c
@@ -411,6 +411,11 @@ static int init_vmcs_bios(struct guest_info * core, struct vmx_data * vmx_state)
        v3_hook_msr(core->vm_info, SYSENTER_CS_MSR, NULL, NULL, NULL);
        v3_hook_msr(core->vm_info, SYSENTER_ESP_MSR, NULL, NULL, NULL);
        v3_hook_msr(core->vm_info, SYSENTER_EIP_MSR, NULL, NULL, NULL);
+       
+       v3_hook_msr(core->vm_info, FS_BASE_MSR, NULL, NULL, NULL);
+       v3_hook_msr(core->vm_info, GS_BASE_MSR, NULL, NULL, NULL);
+       
+
     }    
 
     /* Sanity check ctrl/reg fields against hw_defaults */