> On April 27, 2013, 7:25 a.m., Ali Saidi wrote: > > This seems good, but I think the Option name is a little bit confusing. How > > about --l1-bank-model rather than --l1-enable-bank? (and similar for the > > other ones.
This patch is merged to #1809 - Xiangyu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1822/#review4310 ----------------------------------------------------------- On July 18, 2013, 7:35 p.m., Xiangyu Dong wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1822/ > ----------------------------------------------------------- > > (Updated July 18, 2013, 7:35 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9818:9f5292877175 > --------------------------- > mem: add a knob to turn on/off bank blocking model > By default, the cache bank model is turned off to match the main branch > regression result. > Enabling this feature by --l1-bank-model, --l2-bank-model, --l3-bank-model > > > Diffs > ----- > > configs/common/CacheConfig.py 3b3b94536547 > configs/common/Options.py 3b3b94536547 > src/mem/cache/BaseCache.py 3b3b94536547 > src/mem/cache/base.hh 3b3b94536547 > src/mem/cache/base.cc 3b3b94536547 > src/mem/cache/cache_impl.hh 3b3b94536547 > > Diff: http://reviews.gem5.org/r/1822/diff/ > > > Testing > ------- > > > Thanks, > > Xiangyu Dong > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
