----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2076/#review4802 -----------------------------------------------------------
Concerning the changes to the DRAM controller, are there any additional comments? If not, I will consider them good to go. - Andreas Hansson On Oct. 31, 2013, 9:46 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2076/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2013, 9:46 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9964:2db0eb2ddd45 > --------------------------- > mem: Fixes for DRAM stats accounting > > This patch fixes a number of stats accounting issues in the DRAM > controller. Most importantly, it separates the system interface and > DRAM interface so that it is clearer what the actual DRAM bandwidth > (and consequently utilisation) is. > > > Diffs > ----- > > src/mem/simple_dram.hh cd0a9c975c8c > src/mem/simple_dram.cc cd0a9c975c8c > > Diff: http://reviews.gem5.org/r/2076/diff/ > > > Testing > ------- > > All regressions pass with stats update > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
