summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNilay Vaish <nilay@cs.wisc.edu>2015-04-29 22:35:22 -0500
committerNilay Vaish <nilay@cs.wisc.edu>2015-04-29 22:35:22 -0500
commitee06fed656d404c19c68c838df1dc8dbba37ec80 (patch)
tree4bca05681a3e4939994200c6974afea72370bc8f
parent179787f31faf12f7f4a31506db40236e82873675 (diff)
downloadgem5-ee06fed656d404c19c68c838df1dc8dbba37ec80.tar.xz
x86: change divide-by-zero fault to divide-error
Same exception is raised whether division with zero is performed or the quotient is greater than the maximum value that the provided space can hold. Divide-by-Zero is the AMD terminology, while Divide-Error is Intel's.
-rw-r--r--src/arch/x86/faults.hh9
-rw-r--r--src/arch/x86/isa/microops/regop.isa4
2 files changed, 7 insertions, 6 deletions
diff --git a/src/arch/x86/faults.hh b/src/arch/x86/faults.hh
index b9eb85e21..f8715ded2 100644
--- a/src/arch/x86/faults.hh
+++ b/src/arch/x86/faults.hh
@@ -167,7 +167,7 @@ namespace X86ISA
// Class | Type | vector | Cause | mnem
//------------------------------------------------------------------------
- //Contrib Fault 0 Divide-by-Zero-Error #DE
+ //Contrib Fault 0 Divide Error #DE
//Benign Either 1 Debug #DB
//Benign Interrupt 2 Non-Maskable-Interrupt #NMI
//Benign Trap 3 Breakpoint #BP
@@ -193,11 +193,12 @@ namespace X86ISA
//Benign Interrupt 0-255 External Interrupts #INTR
//Benign Interrupt 0-255 Software Interrupts INTn
- class DivideByZero : public X86Fault
+ // Note that
+ class DivideError : public X86Fault
{
public:
- DivideByZero() :
- X86Fault("Divide-by-Zero-Error", "#DE", 0)
+ DivideError() :
+ X86Fault("Divide-Error", "#DE", 0)
{}
};
diff --git a/src/arch/x86/isa/microops/regop.isa b/src/arch/x86/isa/microops/regop.isa
index 7d0374f02..15515ed12 100644
--- a/src/arch/x86/isa/microops/regop.isa
+++ b/src/arch/x86/isa/microops/regop.isa
@@ -629,7 +629,7 @@ let {{
uint64_t dividend = remainder;
//Do the division.
if (divisor == 0) {
- fault = std::make_shared<DivideByZero>();
+ fault = std::make_shared<DivideError>();
} else {
divide(dividend, divisor, quotient, remainder);
//Record the final results.
@@ -652,7 +652,7 @@ let {{
//If we overshot, do nothing. This lets us unrool division loops a
//little.
if (divisor == 0) {
- fault = std::make_shared<DivideByZero>();
+ fault = std::make_shared<DivideError>();
} else if (remaining) {
if (divisor & (ULL(1) << 63)) {
while (remaining && !(dividend & (ULL(1) << 63))) {