summaryrefslogtreecommitdiff
path: root/src/arch
diff options
context:
space:
mode:
authorLena Olson <lena@cs.wisc.edu>2015-04-03 11:42:10 -0500
committerLena Olson <lena@cs.wisc.edu>2015-04-03 11:42:10 -0500
commit333988a73e06cb4067f113b7043ab68dd0e86c9d (patch)
treed1be35ab40f368e2f62bbd3b08944a0af7b5dbb4 /src/arch
parentaeffde5ed5007b5ebe330b03156e5cc58c5fe609 (diff)
downloadgem5-333988a73e06cb4067f113b7043ab68dd0e86c9d.tar.xz
x86: fix debug trace output for mwait
When running with the Exec flag, the mwait instruction attempted to print out its source registers, which were never actually initialized. This led to sporadic assertion failures when the value stored there was invalid. Committed by: Nilay Vaish <nilay@cs.wisc.edu>
Diffstat (limited to 'src/arch')
-rw-r--r--src/arch/x86/isa/formats/monitor_mwait.isa4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/arch/x86/isa/formats/monitor_mwait.isa b/src/arch/x86/isa/formats/monitor_mwait.isa
index 493b7c58a..b26c1f267 100644
--- a/src/arch/x86/isa/formats/monitor_mwait.isa
+++ b/src/arch/x86/isa/formats/monitor_mwait.isa
@@ -112,9 +112,9 @@ output decoder {{
{
std::stringstream response;
+ // Although mwait could take hints from eax and ecx, the _srcRegIdx
+ // is not set, and thus should not be printed here
printMnemonic(response, mnemonic);
- ccprintf(response, " ");
- printReg(response, _srcRegIdx[0], machInst.opSize);
return response.str();
}
}};