Fix order of shadow page status and release of memory page

Change-Id: Id9371157bd51a7c646fd9b7440d09532e68c0577
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/579
Tested-by: Jenkins Server
Reviewed-by: MATTHEW S. BARTH <msbarth@us.ibm.com>
Reviewed-by: Douglas R. Gilbert <dgilbert@us.ibm.com>

1 files changed, 6 insertions, 14 deletions

diff --git a/src/kernel/block.C b/src/kernel/block.C
index ce0ec5a65..474f67c53 100644
--- a/src/kernel/block.C
+++ b/src/kernel/block.C
@@ -255,23 +255,15 @@ void Block::releaseAllPages()
         page += PAGESIZE)
     {
         ShadowPTE* pte = getPTE(page);
-        if (pte->isPresent() && (0 != pte->getPageAddr()))
+        if (pte->isPresent())
         {
-
-            // set the permission of the physical address pte entry back to writable now that
-            // the associated VA Spte has been released.
-            if (BaseSegment::mmSetPermission(reinterpret_cast<void*>(pte->getPageAddr()),
-                                         0, WRITABLE))
+            uint64_t addr = pte->getPageAddr();
+            if (0 != addr)
             {
-                printkd("Got an error trying to set permissions in release all pages\n");
+                releaseSPTE(pte);
+                PageManager::freePage(reinterpret_cast<void*>(addr));
             }
-
-
-            PageManager::freePage(reinterpret_cast<void*>(pte->getPageAddr()));
-
-            pte->setPresent(false);
-            pte->setPageAddr(NULL);
-	}
+        }
     }
 }