From: Brad Beckmann <Brad.Beckmann@amd.com>
Date: Fri, 6 Apr 2012 20:47:06 +0000 (-0700)
Subject: MOESI_hammer: fixed bug with single cpu + flushes, then modified the regression teste... 
X-Git-Tag: stable_2012_06_28~140
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=f050ebe3a87b0db01cfe2d6bee4fac083d659665;p=gem5.git

MOESI_hammer: fixed bug with single cpu + flushes, then modified the regression tester to check this functionality
---

diff --git a/src/mem/protocol/MOESI_hammer-dir.sm b/src/mem/protocol/MOESI_hammer-dir.sm
index dfb62e844..24d64349e 100644
--- a/src/mem/protocol/MOESI_hammer-dir.sm
+++ b/src/mem/protocol/MOESI_hammer-dir.sm
@@ -493,7 +493,7 @@ machine(Directory, "AMD Hammer-like protocol")
 
   action(oc_sendBlockAck, "oc", desc="Send block ack to the owner") {
     peek(requestQueue_in, RequestMsg) {
-      if ((probe_filter_enabled || full_bit_dir_enabled) && (in_msg.Requestor == cache_entry.Owner)) {
+        if (((probe_filter_enabled || full_bit_dir_enabled) && (in_msg.Requestor == cache_entry.Owner)) || machineCount(MachineType:L1Cache) == 1) {
         enqueue(forwardNetwork_out, RequestMsg, latency=memory_controller_latency) {
           out_msg.Address := address;
           out_msg.Type := CoherenceRequestType:BLOCK_ACK;
@@ -1033,7 +1033,7 @@ machine(Directory, "AMD Hammer-like protocol")
   }
 
   action(nofc_forwardRequestConditionalOwner, "nofc", desc="Forward request to one or more nodes if the requestor is not the owner") {
-    assert(machineCount(MachineType:L1Cache) > 1);
+   if (machineCount(MachineType:L1Cache) > 1) {
 
      if (probe_filter_enabled || full_bit_dir_enabled) {
        peek(requestQueue_in, RequestMsg) {
@@ -1065,7 +1065,7 @@ machine(Directory, "AMD Hammer-like protocol")
         }
       }
      }
-
+   }
   }
 
   action(f_forwardWriteFromDma, "fw", desc="Forward requests") {
diff --git a/tests/configs/rubytest-ruby.py b/tests/configs/rubytest-ruby.py
index 7bb7b9014..979295cf5 100644
--- a/tests/configs/rubytest-ruby.py
+++ b/tests/configs/rubytest-ruby.py
@@ -65,10 +65,16 @@ options.l1i_assoc=2
 options.l2_assoc=2
 options.l3_assoc=2
 
+# Turn on flush check for the hammer protocol
+check_flush = False
+if buildEnv['PROTOCOL'] == 'MOESI_hammer':
+    check_flush = True
+
 #
 # create the tester and system, including ruby
 #
-tester = RubyTester(checks_to_complete = 100, wakeup_frequency = 10)
+tester = RubyTester(check_flush = check_flush, checks_to_complete = 100,
+                    wakeup_frequency = 10)
 
 system = System(tester = tester, physmem = SimpleMemory())