From f6182f948bdc05f3d0949627378ac5d15eea8e58 Mon Sep 17 00:00:00 2001
From: Gabe Black <gblack@eecs.umich.edu>
Date: Mon, 23 Aug 2010 09:44:19 -0700
Subject: X86: Make the TLB fault instead of panic when something is unmapped
 in SE mode.

The fault object, if invoked, would then panic. This is a bit less direct, but
it means speculative execution won't panic the simulator.
---
 src/arch/x86/faults.cc | 16 ++++++++++++++++
 src/arch/x86/faults.hh |  2 +-
 src/arch/x86/tlb.cc    | 16 +---------------
 3 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/src/arch/x86/faults.cc b/src/arch/x86/faults.cc
index 20b5a931e..836a78567 100644
--- a/src/arch/x86/faults.cc
+++ b/src/arch/x86/faults.cc
@@ -267,6 +267,22 @@ namespace X86ISA
         tc->setNextPC(tc->readPC() + sizeof(MachInst));
     }
 
+#else
+
+    void
+    PageFault::invoke(ThreadContext * tc)
+    {
+        PageFaultErrorCode code = errorCode;
+        const char *modeStr = "";
+        if (code.fetch)
+            modeStr = "execute";
+        else if (code.write)
+            modeStr = "write";
+        else
+            modeStr = "read";
+        panic("Tried to %s unmapped address %#x.\n", modeStr, addr);
+    }
+
 #endif
 } // namespace X86ISA
 
diff --git a/src/arch/x86/faults.hh b/src/arch/x86/faults.hh
index 01d75fe17..bf3b6c8de 100644
--- a/src/arch/x86/faults.hh
+++ b/src/arch/x86/faults.hh
@@ -327,9 +327,9 @@ namespace X86ISA
             errorCode = code;
         }
 
-#if FULL_SYSTEM
         void invoke(ThreadContext * tc);
 
+#if FULL_SYSTEM
         virtual std::string describe() const;
 #endif
     };
diff --git a/src/arch/x86/tlb.cc b/src/arch/x86/tlb.cc
index 43b1ba04e..78efd5b69 100644
--- a/src/arch/x86/tlb.cc
+++ b/src/arch/x86/tlb.cc
@@ -619,21 +619,7 @@ TLB::translate(RequestPtr req, ThreadContext *tc, Translation *translation,
                     success = p->pTable->lookup(vaddr, newEntry);
                 }
                 if (!success) {
-                    if (req->isPrefetch()) {
-                        return new PageFault(vaddr, true, mode, true, false);
-                    } else {
-                        const char *modeStr = "";
-                        if (mode == Execute)
-                            modeStr = "execute";
-                        else if (mode == Read)
-                            modeStr = "read";
-                        else if (mode == Write)
-                            modeStr = "write";
-                        else
-                            modeStr = "?";
-                        panic("Tried to %s unmapped address %#x.\n",
-                                modeStr, vaddr);
-                    }
+                    return new PageFault(vaddr, true, mode, true, false);
                 } else {
                     Addr alignedVaddr = p->pTable->pageAlign(vaddr);
                     DPRINTF(TLB, "Mapping %#x to %#x\n", alignedVaddr,
-- 
cgit v1.2.3