cpu,sim-se: move error checks in syscall methods
authorBrandon Potter <brandon.potter@amd.com>
Sun, 1 Dec 2019 19:23:22 +0000 (14:23 -0500)
committerBrandon Potter <Brandon.Potter@amd.com>
Tue, 3 Dec 2019 16:52:59 +0000 (16:52 +0000)
commitc63a00d65737a2f556637945a548a0bbaeb61b39
treecbec51a00141ea61bbee429668afb628929d2119
parente5203b88eac6eed12674327c5cd0ac658a27573b
cpu,sim-se: move error checks in syscall methods

There is a check on a global flag denoting that the simulator
has been configured to run in fullsystem mode. The check is
conducted at runtime during calls to syscall methods.

The high-level models are checking the flag when the check
could be conducted further down the call chain (nearer to the
actual Process invocation). Moving the checks should result
in less copy-pasta as new models are developed. It might be
argued that the checks should stay in place since an error
would detected earlier; that may be true, but the error
would be the same and the simulation should fail in either
case. This arrangement requires fewer lines of code.

The changeset also changes the check into a fatal error
instead of a panic since usage (in fs mode) should result
in immediate corruption.

Change-Id: If387e27f166ac1374f3fe8b7befe3546e69adba7
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/23240
Reviewed-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/cpu/minor/exec_context.hh
src/cpu/o3/dyn_inst_impl.hh
src/cpu/o3/thread_state.hh
src/cpu/simple/exec_context.hh
src/cpu/simple_thread.hh