> On Nov. 3, 2015, 9:53 p.m., Joel Hestness wrote: > > Can you please provide a clearer patch title and description? To me, it is > > unclear how this is a 'cleaner' solution. I understand that this fixes some > > issues testing multiple sequencers, but I don't understand where the > > retries, and port inst vs. data designation changes fit in all this. > > Brad Beckmann wrote: > Sure. We updated the description to say 'more flexible' rather than > 'cleaner'. I think that is a more accurate summary of this patch. Does the > new description below look good to you? > > ruby: more flexible ruby tester support > > This patch allows the ruby random tester to use ruby ports that may only > support instr or data requests. This patch is similar to a previous > changeset > (8932:1b2c17565ac8) that was unfortunately broken by subsequent > changesets. > This current patch implements the support in a more straight-forward way. > Since retries are now tested when running the ruby random tester, this > patch > splits up the retry and drain check behavior so that RubyPort children, > such > as the GPUCoalescer, can perform those operations correctly without > having to > duplicate code. Finally, the patch also includes better DPRINTFs for > debugging the tester.
Yes, this is clearer than the current review description. Thanks! - Joel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3178/#review7460 ----------------------------------------------------------- On Oct. 30, 2015, 9:49 p.m., Tony Gutierrez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3178/ > ----------------------------------------------------------- > > (Updated Oct. 30, 2015, 9:49 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11184:1160e90d0f03 > --------------------------- > ruby: cleaner ruby tester support > > This patch allows the ruby random tester to use ruby ports that may only > support instr or data requests. This patch is similar to a previous changeset > (8932:1b2c17565ac8) that was unfortunately broken by subsequent changesets. > This current patch implements the support in a more straight-forward way. > The patch also includes better DPRINTFs and generalizes the retry behavior > needed by the ruby tester so that other testers/cpu models can use it as well. > > > Diffs > ----- > > configs/example/ruby_random_test.py > 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > configs/ruby/MESI_Three_Level.py 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > configs/ruby/MESI_Two_Level.py 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > configs/ruby/MI_example.py 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > configs/ruby/MOESI_CMP_directory.py > 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > configs/ruby/MOESI_CMP_token.py 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > configs/ruby/MOESI_hammer.py 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/cpu/testers/rubytest/Check.cc 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/cpu/testers/rubytest/CheckTable.cc > 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/cpu/testers/rubytest/RubyTester.hh > 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/cpu/testers/rubytest/RubyTester.cc > 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/cpu/testers/rubytest/RubyTester.py > 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/mem/ruby/system/RubyPort.hh 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/mem/ruby/system/RubyPort.cc 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/mem/ruby/system/Sequencer.py 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > > Diff: http://reviews.gem5.org/r/3178/diff/ > > > Testing > ------- > > > Thanks, > > Tony Gutierrez > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
