---
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/624/
---
Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and Nathan
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/624/#review1065
---
Ship it!
I hope you have tested the existing protocols with these
On 2011-03-31 14:22:16, Nilay Vaish wrote:
I hope you have tested the existing protocols with these changes.
Yes - MOESI_[CMP_[directory|token]|hammer] all compile and run -l 1000 -n 4 on
the Ruby Tester. Since no logic has changed (for all my Ruby changes), I
believe it's sufficient
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/624/#review1069
---
src/mem/slicc/symbols/StateMachine.py
On 2011-03-31 14:22:16, Nilay Vaish wrote:
I hope you have tested the existing protocols with these changes.
Lisa Hsu wrote:
Yes - MOESI_[CMP_[directory|token]|hammer] all compile and run -l 1000 -n
4 on the Ruby Tester. Since no logic has changed (for all my Ruby changes),
I
On 2011-03-31 15:58:14, Brad Beckmann wrote:
src/mem/slicc/symbols/StateMachine.py, line 477
http://reviews.m5sim.org/r/624/diff/1/?file=11400#file11400line477
Do you also need to declare the contains_dma_sequencer flag here and
set it to False?
Brad, great catch - since