Giacomo Travaglini has submitted this change and it was merged. ( https://gem5-review.googlesource.com/11831 )

Change subject: cpu: Warn when (un)serializing a traffic generator
......................................................................

cpu: Warn when (un)serializing a traffic generator

When checkpointing a system with a traffic generator, a warning is
produced so that the user is reminded serialization does not keep all
the traffic generator internal state.

Change-Id: I3c49c912c9ff3a4208f55b2da0a88fc694147280
Signed-off-by: Giacomo Travaglini <[email protected]>
Reviewed-on: https://gem5-review.googlesource.com/11831
Reviewed-by: Anthony Gutierrez <[email protected]>
Reviewed-by: Jason Lowe-Power <[email protected]>
Maintainer: Jason Lowe-Power <[email protected]>
---
M src/cpu/testers/traffic_gen/base.cc
1 file changed, 6 insertions(+), 0 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  Anthony Gutierrez: Looks good to me, approved



diff --git a/src/cpu/testers/traffic_gen/base.cc b/src/cpu/testers/traffic_gen/base.cc
index 9ca084d..41d0052 100644
--- a/src/cpu/testers/traffic_gen/base.cc
+++ b/src/cpu/testers/traffic_gen/base.cc
@@ -124,6 +124,9 @@
 void
 BaseTrafficGen::serialize(CheckpointOut &cp) const
 {
+    warn("%s serialization does not keep all traffic generator"
+         " internal state\n", name());
+
     DPRINTF(Checkpoint, "Serializing BaseTrafficGen\n");

     // save ticks of the graph event if it is scheduled
@@ -141,6 +144,9 @@
 void
 BaseTrafficGen::unserialize(CheckpointIn &cp)
 {
+    warn("%s serialization does not restore all traffic generator"
+         " internal state\n", name());
+
     // restore scheduled events
     Tick nextEvent;
     UNSERIALIZE_SCALAR(nextEvent);

--
To view, visit https://gem5-review.googlesource.com/11831
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I3c49c912c9ff3a4208f55b2da0a88fc694147280
Gerrit-Change-Number: 11831
Gerrit-PatchSet: 4
Gerrit-Owner: Giacomo Travaglini <[email protected]>
Gerrit-Reviewer: Anthony Gutierrez <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Giacomo Travaglini <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to