----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3599/#review8640 -----------------------------------------------------------
src/mem/dram_ctrl.hh (line 340) <http://reviews.gem5.org/r/3599/#comment7495> Since you want this to be sorted, would it make more sense to use a sorted list (e.g., priority_queue)? - Jason Lowe-Power On Aug. 11, 2016, 9:07 a.m., Curtis Dunham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3599/ > ----------------------------------------------------------- > > (Updated Aug. 11, 2016, 9:07 a.m.) > > > Review request for Default and Matthias Jung. > > > Repository: gem5 > > > Description > ------- > > mem: Sort memory commands and update DRAMPower > > Add local variable to stores commands to be issued. > These commands are in order within a single bank but will be out > of order across banks & ranks. > > A new procedure, flushCmdList, sorts commands across banks / ranks, > and flushes the sorted list, up to curTick() to DRAMPower. > This is currently called in refresh, once all previous commands are > guaranteed to have completed. Could be called in other events like > the powerEvent as well. > > By only flushing commands up to curTick(), will not get out of sync > when flushed at a periodic stats dump (done in subsequent patch). > > Change-Id: I4ac65a52407f64270db1e16a1fb04cfe7f638851 > Reviewed-by: Radhika Jagtap <[email protected]> > > > Diffs > ----- > > src/mem/dram_ctrl.hh e9096175eb38ac39f37c91bfdf2a450b9664e222 > src/mem/dram_ctrl.cc e9096175eb38ac39f37c91bfdf2a450b9664e222 > > Diff: http://reviews.gem5.org/r/3599/diff/ > > > Testing > ------- > > > Thanks, > > Curtis Dunham > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
