----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1574/#review3729 -----------------------------------------------------------
Ship it! Ship It! - Ali Saidi On Dec. 6, 2012, 8:20 p.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1574/ > ----------------------------------------------------------- > > (Updated Dec. 6, 2012, 8:20 p.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9369:64fdde7bd4a2 > --------------------------- > mem: Add tracing support in the communication monitor > > This patch adds packet tracing to the communication monitor using a > protobuf as the mechanism for creating the trace. > > If no file is specified, then the tracing is disabled. If a file is > specified, then for every packet that is successfully sent, a protobuf > message is serialized to the file. > > > Diffs > ----- > > src/base/output.hh 844f9e724343 > src/base/output.cc 844f9e724343 > src/mem/CommMonitor.py 844f9e724343 > src/mem/comm_monitor.hh 844f9e724343 > src/mem/comm_monitor.cc 844f9e724343 > src/proto/SConscript PRE-CREATION > src/proto/packet.proto PRE-CREATION > tests/configs/tgen-simple-mem.py 844f9e724343 > > Diff: http://reviews.gem5.org/r/1574/diff/ > > > Testing > ------- > > util/regress all passing (disregarding t1000 and eio) > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
