arch: alpha: Fix an 8 year old bug from the transition to pc objects.
authorGabe Black <gabeblack@google.com>
Tue, 3 Apr 2018 01:59:14 +0000 (18:59 -0700)
committerGabe Black <gabeblack@google.com>
Fri, 6 Apr 2018 23:47:22 +0000 (23:47 +0000)
When we transitioned from having PCs and nextPCs stored as individual items
in the ThreadContext and went to having PC objects with abstracted
components (micro PCs, delay slots, variable instruction lengths, etc.), a
small mistake was made when translating the alpha StackTrace code.

Change-Id: Ib5dd65c53a26920e66899f421902607efd139e71
Reviewed-on: https://gem5-review.googlesource.com/9702
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Gabe Black <gabeblack@google.com>

src/arch/alpha/stacktrace.cc

index 1dc5956c7e627018e598eef9adfae2a64d2bd0de..3c302d27d25078dde0e1f6555d8629d9a74ad094 100644 (file)
@@ -143,7 +143,7 @@ StackTrace::trace(ThreadContext *_tc, bool is_call)
     bool usermode =
         (tc->readMiscRegNoEffect(IPR_DTB_CM) & 0x18) != 0;
 
-    Addr pc = tc->pcState().pc();
+    Addr pc = tc->pcState().npc();
     bool kernel = sys->kernelStart <= pc && pc <= sys->kernelEnd;
 
     if (usermode) {