On Thu, 27 Sep 2012, Ashish Venkat wrote:
I applied the following patch to fix the IMUL_R_P_I macroop:
http://repo.gem5.org/gem5?cmd=changeset;node=d69afa89c2ee
However, it is still not working for me. The execution trace shows:
39579500: system.cpu T0 : @loadAndRLEsource+130.2 : IMUL_R_P_I : ld
t2d, DS:[t7] : MemRead : D=0x00000000ca3a1d8b A=0x1000361c
instead of
39579500: system.cpu T0 : @loadAndRLEsource+130.2 : IMUL_R_P_I : ld
t2d, DS:[t7 + 0x54fca3a] : MemRead : D=0x00000000ca3a1d8b
A=0x1000361c
In fact, any change I make to the file multiply_and_divide.py does not
have any effect on the execution. For instance, I removed some
micro-instructions completely from one of the macro-ops in the file.
The execution trace, however, still shows those instructions.
I looked into the ISA code for X86 and it looks to me that this is the
only place that needs the change. Just to make sure that it is not my
build environment, I removed my build directory and performed a clean
build. Am I missing something here? Can someone please clarify?
This is really surprising and I strongly suggest that you recheck the
observations that you have made in your email. Firstly, the patch made it
to the mainline five months ago. Why do you need to apply it? Unless you
have a strong reason to work with an older version, I suggest that you
update to a recent one. Secondly, it seems that you did not recompile gem5
after you removed micro instructions. IIRC, you should be able to confirm
that the micro instructions you removed will no longer be executed by
checking the file build/X86/arch/x86/generated/decoder.cc.
--
Nilay
_______________________________________________
gem5-users mailing list
gem5-users@gem5.org
http://m5sim.org/cgi-bin/mailman/listinfo/gem5-users
