fastmodel: Let the EVS set an attribute for getSendFunctional to return.

The iris CPU model doesn't necessarily know the best way to send
functional packets (what port? what type is that port?), but only has
a generic sc_module pointer to the EVS and so can't call specialized
methods on it. There also isn't any common base class for EVSes to cast
into in a generic way.

This attribute mechanism lets the EVS set up its own sendFunctional
implementation however it needs to using facilities that are built
into generic sc_objects.

Change-Id: I69bf364908c2a5360bd6ce7d3e49ce67c6f771b0
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/21046
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Maintainer: Giacomo Travaglini <giacomo.travaglini@arm.com>

diff --git a/src/arch/arm/fastmodel/iris/cpu.cc b/src/arch/arm/fastmodel/iris/cpu.cc
index 234a1ca0da17af1d82daf9addf241c1f727f8d94..246fe0cf0ea65d314095540c0c5de851d464856b 100644 (file)
--- a/src/arch/arm/fastmodel/iris/cpu.cc
+++ b/src/arch/arm/fastmodel/iris/cpu.cc
@@ -61,6 +61,14 @@ BaseCPU::BaseCPU(BaseCPUParams *params, sc_core::sc_module *_evs) :
     panic_if(base && !periodAttribute,
             "The EVS clock period attribute is not of type "
             "sc_attribute<Tick>.");
+
+    base = evs->get_attribute(SendFunctionalAttributeName);
+    sendFunctional =
+        dynamic_cast<sc_core::sc_attribute<PortProxy::SendFunctionalFunc> *>(
+                base);
+    panic_if(base && !sendFunctional,
+            "The EVS send functional attribute is not of type "
+            "sc_attribute<PortProxy::SendFunctionalFunc>.");
 }
 
 BaseCPU::~BaseCPU()

diff --git a/src/arch/arm/fastmodel/iris/cpu.hh b/src/arch/arm/fastmodel/iris/cpu.hh
index c6c75a2fffdb61301aad62b6673b17216da791da..f7be5cb761b9d346739e88f6c08ace5d432f323b 100644 (file)
--- a/src/arch/arm/fastmodel/iris/cpu.hh
+++ b/src/arch/arm/fastmodel/iris/cpu.hh
@@ -49,6 +49,9 @@ static const std::string PeriodAttributeName = "gem5_clock_period_attribute";
 // The name of the attribute the subsystem should create which will be set to
 // a pointer to its corresponding gem5 CPU.
 static const std::string Gem5CpuAttributeName = "gem5_cpu";
+// The name of the attribute the subsystem should create to hold the
+// sendFunctional delegate for port proxies.
+static const std::string SendFunctionalAttributeName = "gem5_send_functional";
 
 // This CPU class adds some mechanisms which help attach the gem5 and fast
 // model CPUs to each other. It acts as a base class for the gem5 CPU, and
@@ -83,12 +86,21 @@ class BaseCPU : public ::BaseCPU
     Counter totalInsts() const override;
     Counter totalOps() const override { return totalInsts(); }
 
+    PortProxy::SendFunctionalFunc
+    getSendFunctional() override
+    {
+        if (sendFunctional)
+            return sendFunctional->value;
+        return ::BaseCPU::getSendFunctional();
+    }
+
   protected:
     sc_core::sc_module *evs;
 
   private:
     sc_core::sc_event *clockEvent;
     sc_core::sc_attribute<Tick> *periodAttribute;
+    sc_core::sc_attribute<PortProxy::SendFunctionalFunc> *sendFunctional;
 
   protected:
     void