-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/1837/
-----------------------------------------------------------

Review request for Default.


Description
-------

Changeset 9678:0433b042cf1d
---------------------------
mem: Separate the two snoop response cases in the bus

This patch makes the flow control and state updates of the coherent
bus more clear by separating the two cases, i.e. forward as a snoop
response, or turn it into a normal response.

With this change it is also more clear what resources are being
occupied, and that we effectively bypass the busy check for the second
case. As a result of the change in resource usage some stats change.


Diffs
-----

  src/mem/bus.hh c5b24e8ed428 
  src/mem/bus.cc c5b24e8ed428 
  src/mem/coherent_bus.cc c5b24e8ed428 

Diff: http://reviews.gem5.org/r/1837/diff/


Testing
-------

All regressions passing after minor stats updates


Thanks,

Andreas Hansson

_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to