Yes, that is true. However, I have seen some problems after enabling the bank model on the instruction-L1 cache. So, for debugging purpose, I separate them at this point.
Thank you! Best, Xiangyu From: Ali Saidi [mailto:[email protected]] On Behalf Of Ali Saidi Sent: Monday, April 15, 2013 6:03 PM To: Ali Saidi; Default; Xiangyu Dong Subject: Re: Review Request: mem: add a knob to turn on/off bank blocking model This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1822/ Thanks for this too. Is there a a use case for the model being turned on/off on banks separately? It seems like one option might do. <http://reviews.gem5.org/r/1822/diff/1/?file=35048#file35048line95> configs/common/CacheConfig.py (Diff revision 1) def config_cache(options, system): 95 enable_bank_model=options.l1_enable_bank, maybe just one option for either all on or all off? - Ali On April 15th, 2013, 11:55 a.m., Xiangyu Dong wrote: Review request for Default. By Xiangyu Dong. Updated April 15, 2013, 11:55 a.m. Description Changeset 9638:187a47b3551d --------------------------- 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 (6d4158ff7b82) * configs/common/Options.py (6d4158ff7b82) * src/mem/cache/BaseCache.py (6d4158ff7b82) * src/mem/cache/base.hh (6d4158ff7b82) * src/mem/cache/base.cc (6d4158ff7b82) * src/mem/cache/cache_impl.hh (6d4158ff7b82) View Diff <http://reviews.gem5.org/r/1822/diff/> _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
