arch: Get rid of the generic mmapped IPR mechanism.
authorGabe Black <gabeblack@google.com>
Tue, 26 Nov 2019 01:30:22 +0000 (17:30 -0800)
committerGabe Black <gabeblack@google.com>
Tue, 11 Feb 2020 11:58:06 +0000 (11:58 +0000)
Jira Issue: https://gem5.atlassian.net/browse/GEM5-187

Change-Id: I4ab6f80581eee39e90fb91c672eca8e1a8fd9046
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/23186
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>

src/arch/alpha/mmapped_ipr.hh
src/arch/arm/tlb.cc
src/arch/generic/SConscript
src/arch/generic/mmapped_ipr.cc [deleted file]
src/arch/generic/mmapped_ipr.hh [deleted file]
src/arch/mips/mmapped_ipr.hh
src/arch/power/mmapped_ipr.hh
src/arch/riscv/mmapped_ipr.hh
src/arch/sparc/mmapped_ipr.hh
src/arch/x86/tlb.cc

index 46ed6bd49153009179d5ab799855ed80944bc547..f10e36ba2a1f975a61163574b18ee521304cc57e 100644 (file)
  * ISA-specific helper functions for memory mapped IPR accesses.
  */
 
-#include "arch/generic/mmapped_ipr.hh"
+#include "base/types.hh"
+
+class Packet;
+class ThreadContext;
+
+namespace AlphaISA
+{
+
+inline Cycles handleIprRead(ThreadContext *, Packet *) { return Cycles(1); }
+inline Cycles handleIprWrite(ThreadContext *, Packet *) { return Cycles(1); }
 
-namespace AlphaISA {
-    using GenericISA::handleIprRead;
-    using GenericISA::handleIprWrite;
 } // namespace AlphaISA
 
 #endif // __ARCH_ALPHA_MMAPPED_IPR_HH__
index 5358159fbca0b60972312792cb01f703f933c2bb..cec459770ba2552270329c3f9aaf59361c42f197 100644 (file)
@@ -55,7 +55,6 @@
 #include "arch/arm/system.hh"
 #include "arch/arm/table_walker.hh"
 #include "arch/arm/utility.hh"
-#include "arch/generic/mmapped_ipr.hh"
 #include "base/inifile.hh"
 #include "base/str.hh"
 #include "base/trace.hh"
index 64be7ce9b1b629c7e004fa1faa831dfd9c9a7ccb..ec46b075994d7292b6b19f1e0aff8db1f9095688 100644 (file)
@@ -44,7 +44,6 @@ if env['TARGET_ISA'] == 'null':
     Return()
 
 Source('decode_cache.cc')
-Source('mmapped_ipr.cc')
 
 SimObject('BaseInterrupts.py')
 SimObject('BaseISA.py')
diff --git a/src/arch/generic/mmapped_ipr.cc b/src/arch/generic/mmapped_ipr.cc
deleted file mode 100644 (file)
index 754ee02..0000000
+++ /dev/null
@@ -1,86 +0,0 @@
-/*
- * Copyright (c) 2013 Andreas Sandberg
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met: redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer;
- * redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution;
- * neither the name of the copyright holders nor the names of its
- * contributors may be used to endorse or promote products derived from
- * this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- *
- * Authors: Andreas Sandberg
- */
-
-#include "arch/generic/mmapped_ipr.hh"
-
-#include "arch/isa_traits.hh"
-#include "config/the_isa.hh"
-#include "mem/packet.hh"
-#include "mem/packet_access.hh"
-#include "sim/pseudo_inst.hh"
-
-using namespace GenericISA;
-
-static void
-handlePseudoInst(ThreadContext *xc, Packet *pkt)
-{
-    const Addr offset(pkt->getAddr() & IPR_IN_CLASS_MASK);
-    const uint8_t func((offset >> 8) & 0xFF);
-    uint64_t ret;
-
-    assert((offset >> 16) == 0);
-    ret = PseudoInst::pseudoInst<PseudoInstABI>(xc, func);
-    if (pkt->isRead())
-        pkt->set(ret, TheISA::GuestByteOrder);
-}
-
-Cycles
-GenericISA::handleGenericIprRead(ThreadContext *xc, Packet *pkt)
-{
-    Addr va(pkt->getAddr());
-    Addr cls(va >> IPR_CLASS_SHIFT);
-
-    switch (cls) {
-    case IPR_CLASS_PSEUDO_INST:
-        handlePseudoInst(xc, pkt);
-        break;
-    default:
-        panic("Unhandled generic IPR read: 0x%x\n", va);
-    }
-
-    return Cycles(1);
-}
-
-Cycles
-GenericISA::handleGenericIprWrite(ThreadContext *xc, Packet *pkt)
-{
-    Addr va(pkt->getAddr());
-    Addr cls(va >> IPR_CLASS_SHIFT);
-
-    switch (cls) {
-    case IPR_CLASS_PSEUDO_INST:
-        handlePseudoInst(xc, pkt);
-        break;
-    default:
-        panic("Unhandled generic IPR write: 0x%x\n", va);
-    }
-
-    return Cycles(1);
-}
diff --git a/src/arch/generic/mmapped_ipr.hh b/src/arch/generic/mmapped_ipr.hh
deleted file mode 100644 (file)
index a371699..0000000
+++ /dev/null
@@ -1,172 +0,0 @@
-/*
- * Copyright (c) 2013 Andreas Sandberg
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met: redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer;
- * redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution;
- * neither the name of the copyright holders nor the names of its
- * contributors may be used to endorse or promote products derived from
- * this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- *
- * Authors: Andreas Sandberg
- */
-
-#ifndef __ARCH_GENERIC_MMAPPED_IPR_HH__
-#define __ARCH_GENERIC_MMAPPED_IPR_HH__
-
-#include "base/types.hh"
-#include "mem/packet.hh"
-
-class ThreadContext;
-
-/**
- * @file
- *
- * ISA-generic helper functions for memory mapped IPR accesses.
- */
-
-namespace GenericISA
-{
-    /** @{ */
-    /**
-     * Memory requests with the MMAPPED_IPR flag are generally mapped
-     * to registers. There is a class of these registers that are
-     * internal to gem5, for example gem5 pseudo-ops in virtualized
-     * mode. Such IPRs always have the flag GENERIC_IPR set and are
-     * handled by this code.
-     */
-
-    /** Shift amount when extracting the class of a generic IPR */
-    const int IPR_CLASS_SHIFT = 48;
-
-    /** Mask to extract the offset in within a generic IPR class */
-    const Addr IPR_IN_CLASS_MASK = ULL(0x0000FFFFFFFFFFFF);
-
-    /** gem5 pseudo-inst emulation.
-     *
-     * Read and writes to this class execute gem5
-     * pseudo-instructions. A write discards the return value of the
-     * instruction, while a read returns it.
-     *
-     * @see pseudoInst()
-     */
-    const Addr IPR_CLASS_PSEUDO_INST = 0x0;
-
-    /** @} */
-
-    /**
-     * Generate a generic IPR address that emulates a pseudo inst
-     *
-     * @see PseudoInst::pseudoInst()
-     *
-     * @param func Function ID to call.
-     * @param subfunc Sub-function, usually 0.
-     * @return Address in the IPR space corresponding to the call.
-     */
-    inline Addr
-    iprAddressPseudoInst(uint8_t func, uint8_t subfunc)
-    {
-        return (IPR_CLASS_PSEUDO_INST << IPR_CLASS_SHIFT)  |
-            (func << 8) | subfunc;
-    }
-
-    /**
-     * Check if this is an platform independent IPR access
-     *
-     * Accesses to internal platform independent gem5 registers are
-     * handled by handleGenericIprRead() and
-     * handleGenericIprWrite(). This method determines if a packet
-     * should be routed to those functions instead of the platform
-     * specific code.
-     *
-     * @see handleGenericIprRead
-     * @see handleGenericIprWrite
-     */
-    inline bool
-    isGenericIprAccess(const Packet *pkt)
-    {
-        Request::Flags flags(pkt->req->getFlags());
-        return (flags & Request::MMAPPED_IPR) &&
-            (flags & Request::GENERIC_IPR);
-    }
-
-    /**
-     * Handle generic IPR reads
-     *
-     * @param xc Thread context of the current thread.
-     * @param pkt Packet from the CPU
-     * @return Latency in CPU cycles
-     */
-    Cycles handleGenericIprRead(ThreadContext *xc, Packet *pkt);
-    /**
-     * Handle generic IPR writes
-     *
-     * @param xc Thread context of the current thread.
-     * @param pkt Packet from the CPU
-     * @return Latency in CPU cycles
-     */
-    Cycles handleGenericIprWrite(ThreadContext *xc, Packet *pkt);
-
-    /**
-     * Helper function to handle IPRs when the target architecture doesn't
-     * need its own IPR handling.
-     *
-     * This function calls handleGenericIprRead if the accessing a
-     * generic IPR and panics otherwise.
-     *
-     * @param xc Thread context of the current thread.
-     * @param pkt Packet from the CPU
-     * @return Latency in CPU cycles
-     */
-    inline Cycles
-    handleIprRead(ThreadContext *xc, Packet *pkt)
-    {
-        if (!isGenericIprAccess(pkt))
-            panic("Unhandled IPR access\n");
-
-        return handleGenericIprRead(xc, pkt);
-    }
-
-
-    /**
-     * Helper function to handle IPRs when the target architecture
-     * doesn't need its own IPR handling.
-     *
-     * This function calls handleGenericIprWrite if the accessing a
-     * generic IPR and panics otherwise.
-     *
-     * @param xc Thread context of the current thread.
-     * @param pkt Packet from the CPU
-     * @return Latency in CPU cycles
-     */
-    inline Cycles
-    handleIprWrite(ThreadContext *xc, Packet *pkt)
-    {
-        if (!isGenericIprAccess(pkt))
-            panic("Unhandled IPR access\n");
-
-        return handleGenericIprWrite(xc, pkt);
-    }
-
-} // namespace GenericISA
-
-
-
-#endif
index 032fa28b22af8ca764c82e0009582be906fbfc9f..c90f755d76d3e993b424452b1f372e55d63924e5 100644 (file)
  * ISA-specific helper functions for memory mapped IPR accesses.
  */
 
-#include "arch/generic/mmapped_ipr.hh"
+#include "base/types.hh"
 
+class Packet;
 class ThreadContext;
 
 namespace MipsISA
 {
-    using GenericISA::handleIprRead;
-    using GenericISA::handleIprWrite;
+
+inline Cycles handleIprRead(ThreadContext *, Packet *) { return Cycles(1); }
+inline Cycles handleIprWrite(ThreadContext *, Packet *) { return Cycles(1); }
+
 } // namespace MipsISA
 
 #endif
index 28e3b613c74f3d42396599e81de512b3342e6f92..0d829c098fc944282f286bf75b80ff50084ebf78 100644 (file)
  * ISA-specific helper functions for memory mapped IPR accesses.
  */
 
-#include "arch/generic/mmapped_ipr.hh"
+#include "base/types.hh"
 
+class Packet;
 class ThreadContext;
 
 namespace PowerISA
 {
-    using GenericISA::handleIprRead;
-    using GenericISA::handleIprWrite;
+
+inline Cycles handleIprRead(ThreadContext *, Packet *) { return Cycles(1); }
+inline Cycles handleIprWrite(ThreadContext *, Packet *) { return Cycles(1); }
+
 } // namespace PowerISA
 
 #endif // __ARCH_POWER_MMAPPED_IPR_HH__
index 023c49c3074ff4482d1488d84c5f6730b2a38a16..553d83e5bda2635de52c535cfb2548fe6f8a949a 100644 (file)
  * ISA-specific helper functions for memory mapped IPR accesses.
  */
 
-#include "arch/generic/mmapped_ipr.hh"
+#include "base/types.hh"
 
+class Packet;
 class ThreadContext;
 
 namespace RiscvISA
 {
-    using GenericISA::handleIprRead;
-    using GenericISA::handleIprWrite;
+
+inline Cycles handleIprRead(ThreadContext *, Packet *) { return Cycles(1); }
+inline Cycles handleIprWrite(ThreadContext *, Packet *) { return Cycles(1); }
+
 } // namespace RiscvISA
 
 #endif
index 35e4d891b0f544aa5237e7cba1bce7d7deb28dd9..4fcaa1b69f6f49abc50a87929fd7425d96e163e9 100644 (file)
@@ -37,7 +37,6 @@
  * ISA-specific helper functions for memory mapped IPR accesses.
  */
 
-#include "arch/generic/mmapped_ipr.hh"
 #include "arch/sparc/tlb.hh"
 #include "cpu/thread_context.hh"
 #include "mem/packet.hh"
@@ -48,19 +47,13 @@ namespace SparcISA
 inline Cycles
 handleIprRead(ThreadContext *xc, Packet *pkt)
 {
-    if (GenericISA::isGenericIprAccess(pkt))
-        return GenericISA::handleGenericIprRead(xc, pkt);
-    else
-        return dynamic_cast<TLB *>(xc->getDTBPtr())->doMmuRegRead(xc, pkt);
+    return dynamic_cast<TLB *>(xc->getDTBPtr())->doMmuRegRead(xc, pkt);
 }
 
 inline Cycles
 handleIprWrite(ThreadContext *xc, Packet *pkt)
 {
-    if (GenericISA::isGenericIprAccess(pkt))
-        return GenericISA::handleGenericIprWrite(xc, pkt);
-    else
-        return dynamic_cast<TLB *>(xc->getDTBPtr())->doMmuRegWrite(xc, pkt);
+    return dynamic_cast<TLB *>(xc->getDTBPtr())->doMmuRegWrite(xc, pkt);
 }
 
 
index a649871855cc1c27fa82497f56ff7d3cafb1506a..e665769037cd8d7dc126b8c0084a35d762449d74 100644 (file)
@@ -42,7 +42,6 @@
 #include <cstring>
 #include <memory>
 
-#include "arch/generic/mmapped_ipr.hh"
 #include "arch/x86/faults.hh"
 #include "arch/x86/insts/microldstop.hh"
 #include "arch/x86/pagetable_walker.hh"