sim_cur_ticks: display curTick in stats

Information
Submitter:	Ali Saidi
Repository:	gem5
Branch:
Bugs:
Depends On:
Reviewers
Groups:	Default
People:	ali, gblack, nate, stever

Description

sim: display curTick in stats
Different from sim_ticks in that this value is restored from checkpoints and is never reset.
Useful for aligning with framebuffer output ticks

Testing Done

src/sim/stat_control.cc (Diff revision 1)

How about calling it simTotTicks or simAbsTicks? (Total or Absolute that is)

Korey Sewell Dec. 1, 2011, 3:48 a.m. (Dec. 1, 2011, 3:48 a.m.)

I think that the "sim_insts" stat is a persistent one whereas the "sim_ticks" stat is not persistent.

That typically causes a lot of confusion.

I would suggest that we migrate to making "sim_ticks" persistent and then using the simCurTicks to match the reset/checkpointed stat value.

Nathan Binkert Dec. 1, 2011, 3:53 a.m. (Dec. 1, 2011, 3:53 a.m.)

I don't have much to say on specific names, but if we want to avoid confusion, we should have unambiguous names.  like sim_insts and sim_insts_non_reset.  We should be making it very clear what if any stats are unaffected by reset() or are restored from a checkpoint.

Gabe Black Dec. 1, 2011, 7:14 a.m. (Dec. 1, 2011, 7:14 a.m.)

My vote would be to make sim_ticks persistent like sim_insts and then add new stats like interval_ticks and interval_insts or sim_ticks_since_reset and sim_insts_since_reset.

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Status: Closed (submitted)