----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2167/#review4928 -----------------------------------------------------------
src/mem/cache/tags/random_repl.cc <http://reviews.gem5.org/r/2167/#comment4580> should this be const? src/mem/cache/tags/random_repl.cc <http://reviews.gem5.org/r/2167/#comment4581> should this not be using the mersenne twister rather? random_mt.random<int>(...) - Andreas Hansson On Feb. 22, 2014, 5:20 p.m., Anthony Gutierrez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2167/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2014, 5:20 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10084:209fc6fadbe5 > --------------------------- > mem: add random replacement cache tags > > this patch implements a new tags class that uses a random replacement policy. > these tags prefer to evict invalid blocks first, if none are available a > replacement candidate is chosen at random. > > this patch reuses the LRU tag code and simply modifies the following methods: > > accessBlock() > findVictim() > insertBlock() > invalidate() > > > Diffs > ----- > > src/mem/cache/base.cc 2beea2a439b43bf95bc040198322d4869b07f4f3 > src/mem/cache/cache.cc 2beea2a439b43bf95bc040198322d4869b07f4f3 > src/mem/cache/tags/SConscript 2beea2a439b43bf95bc040198322d4869b07f4f3 > src/mem/cache/tags/Tags.py 2beea2a439b43bf95bc040198322d4869b07f4f3 > src/mem/cache/tags/random_repl.hh PRE-CREATION > src/mem/cache/tags/random_repl.cc PRE-CREATION > > Diff: http://reviews.gem5.org/r/2167/diff/ > > > Testing > ------- > > > Thanks, > > Anthony Gutierrez > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
