----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3235/ -----------------------------------------------------------
Review request for Default. Repository: gem5 Description ------- Changeset 11229:f4b15e5109f9 --------------------------- stats: make sim_insts and sim_ops respect stats reset Because sim_insts and sim_ops were being calculated using the ThreadState variable numInst/numOp (type Counter) rather than numInsts/numOps (type Stats::Scalar), they were not getting reset. This behavior is confusing because almost all other entries in the stats file do get reset (with the exception of final_tick, which notes it is never reset in the stats file). It also leads to incorrect behavior with stats like host_inst_rate, which reset the host time but not the instructions executed. This patch resets sim_insts and sim_ops. Diffs ----- src/cpu/minor/cpu.cc 021524c21cbc src/cpu/o3/cpu.cc 021524c21cbc src/cpu/simple/base.cc 021524c21cbc Diff: http://reviews.gem5.org/r/3235/diff/ Testing ------- Thanks, Lena Olson _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
