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

(Updated Sept. 2, 2015, 4:36 p.m.)


Review request for Default.


Changes
-------

For Jason's point: There should be no locations that allocate packets without a 
request. A handful of tests indicate the conditional was unnecessary.


Repository: gem5


Description (updated)
-------

Changeset 11079:16c8f4c7dc34
---------------------------
ruby: Fix memory leak in AbstractController

Recent changes to memory access queuing allocate requests for packets sent to
memory controllers, but did not free the requests. Delete them to avoid leaks.


Diffs (updated)
-----

  src/mem/ruby/slicc_interface/AbstractController.cc 2763a59c73ff 

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


Testing
-------


Thanks,

Joel Hestness

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

Reply via email to