From f050ebe3a87b0db01cfe2d6bee4fac083d659665 Mon Sep 17 00:00:00 2001 From: Brad Beckmann Date: Fri, 6 Apr 2012 13:47:06 -0700 Subject: MOESI_hammer: fixed bug with single cpu + flushes, then modified the regression tester to check this functionality --- src/mem/protocol/MOESI_hammer-dir.sm | 6 +++--- tests/configs/rubytest-ruby.py | 8 +++++++- 2 files changed, 10 insertions(+), 4 deletions(-) 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()) -- cgit v1.2.3