X-Git-Url: http://v3vee.org/palacios/gitweb/gitweb.cgi?p=palacios.git;a=blobdiff_plain;f=palacios%2Fsrc%2Fpalacios%2Fvmm_shadow_paging_32.h;h=b6ac4246dfa23bc7e3d4ac5752f3e2e9014784f8;hp=03e1f7ac0e2a6061694803e5e8e3408a87cd0305;hb=449bf14185a1b3e2db53855b14e0e3ef3a803fd9;hpb=23b1c1715de4e064ee134b15273b3935ecfb8b74

diff --git a/palacios/src/palacios/vmm_shadow_paging_32.h b/palacios/src/palacios/vmm_shadow_paging_32.h
index 03e1f7a..b6ac424 100644
--- a/palacios/src/palacios/vmm_shadow_paging_32.h
+++ b/palacios/src/palacios/vmm_shadow_paging_32.h
@@ -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; 
     }