----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2917/#review6661 -----------------------------------------------------------
Going once...going twice... - Andreas Hansson On June 25, 2015, 7:30 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2917/ > ----------------------------------------------------------- > > (Updated June 25, 2015, 7:30 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10884:821180645b18 > --------------------------- > mem: Update DRAM command scheduler for bank groups > > This patch updates the command arbitration so that bank group timing > as well as rank-to-rank delays will be taken into account. The > resulting arbitration no longer selects commands (prepped or not) that > cannot issue seamlessly if there are commands that can issue > back-to-back, minimizing the effect of rank-to-rank (tCS) & same bank > group (tCCD_L) delays. > > The arbitration selects a new command based on the following priority. > Within each priority band, the arbitration will use FCFS to select the > appropriate command: > > 1) Bank is prepped and burst can issue seamlessly, without a bubble > > 2) Bank is not prepped, but can prep and issue seamlessly, without a > bubble > > 3) Bank is prepped but burst cannot issue seamlessly. In this case, a > bubble will occur on the bus > > Thus, to enable more parallelism in subsequent selections, an > unprepped packet is given higher priority if the bank prep can be > hidden. If the bank prep cannot be hidden, the selection logic will > choose a prepped packet that cannot issue seamlessly if one exist. > Otherwise, the default selection will choose the packet with the > minimum bank prep delay. > > > Diffs > ----- > > src/mem/dram_ctrl.hh e4f63f1d502d > src/mem/dram_ctrl.cc e4f63f1d502d > > Diff: http://reviews.gem5.org/r/2917/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
