sim: added option to serialize SimLoopExitEvent
Review Request #2064 - Created Oct. 17, 2013 and submitted
| Information | |
|---|---|
| Ali Saidi | |
| gem5 | |
| default | |
| Reviewers | |
| Default | |
Changeset 9870:e36d1a8d6ffc --------------------------- sim: added option to serialize SimLoopExitEvent SimLoopExitEvents weren't serialized by default. Some benchmarks utilize a delayed m5 exit pseudo op call to terminate the simulation and this event was lost when resuming from a checkpoint generated after the pseudo op call. This patch adds the capability to serialize the SimLoopExitEvents and enable serialization for m5_exit and m5_fail pseudo ops by default. Does not affect other generic SimLoopExitEvents.
Posted (Oct. 18, 2013, 1:19 a.m.)
Looks good other than some indentation issues. I quickly checked the style guide and it doesn't mention how parameter lists should be indented, but it seems like the most common way to do it is to align the first parameter on each row.
-
src/sim/sim_events.hh (Diff revision 1) -
Indentation
-
src/sim/sim_events.cc (Diff revision 1) -
Indentation
-
src/sim/sim_events.cc (Diff revision 1) -
Shouldn't this just be SERIALIZE_SCALAR(cause)?
-
src/sim/sim_events.cc (Diff revision 1) -
UNSERIALIZE_SCALAR(cause)?
-
src/sim/sim_events.cc (Diff revision 1) -
Indentation