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.


added symbiotic swap telemetry data
[palacios.git] / palacios / src / palacios / vmm_shadow_paging_32.h
index 03e1f7a..b6ac424 100644 (file)
@@ -228,53 +228,62 @@ static int handle_pte_shadow_pagefault_32(struct guest_info * info, addr_t fault
 
        PrintDebug("Access error injecting pf to guest (guest access error=%d) (pf error code=%d)\n", 
                   guest_pte_access, *(uint_t*)&error_code);
+       
 #ifdef CONFIG_SYMBIOTIC_SWAP
-       if ((error_code.write == 0) && (is_swapped_pte32(guest_pte))) {
-           PrintError("Page fault on swapped out page (pte=%x)\n", *(uint32_t *)guest_pte);
-
-           addr_t swp_pg_addr = v3_get_swapped_pg_addr(info, shadow_pte, guest_pte);
+       if (is_swapped_pte32(guest_pte)) {
 
-           if (swp_pg_addr == 0) {
-               if (inject_guest_pf(info, fault_addr, error_code) == -1) {
-                   PrintError("Could not inject guest page fault\n");
-                   return -1;
-               }
+#ifdef CONFIG_SYMBIOTIC_SWAP_TELEMETRY
+           if (error_code.write == 0) {
+               info->swap_state.read_faults++;
            } else {
-               /* 
-                *  Setup shadow paging state
-                */
+               info->swap_state.write_faults++;
+           }
+#endif
+           if (error_code.write == 0) {
+               PrintError("Page fault on swapped out page (pte=%x) (error_code=%x)\n", *(uint32_t *)guest_pte, *(uint32_t *)&error_code);
                
-               /* We need some way to check permissions.... */
+               addr_t swp_pg_addr = v3_get_swapped_pg_addr(info, shadow_pte, guest_pte);
                
-               shadow_pte->accessed = 1;
-               shadow_pte->writable = 0;
-
-               if (fault_addr & 0xc0000000) {
-                   shadow_pte->user_page = 0;
+               if (swp_pg_addr == 0) {
+                   if (inject_guest_pf(info, fault_addr, error_code) == -1) {
+                       PrintError("Could not inject guest page fault\n");
+                       return -1;
+                   }
                } else {
-                   shadow_pte->user_page = 1;
+                   /* 
+                    *  Setup shadow paging state
+                    */
+                   
+                   /* We need some way to check permissions.... */
+                   
+                   shadow_pte->accessed = 1;
+                   shadow_pte->writable = 0;
+                   
+                   if (fault_addr & 0xc0000000) {
+                       shadow_pte->user_page = 0;
+                   } else {
+                       shadow_pte->user_page = 1;
+                   }
+                   
+                   shadow_pte->write_through = 0;
+                   shadow_pte->cache_disable = 0;
+                   shadow_pte->global_page = 0;
+                   
+                   shadow_pte->present = 1;
+                   
+                   shadow_pte->page_base_addr = swp_pg_addr;
                }
-
-               shadow_pte->write_through = 0;
-               shadow_pte->cache_disable = 0;
-               shadow_pte->global_page = 0;
-       
-               shadow_pte->present = 1;
                
-               shadow_pte->page_base_addr = swp_pg_addr;
-           }
-       } else {
-           if (inject_guest_pf(info, fault_addr, error_code) == -1) {
-               PrintError("Could not inject guest page fault\n");
-               return -1;
+               return 0;
            }
        }
-#else
+#endif
+       
+
        if (inject_guest_pf(info, fault_addr, error_code) == -1) {
            PrintError("Could not inject guest page fault\n");
            return -1;
-       }
-#endif
+       }       
 
        return 0; 
     }