diff options
author | Gabe Black <gabeblack@google.com> | 2018-01-14 11:44:04 -0800 |
---|---|---|
committer | Gabe Black <gabeblack@google.com> | 2018-01-15 06:13:55 +0000 |
commit | 3e8d76e2e23ab23d68e569e9b2c4498a06c19f59 (patch) | |
tree | 24cd226f1a7327b4969ceabe65faa44d5353df42 | |
parent | 5320a97ced43d4452014ff54c0ba45246db90a00 (diff) | |
download | gem5-3e8d76e2e23ab23d68e569e9b2c4498a06c19f59.tar.xz |
arch: Fix a fatal_if in most of the arch's process classes.
When switching an assert to a fatal while addressing recent review
feedback, I forgot to reverse the polarity of the condition, making
the fatal fire in exactly the opposite of the conditions it was meant
to.
Change-Id: Icf49864ef449052bbb0d427dca786006166575c4
Reviewed-on: https://gem5-review.googlesource.com/7381
Reviewed-by: Matthias Jung <jungma@eit.uni-kl.de>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>
-rw-r--r-- | src/arch/alpha/process.cc | 2 | ||||
-rw-r--r-- | src/arch/arm/process.cc | 2 | ||||
-rw-r--r-- | src/arch/mips/process.cc | 2 | ||||
-rw-r--r-- | src/arch/power/process.cc | 2 | ||||
-rw-r--r-- | src/arch/riscv/process.cc | 2 | ||||
-rw-r--r-- | src/arch/sparc/process.cc | 2 |
6 files changed, 6 insertions, 6 deletions
diff --git a/src/arch/alpha/process.cc b/src/arch/alpha/process.cc index bcfe36270..2c6f83309 100644 --- a/src/arch/alpha/process.cc +++ b/src/arch/alpha/process.cc @@ -52,7 +52,7 @@ AlphaProcess::AlphaProcess(ProcessParams *params, ObjectFile *objFile) : Process(params, new FuncPageTable(params->name, params->pid, PageBytes), objFile) { - fatal_if(!params->useArchPT, "Arch page tables not implemented."); + fatal_if(params->useArchPT, "Arch page tables not implemented."); Addr brk_point = objFile->dataBase() + objFile->dataSize() + objFile->bssSize(); brk_point = roundUp(brk_point, PageBytes); diff --git a/src/arch/arm/process.cc b/src/arch/arm/process.cc index 5daa54a1f..aef714ed3 100644 --- a/src/arch/arm/process.cc +++ b/src/arch/arm/process.cc @@ -67,7 +67,7 @@ ArmProcess::ArmProcess(ProcessParams *params, ObjectFile *objFile, objFile), arch(_arch) { - fatal_if(!params->useArchPT, "Arch page tables not implemented."); + fatal_if(params->useArchPT, "Arch page tables not implemented."); } ArmProcess32::ArmProcess32(ProcessParams *params, ObjectFile *objFile, diff --git a/src/arch/mips/process.cc b/src/arch/mips/process.cc index 76f7e869c..62ccd5afe 100644 --- a/src/arch/mips/process.cc +++ b/src/arch/mips/process.cc @@ -53,7 +53,7 @@ MipsProcess::MipsProcess(ProcessParams *params, ObjectFile *objFile) : Process(params, new FuncPageTable(params->name, params->pid, PageBytes), objFile) { - fatal_if(!params->useArchPT, "Arch page tables not implemented."); + fatal_if(params->useArchPT, "Arch page tables not implemented."); // Set up stack. On MIPS, stack starts at the top of kuseg // user address space. MIPS stack grows down from here Addr stack_base = 0x7FFFFFFF; diff --git a/src/arch/power/process.cc b/src/arch/power/process.cc index 343cd4b4e..22627efe5 100644 --- a/src/arch/power/process.cc +++ b/src/arch/power/process.cc @@ -53,7 +53,7 @@ PowerProcess::PowerProcess(ProcessParams *params, ObjectFile *objFile) : Process(params, new FuncPageTable(params->name, params->pid, PageBytes), objFile) { - fatal_if(!params->useArchPT, "Arch page tables not implemented."); + fatal_if(params->useArchPT, "Arch page tables not implemented."); // Set up break point (Top of Heap) Addr brk_point = objFile->dataBase() + objFile->dataSize() + objFile->bssSize(); diff --git a/src/arch/riscv/process.cc b/src/arch/riscv/process.cc index 44b276a4b..73df5f50d 100644 --- a/src/arch/riscv/process.cc +++ b/src/arch/riscv/process.cc @@ -64,7 +64,7 @@ RiscvProcess::RiscvProcess(ProcessParams *params, ObjectFile *objFile) : PageBytes), objFile) { - fatal_if(!params->useArchPT, "Arch page tables not implemented."); + fatal_if(params->useArchPT, "Arch page tables not implemented."); const Addr stack_base = 0x7FFFFFFFFFFFFFFFL; const Addr max_stack_size = 8 * 1024 * 1024; const Addr next_thread_stack_base = stack_base - max_stack_size; diff --git a/src/arch/sparc/process.cc b/src/arch/sparc/process.cc index 59ef5c4ac..1fcb6cba3 100644 --- a/src/arch/sparc/process.cc +++ b/src/arch/sparc/process.cc @@ -60,7 +60,7 @@ SparcProcess::SparcProcess(ProcessParams *params, ObjectFile *objFile, objFile), StackBias(_StackBias) { - fatal_if(!params->useArchPT, "Arch page tables not implemented."); + fatal_if(params->useArchPT, "Arch page tables not implemented."); // Initialize these to 0s fillStart = 0; spillStart = 0; |