Palacios Public Git Repository

To checkout Palacios execute

  git clone http://v3vee.org/palacios/palacios.web/palacios.git
This will give you the master branch. You probably want the devel branch or one of the release branches. To switch to the devel branch, simply execute
  cd palacios
  git checkout --track -b devel origin/devel
The other branches are similar.


Cleanup and sanity-checking of assorted additional issues from second pass (Coverity...
[palacios.git] / palacios / src / palacios / svm_msr.c
index bd16fb2..bca1dd2 100644 (file)
@@ -19,9 +19,9 @@
 
 #include <palacios/svm_msr.h>
 #include <palacios/vmm_msr.h>
-
+#include <palacios/vmm_sprintf.h>
 #include <palacios/vmm_list.h>
-
+#include <palacios/vm_guest.h>
 
 #define PENTIUM_MSRS_START            0x00000000
 #define PENTIUM_MSRS_END              0x00001fff
 #define AMD_7_8_GEN_MSRS_START        0xc0010000
 #define AMD_7_8_GEN_MSRS_END          0xc0011fff
 
-#define PENTIUM_MSRS_INDEX            (0x0 * 4)
-#define AMD_6_GEN_MSRS_INDEX          (0x800 * 4)
-#define AMD_7_8_GEN_MSRS_INDEX        (0x1000 * 4)
+#define PENTIUM_MSRS_INDEX            (0)
+#define AMD_6_GEN_MSRS_INDEX          (0x2000)
+#define AMD_7_8_GEN_MSRS_INDEX        (0x4000)
 
 
 
 static int get_bitmap_index(uint_t msr) {
-  if ((msr >= PENTIUM_MSRS_START) && 
-      (msr <= PENTIUM_MSRS_END)) {
-    return (PENTIUM_MSRS_INDEX + (msr - PENTIUM_MSRS_START));
-  } else if ((msr >= AMD_6_GEN_MSRS_START) && 
-            (msr <= AMD_6_GEN_MSRS_END)) {
-    return (AMD_6_GEN_MSRS_INDEX + (msr - AMD_6_GEN_MSRS_START));
-  } else if ((msr >= AMD_7_8_GEN_MSRS_START) && 
-            (msr <= AMD_7_8_GEN_MSRS_END)) {
-    return (AMD_7_8_GEN_MSRS_INDEX + (msr - AMD_7_8_GEN_MSRS_START));
-  } else {
-    PrintError("MSR out of range (MSR=0x%x)\n", msr);
-    return -1;
-  }
+    // unsigend cmoparison with zero intentional for clarity
+    if ((msr >= PENTIUM_MSRS_START) && 
+       (msr <= PENTIUM_MSRS_END)) {
+       return (PENTIUM_MSRS_INDEX + (msr - PENTIUM_MSRS_START));
+    } else if ((msr >= AMD_6_GEN_MSRS_START) && 
+              (msr <= AMD_6_GEN_MSRS_END)) {
+       return (AMD_6_GEN_MSRS_INDEX + (msr - AMD_6_GEN_MSRS_START));
+    } else if ((msr >= AMD_7_8_GEN_MSRS_START) && 
+              (msr <= AMD_7_8_GEN_MSRS_END)) {
+       return (AMD_7_8_GEN_MSRS_INDEX + (msr - AMD_7_8_GEN_MSRS_START));
+    } else {
+       PrintError(VM_NONE, VCORE_NONE,  "MSR out of range (MSR=0x%x)\n", msr);
+       return -1;
+    }
 }
 
 
+static int update_map(struct v3_vm_info * vm, uint_t msr, int hook_reads, int hook_writes) {
+    int index = get_bitmap_index(msr);
+    uint_t major = index / 4;
+    uint_t minor = (index % 4) * 2;
+    uint8_t val = 0;
+    uint8_t mask = 0x3;
+    uint8_t * bitmap = (uint8_t *)(vm->msr_map.arch_data);
 
-addr_t v3_init_svm_msr_map(struct guest_info * info) {
-  uchar_t * msr_bitmap = (uchar_t*)V3_VAddr(V3_AllocPages(2));
-  struct v3_msr_map * msr_map = &(info->msr_map);
-  struct v3_msr_hook * hook = NULL;
-  
-  
-  memset(msr_bitmap, 0, PAGE_SIZE * 2);
-
-  list_for_each_entry(hook, &(msr_map->hook_list), link) {
-    int index = get_bitmap_index(hook->msr);
-    uint_t byte_offset = index / 4;
-    uint_t bit_offset = (index % 4) * 2;
-    uchar_t val = 0;
-    uchar_t mask = ~0x3;
+    if (index == -1) {
+       PrintError(vm, VCORE_NONE,  "MSR (0x%x) out of bitmap range\n", msr);
+       return 0;
+    }
 
-    if (hook->read) {
-      val |= 0x1;
+    if (hook_reads != 0) {
+       val |= 0x1;
     } 
-
-    if (hook->write) {
-      val |= 0x2;
+    
+    if (hook_writes != 0) {
+       val |= 0x2;
     }
 
-    val = val << bit_offset;
-    mask = mask << bit_offset;
-
-    *(msr_bitmap + byte_offset) &= mask;
-    *(msr_bitmap + byte_offset) |= val;
-  }
+    *(bitmap + major) &= ~(mask << minor);
+    *(bitmap + major) |= (val << minor);
 
-  return (addr_t)V3_PAddr(msr_bitmap);
+    return 0;
 }
 
 
+int v3_init_svm_msr_map(struct v3_vm_info * vm) {
+    void *temp;
+    struct v3_msr_map * msr_map = &(vm->msr_map);
+  
+    msr_map->update_map = update_map;
 
-int v3_handle_msr_write(struct guest_info * info) {
-  uint_t msr_num = info->vm_regs.rcx;
-  struct v3_msr msr_val;
-  struct v3_msr_hook * hook = NULL;
-
-  hook = v3_get_msr_hook(info, msr_num);
+    temp = V3_AllocPages(2); // need not be shadow-safe, not exposed to guest
+    
+    if (!temp) { 
+       PrintError(vm, VCORE_NONE,  "Cannot allocate msr bitmap\n");
+       return -1;
+    }
 
-  if (!hook) {
-    PrintError("Hook for MSR write %d not found\n", msr_num);
-    return -1;
-  }
+    msr_map->arch_data = V3_VAddr(temp);
 
-  msr_val.value = 0;
-  msr_val.lo = info->vm_regs.rax;
-  msr_val.hi = info->vm_regs.rdx;
+    memset(msr_map->arch_data, 0xff, PAGE_SIZE_4KB * 2);
 
-  if (hook->write(msr_num, msr_val, hook->priv_data) == -1) {
-    PrintError("Error in MSR hook Write\n");
-    return -1;
-  }
+    v3_refresh_msr_map(vm);
 
-  return 0;
+    return 0;
 }
 
-
-
-int v3_handle_msr_read(struct guest_info * info) {
-  uint_t msr_num = info->vm_regs.rcx;
-  struct v3_msr msr_val;
-  struct v3_msr_hook * hook = NULL;
-
-  hook = v3_get_msr_hook(info, msr_num);
-
-  if (!hook) {
-    PrintError("Hook for MSR read %d not found\n", msr_num);
-    return -1;
-  }
-
-  msr_val.value = 0;
-  
-  if (hook->read(msr_num, &msr_val, hook->priv_data) == -1) {
-    PrintError("Error in MSR hook Read\n");
-    return -1;
-  }
-
-  info->vm_regs.rax = msr_val.lo;
-  info->vm_regs.rdx = msr_val.hi;
-  
-  return 0;
+int v3_deinit_svm_msr_map(struct v3_vm_info * vm) {
+    if (vm->msr_map.arch_data) { 
+       V3_FreePages(V3_PAddr(vm->msr_map.arch_data), 2);
+    }
+    return 0;
 }