From 110c59b414e1e93004f56cefcb92ea945c3f3713 Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Mon, 19 Sep 2011 06:17:19 -0700 Subject: [PATCH] MIPS: Get rid of cruft in the fault classes. Get rid of Fault classes left over from when this file was copied from Alpha, and rename ArithmeticOverflowFault to be IntegerOverflowFault and get rid of the old IntegerOverflowFault stub. The Integer version is what's actually in the manual, but the Arithmetic version had the implementation. --- src/arch/mips/faults.cc | 39 ++---------------- src/arch/mips/faults.hh | 62 +---------------------------- src/arch/mips/isa/decoder.isa | 6 +-- src/arch/mips/isa/formats/unimp.isa | 8 ++-- src/arch/mips/tlb.cc | 5 ++- 5 files changed, 15 insertions(+), 105 deletions(-) diff --git a/src/arch/mips/faults.cc b/src/arch/mips/faults.cc index 87a69669e..464c2fe9e 100644 --- a/src/arch/mips/faults.cc +++ b/src/arch/mips/faults.cc @@ -51,9 +51,6 @@ typedef MipsFaultBase::FaultVals FaultVals; template <> FaultVals MipsFault::vals = { "Machine Check", 0x0401 }; -template <> FaultVals MipsFault::vals = - { "Alignment", 0x0301 }; - template <> FaultVals MipsFault::vals = #if FULL_SYSTEM { "Reset Fault", 0xBFC00000}; @@ -79,11 +76,8 @@ template <> FaultVals MipsFault::vals = template <> FaultVals MipsFault::vals = { "Thread Fault", 0x00F1 }; -template <> FaultVals MipsFault::vals = - { "Arithmetic Overflow Exception", 0x180 }; - -template <> FaultVals MipsFault::vals = - { "opdec", 0x0481 }; +template <> FaultVals MipsFault::vals = + { "Integer Overflow Exception", 0x180 }; template <> FaultVals MipsFault::vals = { "interrupt", 0x0180 }; @@ -97,30 +91,9 @@ template <> FaultVals MipsFault::vals = template <> FaultVals MipsFault::vals = { "Invalid TLB Entry Exception (I-Fetch/LW)", 0x0180 }; -template <> FaultVals MipsFault::vals = - { "itbmiss", 0x0181 }; - -template <> FaultVals MipsFault::vals = - { "itbmiss", 0x0181 }; - -template <> FaultVals MipsFault::vals = - { "iaccvio", 0x0081 }; - template <> FaultVals MipsFault::vals = { "TLB Refill Exception (I-Fetch/LW)", 0x0180 }; -template <> FaultVals MipsFault::vals = - { "dtb_miss_single", 0x0201 }; - -template <> FaultVals MipsFault::vals = - { "dtb_miss_double", 0x0281 }; - -template <> FaultVals MipsFault::vals = - { "dfault", 0x0381 }; - -template <> FaultVals MipsFault::vals = - { "dfault", 0x0381 }; - template <> FaultVals MipsFault::vals = { "Invalid TLB Entry Exception (Store)", 0x0180 }; @@ -130,12 +103,6 @@ template <> FaultVals MipsFault::vals = template <> FaultVals MipsFault::vals = { "TLB Modified Exception", 0x0180 }; -template <> FaultVals MipsFault::vals = - { "float_enable_fault", 0x0581 }; - -template <> FaultVals MipsFault::vals = - { "Integer Overflow Fault", 0x0501 }; - template <> FaultVals MipsFault::vals = { "DSP Disabled Fault", 0x001a }; @@ -190,7 +157,7 @@ MipsFaultBase::setExceptionState(ThreadContext *tc, uint8_t excCode) } void -ArithmeticFault::invoke(ThreadContext *tc, StaticInstPtr inst) +IntegerOverflowFault::invoke(ThreadContext *tc, StaticInstPtr inst) { DPRINTF(MipsPRA, "%s encountered.\n", name()); setExceptionState(tc, 0xC); diff --git a/src/arch/mips/faults.hh b/src/arch/mips/faults.hh index 055aa597e..f04f757f0 100644 --- a/src/arch/mips/faults.hh +++ b/src/arch/mips/faults.hh @@ -91,12 +91,6 @@ class NonMaskableInterrupt : public MipsFault bool isNonMaskableInterrupt() {return true;} }; -class AlignmentFault : public MipsFault -{ - public: - bool isAlignmentFault() {return true;} -}; - class AddressErrorFault : public MipsFault { public: @@ -118,37 +112,11 @@ class StoreAddressErrorFault : public MipsFault #endif }; -class UnimplementedOpcodeFault : public MipsFault {}; - -class TLBRefillIFetchFault : public MipsFault -{ - public: - void invoke(ThreadContext * tc, - StaticInstPtr inst = StaticInst::nullStaticInstPtr); -}; - -class TLBInvalidIFetchFault : public MipsFault -{ - public: - void invoke(ThreadContext * tc, - StaticInstPtr inst = StaticInst::nullStaticInstPtr); -}; - -class NDtbMissFault : public MipsFault {}; -class PDtbMissFault : public MipsFault {}; -class DtbPageFault : public MipsFault {}; -class DtbAcvFault : public MipsFault {}; - static inline Fault genMachineCheckFault() { return new MachineCheckFault; } -static inline Fault genAlignmentFault() -{ - return new AlignmentFault; -} - class ResetFault : public MipsFault { public: @@ -173,20 +141,6 @@ class SoftResetFault : public MipsFault StaticInstPtr inst = StaticInst::nullStaticInstPtr); }; -class DebugSingleStep : public MipsFault -{ - public: - void invoke(ThreadContext * tc, - StaticInstPtr inst = StaticInst::nullStaticInstPtr); -}; - -class DebugInterrupt : public MipsFault -{ - public: - void invoke(ThreadContext * tc, - StaticInstPtr inst = StaticInst::nullStaticInstPtr); -}; - class CoprocessorUnusableFault : public MipsFault { protected: @@ -213,7 +167,7 @@ class ThreadFault : public MipsFault StaticInstPtr inst = StaticInst::nullStaticInstPtr); }; -class ArithmeticFault : public MipsFault +class IntegerOverflowFault : public MipsFault { protected: bool skipFaultingInstruction() {return true;} @@ -287,15 +241,6 @@ class DtbRefillFault : public MipsFault #endif }; -class ItbPageFault : public MipsFault -{ - public: -#if FULL_SYSTEM - void invoke(ThreadContext * tc, - StaticInstPtr inst = StaticInst::nullStaticInstPtr); -#endif -}; - class ItbInvalidFault : public MipsFault { public: @@ -347,11 +292,6 @@ class DtbInvalidFault : public MipsFault #endif }; -class FloatEnableFault : public MipsFault {}; -class ItbMissFault : public MipsFault {}; -class ItbAcvFault : public MipsFault {}; -class IntegerOverflowFault : public MipsFault {}; - class DspStateDisabledFault : public MipsFault { public: diff --git a/src/arch/mips/isa/decoder.isa b/src/arch/mips/isa/decoder.isa index 4ed8c0c0c..eca7d7411 100644 --- a/src/arch/mips/isa/decoder.isa +++ b/src/arch/mips/isa/decoder.isa @@ -214,7 +214,7 @@ decode OPCODE_HI default Unknown::unknown() { Rd = result = Rs + Rt; if (FULL_SYSTEM && findOverflow(32, result, Rs, Rt)) { - fault = new ArithmeticFault(); + fault = new IntegerOverflowFault(); } }}); 0x1: addu({{ Rd.sw = Rs.sw + Rt.sw;}}); @@ -223,7 +223,7 @@ decode OPCODE_HI default Unknown::unknown() { Rd = result = Rs - Rt; if (FULL_SYSTEM && findOverflow(32, result, Rs, ~Rt)) { - fault = new ArithmeticFault(); + fault = new IntegerOverflowFault(); } }}); 0x3: subu({{ Rd.sw = Rs.sw - Rt.sw; }}); @@ -327,7 +327,7 @@ decode OPCODE_HI default Unknown::unknown() { Rt = result = Rs + imm; if (FULL_SYSTEM && findOverflow(32, result, Rs, imm)) { - fault = new ArithmeticFault(); + fault = new IntegerOverflowFault(); } }}); 0x1: addiu({{ Rt.sw = Rs.sw + imm; }}); diff --git a/src/arch/mips/isa/formats/unimp.isa b/src/arch/mips/isa/formats/unimp.isa index e599510a4..65b4425af 100644 --- a/src/arch/mips/isa/formats/unimp.isa +++ b/src/arch/mips/isa/formats/unimp.isa @@ -186,7 +186,7 @@ output exec {{ panic("attempt to execute unimplemented instruction '%s' " "(inst 0x%08x, opcode 0x%x, binary:%s)", mnemonic, machInst, OPCODE, inst2string(machInst)); - return new UnimplementedOpcodeFault; + return NoFault; } Fault @@ -202,7 +202,7 @@ output exec {{ panic("attempt to execute unimplemented instruction '%s' " "(inst %#08x, opcode %#x, binary:%s)", mnemonic, machInst, OPCODE, inst2string(machInst)); - return new UnimplementedOpcodeFault; + return NoFault; } } @@ -219,7 +219,7 @@ output exec {{ panic("attempt to execute unimplemented instruction '%s' " "(inst %#08x, opcode %#x, binary:%s)", mnemonic, machInst, OPCODE, inst2string(machInst)); - return new UnimplementedOpcodeFault; + return NoFault; } } @@ -236,7 +236,7 @@ output exec {{ panic("attempt to execute unimplemented instruction '%s' " "(inst %#08x, opcode %#x, binary:%s)", mnemonic, machInst, OPCODE, inst2string(machInst)); - return new UnimplementedOpcodeFault; + return NoFault; } } diff --git a/src/arch/mips/tlb.cc b/src/arch/mips/tlb.cc index b2286850e..f379b9919 100644 --- a/src/arch/mips/tlb.cc +++ b/src/arch/mips/tlb.cc @@ -387,7 +387,10 @@ TLB::translateData(RequestPtr req, ThreadContext *tc, bool write) if (req->getVaddr() & (req->getSize() - 1)) { DPRINTF(TLB, "Alignment Fault on %#x, size = %d", req->getVaddr(), req->getSize()); - return new AlignmentFault(); + if (write) + return new StoreAddressErrorFault(req->getVaddr()); + else + return new AddressErrorFault(req->getVaddr()); } -- 2.30.2