X-Git-Url: http://v3vee.org/palacios/gitweb/gitweb.cgi?a=blobdiff_plain;f=palacios%2Fsrc%2Fpalacios%2Fvmm_direct_paging_32.h;h=ac6f054587c15777e8d671a0c55ca7d830337d6b;hb=1c000bda0560742ad6be011722fa226771b656ff;hp=047f2a6e7e0cf2b77f8119f58f9b977de8a06faf;hpb=fd9690bf0f032246f2d2c596e2467cccc45faff6;p=palacios.git

diff --git a/palacios/src/palacios/vmm_direct_paging_32.h b/palacios/src/palacios/vmm_direct_paging_32.h
index 047f2a6..ac6f054 100644
--- a/palacios/src/palacios/vmm_direct_paging_32.h
+++ b/palacios/src/palacios/vmm_direct_paging_32.h
@@ -39,7 +39,7 @@ static inline int handle_passthrough_pagefault_32(struct guest_info * info,
     int pde_index = PDE32_INDEX(fault_addr);
     int pte_index = PTE32_INDEX(fault_addr);
     
-    struct v3_shadow_region * region = v3_get_shadow_region(info->vm_info, info->cpu_id, fault_addr);
+    struct v3_mem_region * region = v3_get_mem_region(info->vm_info, info->vcpu_id, fault_addr);
 
     if (region == NULL) {
 	PrintError("Invalid region in passthrough page fault 32, addr=%p\n", 
@@ -47,8 +47,6 @@ static inline int handle_passthrough_pagefault_32(struct guest_info * info,
 	return -1;
     }
     
-    host_addr = v3_get_shadow_addr(region, info->cpu_id, fault_addr);
-    
     // Lookup the correct PDE address based on the PAGING MODE
     if (info->shdw_pg_mode == SHADOW_PAGING) {
 	pde = CR3_TO_PDE32_VA(info->ctrl_regs.cr3);
@@ -73,11 +71,12 @@ static inline int handle_passthrough_pagefault_32(struct guest_info * info,
     // Fix up the PTE entry
     if (pte[pte_index].present == 0) {
 	
-	pte[pte_index].user_page = 1;
 	
 	if ((region->flags.alloced == 1) && 
 	    (region->flags.read == 1)) {
 
+	    pte[pte_index].user_page = 1;
+
 	    pte[pte_index].present = 1;
 
 	    if (region->flags.write == 1) {
@@ -85,17 +84,22 @@ static inline int handle_passthrough_pagefault_32(struct guest_info * info,
 	    } else {
 		pte[pte_index].writable = 0;
 	    }
+
+    	    if (v3_gpa_to_hpa(info, fault_addr, &host_addr) == -1) {
+		PrintError("Could not translate fault address (%p)\n", (void *)fault_addr);
+		return -1;
+    	    }
 	    
 	    pte[pte_index].page_base_addr = PAGE_BASE_ADDR(host_addr);
+	} else {
+	    return region->unhandled(info, fault_addr, fault_addr, region, error_code);
 	}
+    } else {
+	// We fix all permissions on the first pass, 
+	// so we only get here if its an unhandled exception
+	return region->unhandled(info, fault_addr, fault_addr, region, error_code);	    
     }
 
-    if (region->flags.hook == 1) {
-	if ((error_code.write == 1) || (region->flags.read == 0))  {
-	    return v3_handle_mem_hook(info, fault_addr, fault_addr, region, error_code);	    
-	}
-    }
-    
     return 0;
 }
 
@@ -125,12 +129,16 @@ static inline int invalidate_addr_32(struct guest_info * info, addr_t inv_addr)
 	return 0;
     } else if (pde[pde_index].large_page) {
 	pde[pde_index].present = 0;
+	pde[pde_index].writable = 0;
+	pde[pde_index].user_page = 0;
 	return 0;
     }
 
     pte = V3_VAddr((void*)BASE_TO_PAGE_ADDR(pde[pde_index].pt_base_addr));
 
     pte[pte_index].present = 0;
+    pte[pte_index].writable = 0;
+    pte[pte_index].user_page = 0;
 
     return 0;
 }