sim: reset stats after startup

Information
Submitter:	Nilay Vaish
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 9868:112a9778300a
---------------------------
sim: reset stats after startup
Currently statistics are reset after the initial / checkpoint state
has been loaded. But ruby does some checkpoint processing in its
startup() function. So the stats need to be reset after the startup()
function has been called. This patch moves the class to stats.reset()
to achieve this change in functionality.

Testing Done

Any comments? I am planning to commit it on Friday.

Andreas Hansson Sept. 18, 2013, 4:14 p.m. (Sept. 18, 2013, 4:14 p.m.)

Do any regressions change?

Would it be an option to change the aforementioned functionality so that what ever is causing the issue does not happen in startup (but rather unserialize I would imagine)?

Nilay Vaish Nov. 16, 2013, 11:31 p.m. (Nov. 16, 2013, 11:31 p.m.)

While it seems to me that we can move ruby's startup code to
the unserialize() function, I don't know if calling simulate()
while objects are still being unserialized is a proper thing.

Seems like it would be simpler just to move the stats.reset() call inside the 'if need_startup:' block, right?

Nilay Vaish Nov. 16, 2013, 11:31 p.m. (Nov. 16, 2013, 11:31 p.m.)
```
Yes
```

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Status: Closed (submitted)