[gem5-dev] changeset in gem5: ruby: set SenderMachine in messages of MOESI_...

[Lluc Alvarez]

changeset 964b9eaab6b0 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=964b9eaab6b0
description:
        ruby: set SenderMachine in messages of MOESI_CMP_directory
        This patch adds missing initializations of the SenderMachine field of
        out_msg's when thery are created in the L2 cache controller of the
        MOESI_CMP_directory coherence protocol. When an out_msg is created and 
this
        field is left uninitialized, it is set to the default value 
MachineType_NUM.
        This causes a panic in the MachineType_to_string function when gem5 is
        executed with the Ruby debug flag on and it tries to print the message.

        Committed by: Nilay Vaish <ni...@cs.wisc.edu>

diffstat:

 src/mem/protocol/MOESI_CMP_directory-L2cache.sm |  6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diffs (51 lines):

diff -r ebd44da818d5 -r 964b9eaab6b0 
src/mem/protocol/MOESI_CMP_directory-L2cache.sm
--- a/src/mem/protocol/MOESI_CMP_directory-L2cache.sm   Wed Oct 30 10:35:05 
2013 -0500
+++ b/src/mem/protocol/MOESI_CMP_directory-L2cache.sm   Wed Oct 30 10:35:06 
2013 -0500
@@ -747,6 +747,7 @@
       out_msg.Addr := address;
       out_msg.Type := CoherenceResponseType:DATA;
       out_msg.Sender := machineID;
+      out_msg.SenderMachine := MachineType:L2Cache;
       out_msg.Destination.addNetDest(tbe.L1_GetS_IDs);
       out_msg.DataBlk := tbe.DataBlk;
       // out_msg.Dirty := tbe.Dirty;
@@ -811,6 +812,7 @@
         out_msg.Addr := address;
         out_msg.Type := CoherenceResponseType:DATA;
         out_msg.Sender := machineID;
+        out_msg.SenderMachine := MachineType:L2Cache;
         out_msg.Destination.add(in_msg.Requestor);
         out_msg.DataBlk := tbe.DataBlk;
         // out_msg.Dirty := tbe.Dirty;
@@ -830,6 +832,7 @@
       out_msg.Addr := address;
       out_msg.Type := CoherenceResponseType:DATA;
       out_msg.Sender := machineID;
+      out_msg.SenderMachine := MachineType:L2Cache;
       out_msg.Destination.addNetDest(tbe.Fwd_GetS_IDs);
       out_msg.DataBlk := tbe.DataBlk;
       // out_msg.Dirty := tbe.Dirty;
@@ -866,6 +869,7 @@
         out_msg.Addr := address;
         out_msg.Type := CoherenceResponseType:DATA;
         out_msg.Sender := machineID;
+        out_msg.SenderMachine := MachineType:L2Cache;
         out_msg.Destination.add(in_msg.Requestor);
         out_msg.DataBlk := cache_entry.DataBlk;
         // out_msg.Dirty := cache_entry.Dirty;
@@ -925,6 +929,7 @@
         out_msg.Addr := address;
         out_msg.Type := CoherenceResponseType:DATA;
         out_msg.Sender := machineID;
+        out_msg.SenderMachine := MachineType:L2Cache;
         out_msg.Destination.add(in_msg.Requestor);
         out_msg.DataBlk := cache_entry.DataBlk;
         // out_msg.Dirty := cache_entry.Dirty;
@@ -944,6 +949,7 @@
         out_msg.Addr := address;
         out_msg.Type := CoherenceResponseType:DATA_EXCLUSIVE;
         out_msg.Sender := machineID;
+        out_msg.SenderMachine := MachineType:L2Cache;
         out_msg.Destination.add(in_msg.Requestor);
         out_msg.DataBlk := cache_entry.DataBlk;
         out_msg.Dirty := cache_entry.Dirty;
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev