mem: Fix memory allocation bug in deferred snoop handling
Review Request #3258 - Created Dec. 14, 2015 and submitted
| Information | |
|---|---|
| Andreas Hansson | |
| gem5 | |
| default | |
| Reviewers | |
| Default | |
Changeset 11271:0b4da061a4e5 --------------------------- mem: Fix memory allocation bug in deferred snoop handling This patch fixes a corner case in the deferred snoop handling, where requests ended up being used by multiple packets with different lifetimes, and inadvertently got deleted while they were still in use.
Andreas, I test this patch with my traces and the results are all good. I like this patch that have same conditions in both Cache::handleSnoop() and MSHR::handleSnoop(). I have no other questions.
Thanks.