diff options
| author | Sean Wilson <spwilson2@wisc.edu> | 2017-06-12 14:20:01 -0500 |
|---|---|---|
| committer | Sean Wilson <spwilson2@wisc.edu> | 2017-06-20 18:03:21 +0000 |
| commit | 3d46619c34f1fc470fcc65b83fe678a70e89be30 (patch) | |
| tree | d3f5409a6011069bb83b49020b435442a08fa045 /src/mem/dram_ctrl.hh | |
| parent | 6de8267fbf74f6be13e9381fe03d9014c211e1e8 (diff) | |
| download | gem5-3d46619c34f1fc470fcc65b83fe678a70e89be30.tar.xz | |
mem: Move the Rank construction logic to the Rank constructor
This change was made so Rank objects have their name assigned
when they are instantiated. Therefore, they can initialize their
member objects with their name and it is less likely to change during
runtime.
(NOTE: I would recommend hiding the fields which would cause the name to
change behind getters. Since modification of `Rank.rank` during runtime
will cause the `name()` to change.)
Change-Id: Id51c3553b40e489792c57950e18b8ce927e43173
Signed-off-by: Sean Wilson <spwilson2@wisc.edu>
Reviewed-on: https://gem5-review.googlesource.com/3742
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
Diffstat (limited to 'src/mem/dram_ctrl.hh')
| -rw-r--r-- | src/mem/dram_ctrl.hh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mem/dram_ctrl.hh b/src/mem/dram_ctrl.hh index 12cb0e922..1883041cc 100644 --- a/src/mem/dram_ctrl.hh +++ b/src/mem/dram_ctrl.hh @@ -451,7 +451,7 @@ class DRAMCtrl : public AbstractMemory /** List to keep track of activate ticks */ std::deque<Tick> actTicks; - Rank(DRAMCtrl& _memory, const DRAMCtrlParams* _p); + Rank(DRAMCtrl& _memory, const DRAMCtrlParams* _p, int rank); const std::string name() const { |