From: Gabe Black <gblack@eecs.umich.edu>
Date: Sat, 7 Jan 2012 10:16:37 +0000 (-0800)
Subject: Another merge with the main repository.
X-Git-Tag: stable_2012_06_28~276
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=241cc0c8402f1b9f2ec20d1cc152d96930959b2a;p=gem5.git

Another merge with the main repository.
---

241cc0c8402f1b9f2ec20d1cc152d96930959b2a
diff --cc src/arch/x86/tlb.cc
index 53f7f978e,131909e50..ff65eb04c
--- a/src/arch/x86/tlb.cc
+++ b/src/arch/x86/tlb.cc
@@@ -287,41 -293,44 +287,44 @@@ TLB::translate(RequestPtr req, ThreadCo
              // The vaddr already has the segment base applied.
              TlbEntry *entry = lookup(vaddr);
              if (!entry) {
 -#if FULL_SYSTEM
 -                Fault fault = walker->start(tc, translation, req, mode);
 -                if (timing || fault != NoFault) {
 -                    // This gets ignored in atomic mode.
 -                    delayedResponse = true;
 -                    return fault;
 -                }
 -                entry = lookup(vaddr);
 -                assert(entry);
 -#else
 -                DPRINTF(TLB, "Handling a TLB miss for "
 -                        "address %#x at pc %#x.\n",
 -                        vaddr, tc->instAddr());
 -
 -                Process *p = tc->getProcessPtr();
 -                TlbEntry newEntry;
 -                bool success = p->pTable->lookup(vaddr, newEntry);
 -                if (!success && mode != Execute) {
 -                    // Check if we just need to grow the stack.
 -                    if (p->fixupStackFault(vaddr)) {
 -                        // If we did, lookup the entry for the new page.
 -                        success = p->pTable->lookup(vaddr, newEntry);
 +                if (FullSystem) {
 +                    Fault fault = walker->start(tc, translation, req, mode);
 +                    if (timing || fault != NoFault) {
 +                        // This gets ignored in atomic mode.
 +                        delayedResponse = true;
 +                        return fault;
                      }
 -                }
 -                if (!success) {
 -                    return new PageFault(vaddr, true, mode, true, false);
 +                    entry = lookup(vaddr);
 +                    assert(entry);
                  } else {
 -                    Addr alignedVaddr = p->pTable->pageAlign(vaddr);
 -                    DPRINTF(TLB, "Mapping %#x to %#x\n", alignedVaddr,
 -                            newEntry.pageStart());
 -                    entry = insert(alignedVaddr, newEntry);
 +                    DPRINTF(TLB, "Handling a TLB miss for "
 +                            "address %#x at pc %#x.\n",
 +                            vaddr, tc->instAddr());
 +
 +                    Process *p = tc->getProcessPtr();
 +                    TlbEntry newEntry;
 +                    bool success = p->pTable->lookup(vaddr, newEntry);
 +                    if (!success && mode != Execute) {
 +                        // Check if we just need to grow the stack.
 +                        if (p->fixupStackFault(vaddr)) {
 +                            // If we did, lookup the entry for the new page.
 +                            success = p->pTable->lookup(vaddr, newEntry);
 +                        }
 +                    }
 +                    if (!success) {
 +                        return new PageFault(vaddr, true, mode, true, false);
 +                    } else {
 +                        Addr alignedVaddr = p->pTable->pageAlign(vaddr);
 +                        DPRINTF(TLB, "Mapping %#x to %#x\n", alignedVaddr,
 +                                newEntry.pageStart());
 +                        entry = insert(alignedVaddr, newEntry);
 +                    }
 +                    DPRINTF(TLB, "Miss was serviced.\n");
                  }
 -                DPRINTF(TLB, "Miss was serviced.\n");
 -#endif
              }
+ 
+             DPRINTF(TLB, "Entry found with paddr %#x, "
+                     "doing protection checks.\n", entry->paddr);
              // Do paging protection checks.
              bool inUser = (m5Reg.cpl == 3 &&
                      !(flags & (CPL0FlagBit << FlagShift)));