mem: Clean up Memory Controller
Make the actual controller more generic
- Rename DRAMCtrl to MemCtrl
- Rename DRAMacket to MemPacket
- Rename dram_ctrl.cc to mem_ctrl.cc
- Rename dram_ctrl.hh to mem_ctrl.hh
- Create MemCtrl debug flag
Move the memory interface classes/functions to separate files
- mem_interface.cc
- mem_interface.hh
Change-Id: I1acba44c855776343e205e7733a7d8bbba92a82c
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/31654
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
committed by
Jason Lowe-Power
parent
dab7c78eca
commit
7a28c82c6e
@@ -113,8 +113,8 @@ MemConfig.config_mem(options, system)
|
||||
|
||||
# the following assumes that we are using the native memory
|
||||
# controller with an NVM interface, check to be sure
|
||||
if not isinstance(system.mem_ctrls[0], m5.objects.DRAMCtrl):
|
||||
fatal("This script assumes the controller is a DRAMCtrl subclass")
|
||||
if not isinstance(system.mem_ctrls[0], m5.objects.MemCtrl):
|
||||
fatal("This script assumes the controller is a MemCtrl subclass")
|
||||
if not isinstance(system.mem_ctrls[0].nvm, m5.objects.NVMInterface):
|
||||
fatal("This script assumes the memory is a NVMInterface class")
|
||||
|
||||
|
||||
@@ -126,8 +126,8 @@ MemConfig.config_mem(options, system)
|
||||
|
||||
# the following assumes that we are using the native controller
|
||||
# with NVM and DRAM interfaces, check to be sure
|
||||
if not isinstance(system.mem_ctrls[0], m5.objects.DRAMCtrl):
|
||||
fatal("This script assumes the controller is a DRAMCtrl subclass")
|
||||
if not isinstance(system.mem_ctrls[0], m5.objects.MemCtrl):
|
||||
fatal("This script assumes the controller is a MemCtrl subclass")
|
||||
if not isinstance(system.mem_ctrls[0].dram, m5.objects.DRAMInterface):
|
||||
fatal("This script assumes the first memory is a DRAMInterface subclass")
|
||||
if not isinstance(system.mem_ctrls[0].nvm, m5.objects.NVMInterface):
|
||||
|
||||
Reference in New Issue
Block a user