----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1962/#review4584 -----------------------------------------------------------
Ship it! Ship It! - Amin Farmahini On Aug. 7, 2013, 5:21 p.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1962/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2013, 5:21 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9831:b5aed07974d4 > --------------------------- > mem: Perform write merging in the DRAM write queue > > This patch implements basic write merging in the DRAM to avoid > redundant bursts. When a new access is added to the queue it is > compared against the existing entries, and if it is either > intersecting or immediately succeeding/preceeding an existing item it > is merged. > > There is currently no attempt made at avoiding iterating over the > existing items in determining whether merging is possible or not. > > > Diffs > ----- > > src/mem/simple_dram.hh 2492d7ccda7e > src/mem/simple_dram.cc 2492d7ccda7e > > Diff: http://reviews.gem5.org/r/1962/diff/ > > > Testing > ------- > > With the Alpha quiesce patch and stats updates all regressions pass. > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
