From 9646c0e93b98b9db40de3b898b26eddbc60a2a46 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89der=20F=2E=20Zulian?= Date: Tue, 28 Jun 2016 11:18:06 +0200 Subject: [PATCH] Issue#12 - removal of template parameters from sockets It is possible to remove the "BUSWIDTH" and "TYPES" parameters from the sockets. What happens is that the default values are used. The default BUSWIDTH is 32, then by removing it we are using 32 instead of the old value 128. It seems that this change does not affect functionality. The default TYPES is tlm::tlm_base_protocol_types, so by removing we do not change anything. --- DRAMSys/simulator/src/controller/Controller.h | 4 ++-- DRAMSys/simulator/src/simulation/Arbiter.h | 4 ++-- DRAMSys/simulator/src/simulation/Dram.h | 2 +- DRAMSys/simulator/src/simulation/ReorderBuffer.h | 4 ++-- DRAMSys/simulator/src/simulation/TracePlayer.h | 2 +- 5 files changed, 8 insertions(+), 8 deletions(-) diff --git a/DRAMSys/simulator/src/controller/Controller.h b/DRAMSys/simulator/src/controller/Controller.h index 7fc07d8a..577e3964 100644 --- a/DRAMSys/simulator/src/controller/Controller.h +++ b/DRAMSys/simulator/src/controller/Controller.h @@ -102,8 +102,8 @@ public: virtual void send(const ScheduledCommand& command, tlm_generic_payload& payload) override; virtual void send(Trigger trigger, sc_time time, tlm_generic_payload& payload) override; - tlm_utils::simple_initiator_socket iSocket; - tlm_utils::simple_target_socket tSocket; + tlm_utils::simple_initiator_socket iSocket; + tlm_utils::simple_target_socket tSocket; unsigned int getTotalNumberOfPayloadsInSystem(); private: diff --git a/DRAMSys/simulator/src/simulation/Arbiter.h b/DRAMSys/simulator/src/simulation/Arbiter.h index 5e190451..86016cc9 100644 --- a/DRAMSys/simulator/src/simulation/Arbiter.h +++ b/DRAMSys/simulator/src/simulation/Arbiter.h @@ -57,8 +57,8 @@ using namespace tlm; template struct Arbiter: public sc_module { public: - tlm_utils::multi_passthrough_initiator_socket iSocket; - tlm_utils::multi_passthrough_target_socket tSocket; + tlm_utils::multi_passthrough_initiator_socket iSocket; + tlm_utils::multi_passthrough_target_socket tSocket; SC_CTOR(Arbiter) : payloadEventQueue(this, &Arbiter::peqCallback) { // The arbiter communicates with one or more memory unity through one or more sockets (one or more memory channels). diff --git a/DRAMSys/simulator/src/simulation/Dram.h b/DRAMSys/simulator/src/simulation/Dram.h index 0d7f95a6..7258e090 100644 --- a/DRAMSys/simulator/src/simulation/Dram.h +++ b/DRAMSys/simulator/src/simulation/Dram.h @@ -65,7 +65,7 @@ template struct Dram : sc_module { // TLM Related: - tlm_utils::simple_target_socket tSocket; + tlm_utils::simple_target_socket tSocket; // Power Model related bool powerAnalysis = Configuration::getInstance().PowerAnalysis; diff --git a/DRAMSys/simulator/src/simulation/ReorderBuffer.h b/DRAMSys/simulator/src/simulation/ReorderBuffer.h index 744a4517..7f20577d 100644 --- a/DRAMSys/simulator/src/simulation/ReorderBuffer.h +++ b/DRAMSys/simulator/src/simulation/ReorderBuffer.h @@ -49,8 +49,8 @@ template struct ReorderBuffer: public sc_module { public: - tlm_utils::simple_initiator_socket iSocket; - tlm_utils::simple_target_socket tSocket; + tlm_utils::simple_initiator_socket iSocket; + tlm_utils::simple_target_socket tSocket; SC_CTOR(ReorderBuffer) : payloadEventQueue(this, &ReorderBuffer::peqCallback), responseIsPendingInInitator(false) diff --git a/DRAMSys/simulator/src/simulation/TracePlayer.h b/DRAMSys/simulator/src/simulation/TracePlayer.h index 070b8127..515373a2 100644 --- a/DRAMSys/simulator/src/simulation/TracePlayer.h +++ b/DRAMSys/simulator/src/simulation/TracePlayer.h @@ -59,7 +59,7 @@ template struct TracePlayer: public sc_module { public: - tlm_utils::simple_initiator_socket iSocket; + tlm_utils::simple_initiator_socket iSocket; TracePlayer(TracePlayerListener* listener); virtual void nextPayload() = 0;