----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1931/#review4466 -----------------------------------------------------------
src/mem/cache/tags/fa_lru.cc <http://reviews.gem5.org/r/1931/#comment4185> I would argue this code is broken, but in any case it is unrelated to the current patch :-) - Andreas Hansson On June 24, 2013, 3:51 p.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1931/ > ----------------------------------------------------------- > > (Updated June 24, 2013, 3:51 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9794:a2b0c64a25d6 > --------------------------- > mem: Reorganize cache tags and make them a SimObject > > This patch reorganizes the cache tags to allow more flexibility to > implement new replacement policies. The base tags class is now a > clocked object so that derived classes can use a clock if they need > one. Also having deriving from SimObject allows specialized Tag > classes to be swapped in/out in .py files. > > The cache set is now templatized to allow it to contain customized > cache blocks with additional informaiton. This involved moving code to > the .hh file and removing cacheset.cc. > > The statistics belonging to the cache tags are now including ".tags" > in their name. Hence, the stats need an update to reflect the change > in naming. > > > Diffs > ----- > > src/mem/cache/cache_impl.hh a0ee1b3aec39 > src/mem/cache/cache.hh a0ee1b3aec39 > src/mem/cache/base.cc a0ee1b3aec39 > src/mem/cache/BaseCache.py a0ee1b3aec39 > src/mem/cache/tags/SConscript a0ee1b3aec39 > src/mem/cache/tags/Tags.py PRE-CREATION > src/mem/cache/tags/base.hh a0ee1b3aec39 > src/mem/cache/tags/base.cc a0ee1b3aec39 > src/mem/cache/tags/cacheset.hh a0ee1b3aec39 > src/mem/cache/tags/cacheset.cc a0ee1b3aec39 > src/mem/cache/tags/fa_lru.hh a0ee1b3aec39 > src/mem/cache/tags/fa_lru.cc a0ee1b3aec39 > src/mem/cache/tags/lru.hh a0ee1b3aec39 > src/mem/cache/tags/lru.cc a0ee1b3aec39 > > Diff: http://reviews.gem5.org/r/1931/diff/ > > > Testing > ------- > > After some sedding of the stats (for tag stats include .tags in the name) all > regressions pass > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
