changeset 5869c83bc8c7 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=5869c83bc8c7
description:
        mem: Ensure deferred snoops are cache-line aligned

        This patch fixes a bug where a deferred snoop ended up being to a
        partial cache line, and not cache-line aligned, all due to how we copy
        the packet.

diffstat:

 src/mem/cache/mshr.cc |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff -r 77b4fd593427 -r 5869c83bc8c7 src/mem/cache/mshr.cc
--- a/src/mem/cache/mshr.cc     Sun Feb 19 05:30:31 2017 -0500
+++ b/src/mem/cache/mshr.cc     Sun Feb 19 05:30:31 2017 -0500
@@ -415,7 +415,7 @@
         // the packet and the request as part of handling the deferred
         // snoop.
         PacketPtr cp_pkt = will_respond ? new Packet(pkt, true, true) :
-            new Packet(new Request(*pkt->req), pkt->cmd);
+            new Packet(new Request(*pkt->req), pkt->cmd, blkSize);
 
         if (will_respond) {
             // we are the ordering point, and will consequently
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to