diff options
| author | Lena Olson <lena@cs.wisc.edu> | 2015-04-03 11:42:10 -0500 |
|---|---|---|
| committer | Lena Olson <lena@cs.wisc.edu> | 2015-04-03 11:42:10 -0500 |
| commit | 333988a73e06cb4067f113b7043ab68dd0e86c9d (patch) | |
| tree | d1be35ab40f368e2f62bbd3b08944a0af7b5dbb4 /src/arch | |
| parent | aeffde5ed5007b5ebe330b03156e5cc58c5fe609 (diff) | |
| download | gem5-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.isa | 4 |
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(); } }}; |