----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3601/#review8638 -----------------------------------------------------------
Just a small thing below. Other than that it looks good. src/mem/dram_ctrl.cc (line 1928) <http://reviews.gem5.org/r/3601/#comment7494> Should this be here? Or in the constructor or ::init()? I'm not sure where the right place is.. but I don't think that regStats() is it. I think it's kind of hidden here. - Jason Lowe-Power On Aug. 11, 2016, 9:08 a.m., Curtis Dunham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3601/ > ----------------------------------------------------------- > > (Updated Aug. 11, 2016, 9:08 a.m.) > > > Review request for Default and Matthias Jung. > > > Repository: gem5 > > > Description > ------- > > mem: Add callback to compute stats prior to dump event > > The per rank statistics are periodically updated based on > state transition and refresh events. > > Add a method to update these when a dump event occurs to > ensure they reflect accurate values. > Specifically, need to ensure that the low-power state > durations, power, and energy are logged correctly. > > Change-Id: Ib642a6668340de8f494a608bb34982e58ba7f1eb > Reviewed-by: Radhika Jagtap <[email protected]> > > > Diffs > ----- > > src/mem/dram_ctrl.cc e9096175eb38ac39f37c91bfdf2a450b9664e222 > src/mem/dram_ctrl.hh e9096175eb38ac39f37c91bfdf2a450b9664e222 > > Diff: http://reviews.gem5.org/r/3601/diff/ > > > Testing > ------- > > > Thanks, > > Curtis Dunham > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
