From 762153a421407def012eb9620cef695faa3f2df8 Mon Sep 17 00:00:00 2001 From: Pouya Fotouhi Date: Tue, 11 Aug 2020 23:31:29 -0700 Subject: [PATCH] mem-ruby: Fix debug prints for regular Stores In the updated implementation of LL/SC (27103) the default value of success was changed, which results in printing "SC_Failed" for any regular stores. Change-Id: I4f2e0b26233ce0cbdf948aadd19c9d81bf18bec0 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/32514 Reviewed-by: Jason Lowe-Power Reviewed-by: Giacomo Travaglini Maintainer: Jason Lowe-Power Tested-by: kokoro --- src/mem/ruby/system/Sequencer.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mem/ruby/system/Sequencer.cc b/src/mem/ruby/system/Sequencer.cc index aa134f4fa..42398e23e 100644 --- a/src/mem/ruby/system/Sequencer.cc +++ b/src/mem/ruby/system/Sequencer.cc @@ -368,7 +368,7 @@ Sequencer::writeCallback(Addr address, DataBlock& data, (seq_req.m_type != RubyRequestType_Load_Linked) && (seq_req.m_type != RubyRequestType_IFETCH)) { // LL/SC support (tested with ARMv8) - bool success = false; + bool success = true; if (seq_req.m_type != RubyRequestType_Store_Conditional) { // Regular stores to addresses being monitored -- 2.30.2