> On Jan. 30, 2015, 9:47 p.m., Nilay Vaish wrote: > > src/cpu/testers/memtest/MemTest.py, line 55 > > <http://reviews.gem5.org/r/2611/diff/1/?file=43345#file43345line55> > > > > Are you sure this should be dropped? I think the coherence protocols > > that provide a dma controller need this for testing. > > Andreas Hansson wrote: > All regressions work just fine (with stats updates). > > I am about to post a separate test script that actually shares data (not > just false sharing), doing so based on the TrafficGen and MemChecker.
You're ok with this Nilay? I'll post the true-sharing checker later this week. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2611/#review5809 ----------------------------------------------------------- On Jan. 21, 2015, 1:23 p.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2611/ > ----------------------------------------------------------- > > (Updated Jan. 21, 2015, 1:23 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10671:94bc71e83168 > --------------------------- > cpu: Tidy up the MemTest and make false sharing more obvious > > The MemTest class really only tests false sharing, and as such there > was a lot of old cruft that could be removed. This patch cleans up the > tester, and also makes it more clear what the assumptions are. As part > of this simplification the reference functional memory is also > removed. > > The regression configs using MemTest are updated to reflect the > changes, and the stats will be bumped in a separate patch. The example > config will be updated in a separate patch due to more extensive > re-work. > > In a follow-on patch a new tester will be introduced that uses the > MemChecker to implement true sharing. > > > Diffs > ----- > > configs/example/memtest.py a6fe75e8296b > src/cpu/testers/memtest/MemTest.py a6fe75e8296b > src/cpu/testers/memtest/memtest.hh a6fe75e8296b > src/cpu/testers/memtest/memtest.cc a6fe75e8296b > tests/configs/memtest-filter.py a6fe75e8296b > tests/configs/memtest-ruby.py a6fe75e8296b > tests/configs/memtest.py a6fe75e8296b > > Diff: http://reviews.gem5.org/r/2611/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev