From 8e111ba2507f191f8a08d98972dd45eab0242250 Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Tue, 27 Mar 2018 02:18:10 -0700 Subject: [PATCH] dev: Remove a bunch of Alpha code from MIPS, and unnecessary TheISAs. There was a bunch of commented out code in the MIPS malta implementation which originally came from Alpha. That code is now deleted. Also, the MIPS code pulled in the TheISA namespace which it didn't use. Change-Id: I8470cc2fecb302f4399e52de4de9daf79c00a711 Reviewed-on: https://gem5-review.googlesource.com/9405 Reviewed-by: Andreas Sandberg Maintainer: Gabe Black --- src/dev/mips/malta.cc | 2 - src/dev/mips/malta_cchip.cc | 410 ++---------------------------------- src/dev/mips/malta_io.cc | 2 - 3 files changed, 16 insertions(+), 398 deletions(-) diff --git a/src/dev/mips/malta.cc b/src/dev/mips/malta.cc index a25904780..ef4360951 100644 --- a/src/dev/mips/malta.cc +++ b/src/dev/mips/malta.cc @@ -39,7 +39,6 @@ #include #include -#include "config/the_isa.hh" #include "cpu/intr_control.hh" #include "debug/Malta.hh" #include "dev/mips/malta_cchip.hh" @@ -48,7 +47,6 @@ #include "sim/system.hh" using namespace std; -using namespace TheISA; Malta::Malta(const Params *p) : Platform(p), system(p->system) diff --git a/src/dev/mips/malta_cchip.cc b/src/dev/mips/malta_cchip.cc index 929702f77..1dd2882f2 100644 --- a/src/dev/mips/malta_cchip.cc +++ b/src/dev/mips/malta_cchip.cc @@ -40,7 +40,6 @@ #include #include "base/trace.hh" -#include "config/the_isa.hh" #include "cpu/intr_control.hh" #include "cpu/thread_context.hh" #include "debug/Malta.hh" @@ -53,7 +52,6 @@ #include "sim/system.hh" using namespace std; -using namespace TheISA; MaltaCChip::MaltaCChip(Params *p) : BasicPioDevice(p, 0xfffffff), malta(p->malta) @@ -68,407 +66,40 @@ MaltaCChip::MaltaCChip(Params *p) Tick MaltaCChip::read(PacketPtr pkt) { - panic("MaltaCCHIP::read() not implemented."); - return pioDelay; - /* - DPRINTF(Malta, "read va=%#x size=%d\n", pkt->getAddr(), pkt->getSize()); - - assert(pkt->result == Packet::Unknown); - assert(pkt->getAddr() >= pioAddr && pkt->getAddr() < pioAddr + pioSize); - - Addr regnum = (pkt->getAddr() - pioAddr) >> 6; - Addr daddr = (pkt->getAddr() - pioAddr); - - switch (pkt->getSize()) { - - case sizeof(uint64_t): - if (daddr & TSDEV_CC_BDIMS) - { - pkt->set(dim[(daddr >> 4) & 0x3F]); - break; - } - - if (daddr & TSDEV_CC_BDIRS) - { - pkt->set(dir[(daddr >> 4) & 0x3F]); - break; - } - - switch(regnum) { - case TSDEV_CC_CSR: - pkt->set(0x0); - break; - case TSDEV_CC_MTR: - panic("TSDEV_CC_MTR not implemeted\n"); - break; - case TSDEV_CC_MISC: - pkt->set((ipint << 8) & 0xF | (itint << 4) & 0xF | - (pkt->req->contextId() & 0x3)); - break; - case TSDEV_CC_AAR0: - case TSDEV_CC_AAR1: - case TSDEV_CC_AAR2: - case TSDEV_CC_AAR3: - pkt->set(0); - break; - case TSDEV_CC_DIM0: - pkt->set(dim[0]); - break; - case TSDEV_CC_DIM1: - pkt->set(dim[1]); - break; - case TSDEV_CC_DIM2: - pkt->set(dim[2]); - break; - case TSDEV_CC_DIM3: - pkt->set(dim[3]); - break; - case TSDEV_CC_DIR0: - pkt->set(dir[0]); - break; - case TSDEV_CC_DIR1: - pkt->set(dir[1]); - break; - case TSDEV_CC_DIR2: - pkt->set(dir[2]); - break; - case TSDEV_CC_DIR3: - pkt->set(dir[3]); - break; - case TSDEV_CC_DRIR: - pkt->set(drir); - break; - case TSDEV_CC_PRBEN: - panic("TSDEV_CC_PRBEN not implemented\n"); - break; - case TSDEV_CC_IIC0: - case TSDEV_CC_IIC1: - case TSDEV_CC_IIC2: - case TSDEV_CC_IIC3: - panic("TSDEV_CC_IICx not implemented\n"); - break; - case TSDEV_CC_MPR0: - case TSDEV_CC_MPR1: - case TSDEV_CC_MPR2: - case TSDEV_CC_MPR3: - panic("TSDEV_CC_MPRx not implemented\n"); - break; - case TSDEV_CC_IPIR: - pkt->set(ipint); - break; - case TSDEV_CC_ITIR: - pkt->set(itint); - break; - default: - panic("default in cchip read reached, accessing 0x%x\n"); - } // uint64_t - - break; - case sizeof(uint32_t): - case sizeof(uint16_t): - case sizeof(uint8_t): - default: - panic("invalid access size(?) for malta register!\n"); - } - DPRINTF(Malta, "Malta CChip: read regnum=%#x size=%d data=%lld\n", - regnum, pkt->getSize(), pkt->get()); - - pkt->result = Packet::Success; + panic("MaltaCCHIP::read() not implemented."); return pioDelay; - */ } Tick MaltaCChip::write(PacketPtr pkt) { - panic("MaltaCCHIP::write() not implemented."); - return pioDelay; - /* - assert(pkt->getAddr() >= pioAddr && pkt->getAddr() < pioAddr + pioSize); - Addr daddr = pkt->getAddr() - pioAddr; - Addr regnum = (pkt->getAddr() - pioAddr) >> 6 ; - - - assert(pkt->getSize() == sizeof(uint64_t)); - - DPRINTF(Malta, "write - addr=%#x value=%#x\n", pkt->getAddr(), pkt->get()); - - bool supportedWrite = false; - - - if (daddr & TSDEV_CC_BDIMS) - { - int number = (daddr >> 4) & 0x3F; - - uint64_t bitvector; - uint64_t olddim; - uint64_t olddir; - - olddim = dim[number]; - olddir = dir[number]; - dim[number] = pkt->get(); - dir[number] = dim[number] & drir; - for (int x = 0; x < Malta::Max_CPUs; x++) - { - bitvector = ULL(1) << x; - // Figure out which bits have changed - if ((dim[number] & bitvector) != (olddim & bitvector)) - { - // The bit is now set and it wasn't before (set) - if ((dim[number] & bitvector) && (dir[number] & bitvector)) - { - malta->intrctrl->post(number, TheISA::INTLEVEL_IRQ1, x); - DPRINTF(Malta, "dim write resulting in posting dir" - " interrupt to cpu %d\n", number); - } - else if ((olddir & bitvector) && - !(dir[number] & bitvector)) - { - // The bit was set and now its now clear and - // we were interrupting on that bit before - malta->intrctrl->clear(number, TheISA::INTLEVEL_IRQ1, x); - DPRINTF(Malta, "dim write resulting in clear" - " dir interrupt to cpu %d\n", number); - - } - - - } - } - } else { - switch(regnum) { - case TSDEV_CC_CSR: - panic("TSDEV_CC_CSR write\n"); - case TSDEV_CC_MTR: - panic("TSDEV_CC_MTR write not implemented\n"); - case TSDEV_CC_MISC: - uint64_t ipreq; - ipreq = (pkt->get() >> 12) & 0xF; - //If it is bit 12-15, this is an IPI post - if (ipreq) { - reqIPI(ipreq); - supportedWrite = true; - } - - //If it is bit 8-11, this is an IPI clear - uint64_t ipintr; - ipintr = (pkt->get() >> 8) & 0xF; - if (ipintr) { - clearIPI(ipintr); - supportedWrite = true; - } - - //If it is the 4-7th bit, clear the RTC interrupt - uint64_t itintr; - itintr = (pkt->get() >> 4) & 0xF; - if (itintr) { - clearITI(itintr); - supportedWrite = true; - } - - // ignore NXMs - if (pkt->get() & 0x10000000) - supportedWrite = true; - - if (!supportedWrite) - panic("TSDEV_CC_MISC write not implemented\n"); - - break; - case TSDEV_CC_AAR0: - case TSDEV_CC_AAR1: - case TSDEV_CC_AAR2: - case TSDEV_CC_AAR3: - panic("TSDEV_CC_AARx write not implemeted\n"); - case TSDEV_CC_DIM0: - case TSDEV_CC_DIM1: - case TSDEV_CC_DIM2: - case TSDEV_CC_DIM3: - int number; - if (regnum == TSDEV_CC_DIM0) - number = 0; - else if (regnum == TSDEV_CC_DIM1) - number = 1; - else if (regnum == TSDEV_CC_DIM2) - number = 2; - else - number = 3; - - uint64_t bitvector; - uint64_t olddim; - uint64_t olddir; - - olddim = dim[number]; - olddir = dir[number]; - dim[number] = pkt->get(); - dir[number] = dim[number] & drir; - for (int x = 0; x < 64; x++) - { - bitvector = ULL(1) << x; - // Figure out which bits have changed - if ((dim[number] & bitvector) != (olddim & bitvector)) - { - // The bit is now set and it wasn't before (set) - if ((dim[number] & bitvector) && (dir[number] & bitvector)) - { - malta->intrctrl->post(number, TheISA::INTLEVEL_IRQ1, x); - DPRINTF(Malta, "posting dir interrupt to cpu 0\n"); - } - else if ((olddir & bitvector) && - !(dir[number] & bitvector)) - { - // The bit was set and now its now clear and - // we were interrupting on that bit before - malta->intrctrl->clear(number, TheISA::INTLEVEL_IRQ1, x); - DPRINTF(Malta, "dim write resulting in clear" - " dir interrupt to cpu %d\n", - x); - - } - - - } - } - break; - case TSDEV_CC_DIR0: - case TSDEV_CC_DIR1: - case TSDEV_CC_DIR2: - case TSDEV_CC_DIR3: - panic("TSDEV_CC_DIR write not implemented\n"); - case TSDEV_CC_DRIR: - panic("TSDEV_CC_DRIR write not implemented\n"); - case TSDEV_CC_PRBEN: - panic("TSDEV_CC_PRBEN write not implemented\n"); - case TSDEV_CC_IIC0: - case TSDEV_CC_IIC1: - case TSDEV_CC_IIC2: - case TSDEV_CC_IIC3: - panic("TSDEV_CC_IICx write not implemented\n"); - case TSDEV_CC_MPR0: - case TSDEV_CC_MPR1: - case TSDEV_CC_MPR2: - case TSDEV_CC_MPR3: - panic("TSDEV_CC_MPRx write not implemented\n"); - case TSDEV_CC_IPIR: - clearIPI(pkt->get()); - break; - case TSDEV_CC_ITIR: - clearITI(pkt->get()); - break; - case TSDEV_CC_IPIQ: - reqIPI(pkt->get()); - break; - default: - panic("default in cchip read reached, accessing 0x%x\n"); - } // swtich(regnum) - } // not BIG_TSUNAMI write - pkt->result = Packet::Success; + panic("MaltaCCHIP::write() not implemented."); return pioDelay; - */ } void MaltaCChip::clearIPI(uint64_t ipintr) { - panic("MaltaCCHIP::clear() not implemented."); - /* - int numcpus = malta->intrctrl->cpu->system->threadContexts.size(); - assert(numcpus <= Malta::Max_CPUs); - - if (ipintr) { - for (int cpunum=0; cpunum < numcpus; cpunum++) { - // Check each cpu bit - uint64_t cpumask = ULL(1) << cpunum; - if (ipintr & cpumask) { - // Check if there is a pending ipi - if (ipint & cpumask) { - ipint &= ~cpumask; - malta->intrctrl->clear(cpunum, TheISA::INTLEVEL_IRQ3, 0); - DPRINTF(IPI, "clear IPI IPI cpu=%d\n", cpunum); - } - else - warn("clear IPI for CPU=%d, but NO IPI\n", cpunum); - } - } - } - else - panic("Big IPI Clear, but not processors indicated\n"); - */ + panic("MaltaCCHIP::clear() not implemented."); } void MaltaCChip::clearITI(uint64_t itintr) { - panic("MaltaCCHIP::clearITI() not implemented."); - /* - int numcpus = malta->intrctrl->cpu->system->threadContexts.size(); - assert(numcpus <= Malta::Max_CPUs); - - if (itintr) { - for (int i=0; i < numcpus; i++) { - uint64_t cpumask = ULL(1) << i; - if (itintr & cpumask & itint) { - malta->intrctrl->clear(i, TheISA::INTLEVEL_IRQ2, 0); - itint &= ~cpumask; - DPRINTF(Malta, "clearing rtc interrupt to cpu=%d\n", i); - } - } - } - else - panic("Big ITI Clear, but not processors indicated\n"); - */ + panic("MaltaCCHIP::clearITI() not implemented."); } void MaltaCChip::reqIPI(uint64_t ipreq) { - panic("MaltaCCHIP::reqIPI() not implemented."); - - /* - int numcpus = malta->intrctrl->cpu->system->threadContexts.size(); - assert(numcpus <= Malta::Max_CPUs); - - if (ipreq) { - for (int cpunum=0; cpunum < numcpus; cpunum++) { - // Check each cpu bit - uint64_t cpumask = ULL(1) << cpunum; - if (ipreq & cpumask) { - // Check if there is already an ipi (bits 8:11) - if (!(ipint & cpumask)) { - ipint |= cpumask; - malta->intrctrl->post(cpunum, TheISA::INTLEVEL_IRQ3, 0); - DPRINTF(IPI, "send IPI cpu=%d\n", cpunum); - } - else - warn("post IPI for CPU=%d, but IPI already\n", cpunum); - } - } - } - else - panic("Big IPI Request, but not processors indicated\n"); - */ - + panic("MaltaCCHIP::reqIPI() not implemented."); } void MaltaCChip::postRTC() { - panic("MaltaCCHIP::postRTC() not implemented."); - - /* - int size = malta->intrctrl->cpu->system->threadContexts.size(); - assert(size <= Malta::Max_CPUs); - - for (int i = 0; i < size; i++) { - uint64_t cpumask = ULL(1) << i; - if (!(cpumask & itint)) { - itint |= cpumask; - malta->intrctrl->post(i, TheISA::INTLEVEL_IRQ2, 0); - DPRINTF(Malta, "Posting RTC interrupt to cpu=%d", i); - } - } - */ - + panic("MaltaCCHIP::postRTC() not implemented."); } void @@ -478,12 +109,12 @@ MaltaCChip::postIntr(uint32_t interrupt) assert(size <= Malta::Max_CPUs); for (int i=0; i < size; i++) { - //Note: Malta does not use index, but this was added to use the pre-existing implementation - malta->intrctrl->post(i, interrupt, 0); - DPRINTF(Malta, "posting interrupt to cpu %d," - "interrupt %d\n",i, interrupt); + //Note: Malta does not use index, but this was added to use the + //pre-existing implementation + malta->intrctrl->post(i, interrupt, 0); + DPRINTF(Malta, "posting interrupt to cpu %d, interrupt %d\n", + i, interrupt); } - } void @@ -493,10 +124,11 @@ MaltaCChip::clearIntr(uint32_t interrupt) assert(size <= Malta::Max_CPUs); for (int i=0; i < size; i++) { - //Note: Malta does not use index, but this was added to use the pre-existing implementation - malta->intrctrl->clear(i, interrupt, 0); - DPRINTF(Malta, "clearing interrupt to cpu %d," - "interrupt %d\n",i, interrupt); + //Note: Malta does not use index, but this was added to use the + //pre-existing implementation + malta->intrctrl->clear(i, interrupt, 0); + DPRINTF(Malta, "clearing interrupt to cpu %d, interrupt %d\n", + i, interrupt); } } @@ -504,21 +136,11 @@ MaltaCChip::clearIntr(uint32_t interrupt) void MaltaCChip::serialize(CheckpointOut &cp) const { - // SERIALIZE_ARRAY(dim, Malta::Max_CPUs); - //SERIALIZE_ARRAY(dir, Malta::Max_CPUs); - //SERIALIZE_SCALAR(ipint); - //SERIALIZE_SCALAR(itint); - //SERIALIZE_SCALAR(drir); } void MaltaCChip::unserialize(CheckpointIn &cp) { - //UNSERIALIZE_ARRAY(dim, Malta::Max_CPUs); - //UNSERIALIZE_ARRAY(dir, Malta::Max_CPUs); - //UNSERIALIZE_SCALAR(ipint); - //UNSERIALIZE_SCALAR(itint); - //UNSERIALIZE_SCALAR(drir); } MaltaCChip * diff --git a/src/dev/mips/malta_io.cc b/src/dev/mips/malta_io.cc index 96de7f977..66f66dfbb 100644 --- a/src/dev/mips/malta_io.cc +++ b/src/dev/mips/malta_io.cc @@ -44,7 +44,6 @@ #include "base/time.hh" #include "base/trace.hh" -#include "config/the_isa.hh" #include "debug/Malta.hh" #include "dev/mips/malta.hh" #include "dev/mips/malta_cchip.hh" @@ -57,7 +56,6 @@ #include "sim/system.hh" using namespace std; -using namespace TheISA; MaltaIO::RTC::RTC(const string &name, const MaltaIOParams *p) : MC146818(p->malta, name, p->time, p->year_is_bcd, p->frequency), -- 2.30.2