diff options
| author | Pouya Fotouhi <pfotouhi@ucdavis.edu> | 2019-01-19 16:38:27 -0800 |
|---|---|---|
| committer | Pouya Fotouhi <pfotouhi@ucdavis.edu> | 2019-02-12 05:51:25 +0000 |
| commit | 15e497d3f11504f957406feee9797cbf6449fffd (patch) | |
| tree | 2b3227687c9438d0f5312d77b5d368cea1bd1a51 | |
| parent | 8d7933293f030f180db5effc01865286ba682ca3 (diff) | |
| download | gem5-15e497d3f11504f957406feee9797cbf6449fffd.tar.xz | |
mem-ruby: Fixing Topology
The constructor assumes the number of nodes (i.e. controllers) equal to
the number of external nodes.
This is a not necessarily valid for all cases (e.g MESI_Three_Level -
where L0s are directly connected to L1s).
MachineType_base_number(MachineType_NUM) provides the total number of
controllers.
Signed-off-by: Pouya Fotouhi <pfotouhi@ucdavis.edu>
Change-Id: Id906099dc967ec70aa34dedb0b55351031ff242c
Reviewed-on: https://gem5-review.googlesource.com/c/15716
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>
| -rw-r--r-- | src/mem/ruby/network/Topology.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/mem/ruby/network/Topology.cc b/src/mem/ruby/network/Topology.cc index f7afffd66..6da251e0b 100644 --- a/src/mem/ruby/network/Topology.cc +++ b/src/mem/ruby/network/Topology.cc @@ -51,7 +51,8 @@ const int INFINITE_LATENCY = 10000; // Yes, this is a big hack Topology::Topology(uint32_t num_routers, const vector<BasicExtLink *> &ext_links, const vector<BasicIntLink *> &int_links) - : m_nodes(ext_links.size()), m_number_of_switches(num_routers), + : m_nodes(MachineType_base_number(MachineType_NUM)), + m_number_of_switches(num_routers), m_ext_link_vector(ext_links), m_int_link_vector(int_links) { // Total nodes/controllers in network |