----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1486/#review3592 -----------------------------------------------------------
I haven't had a chance to review all the patches you just sent out, but this one definitely caught my eye. Can you describe how the prefetcher ties into the protocol? Do you have an example protocol that you can show us? As you probably know, we've integrated prefetchers into Ruby before, but they end up requiring protocol specific changes. It is fine if this does at as. I'm just curious to know if you thought about an easier way to make prefetching protocol independent yet still accurate. BTW, have you ever wondered why the mandatory queue is called "mandatory"? There use to be an optional queue included in each protocol file. :) - Brad Beckmann On Oct. 24, 2012, 12:38 p.m., Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1486/ > ----------------------------------------------------------- > > (Updated Oct. 24, 2012, 12:38 p.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9312:0c088aab5fdf > --------------------------- > ruby: add a prefetcher > This patch adds a prefetcher for the ruby memory system. The prefetcher > is based on a prefetcher implemented by others (well, I don't know > who wrote the original). The prefetcher does stride-based prefetching, > both unit and non-unit. It obseves the misses in the cache and trains on > these. After the training period is over, the prefetcher starts issuing > prefetch requests to the controller. > > > Diffs > ----- > > src/mem/SConscript f634a34f2f0b > src/mem/ruby/SConscript f634a34f2f0b > src/mem/ruby/slicc_interface/AbstractController.hh f634a34f2f0b > src/mem/ruby/structures/Prefetcher.hh PRE-CREATION > src/mem/ruby/structures/Prefetcher.cc PRE-CREATION > src/mem/ruby/structures/RubyPrefetcher.py PRE-CREATION > src/mem/ruby/structures/SConscript PRE-CREATION > > Diff: http://reviews.gem5.org/r/1486/diff/ > > > Testing > ------- > > > Thanks, > > Nilay Vaish > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
