From 643918fe491421823ecfba72500b19cc2b435ffc Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Fri, 31 Aug 2018 17:34:38 -0700 Subject: [PATCH] systemc: Toggle off clumping methods together in the ready list. This is totally legal and good for performance, but because some of Accellera's tests depend on processes which can run in any order running in a particular order to reproduce the golden output, it needs to be disabled to pass the tests. This change leaves it as an option which could even be plumbed out in the future to support some sort of "compatibility" mode with the tests. An alternative would be to verify that the tests pass, change the ordering to the alternative (but still correct) order, and then update the reference output. Change-Id: I113a40dec52f8b623253f8a27886b4a0abe89485 Reviewed-on: https://gem5-review.googlesource.com/c/12457 Reviewed-by: Gabe Black Maintainer: Gabe Black --- src/systemc/core/scheduler.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/systemc/core/scheduler.cc b/src/systemc/core/scheduler.cc index d5081c667..eda4ed7e4 100644 --- a/src/systemc/core/scheduler.cc +++ b/src/systemc/core/scheduler.cc @@ -190,7 +190,9 @@ void Scheduler::ready(Process *p) { // Clump methods together to minimize context switching. - if (p->procKind() == ::sc_core::SC_METHOD_PROC_) + static bool cluster_methods = false; + + if (cluster_methods && p->procKind() == ::sc_core::SC_METHOD_PROC_) readyList.pushFirst(p); else readyList.pushLast(p); -- 2.30.2