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

(Updated Dec. 29, 2015, 4:43 p.m.)


Review request for Default.


Repository: gem5


Description (updated)
-------

Changeset 11283:cabe32e23f17
---------------------------
mem: Do not allocate space for packet data if not needed

This patch looks at the request and response command to determine if
either actually has any data payload, and if not, we do not allocate
any space for packet data.

The only tricky case is where the command type is changed as part of
the MSHR functionality. In these cases where the original packet had
no data, but the new packet does, we need to explicitly call
allocate().


Diffs (updated)
-----

  src/mem/cache/cache.cc 3fd1142adad9 
  src/mem/cache/mshr.cc 3fd1142adad9 
  src/mem/packet.hh 3fd1142adad9 

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


Testing
-------


Thanks,

Andreas Hansson

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

Reply via email to