mem: Rename SimpleDRAM to a more suitable DRAMCtrl

Information
Submitter:	Andreas Hansson
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 10121:3f0e8a243002
---------------------------
mem: Rename SimpleDRAM to a more suitable DRAMCtrl

This patch renames the not-so-simple SimpleDRAM to a more suitable
DRAMCtrl. The name change is intended to ensure that we do not send
the wrong message (although the "simple" in SimpleDRAM was originally
intended as in cleverly simple, or elegant).

As the DRAM controller modelling work is being presented at ISPASS'14
our hope is that a broader audience will use the model in the future.

Testing Done

All regressions pass

Ship It!

I am not opposed to this patch, but I would keep it as SimpleDRAM until we support more fine grained timing parameters such as tCCD and tWR. On the other hand, it is not that of a simple model any more, so it is not a bad idea for marketing purposes.

Andreas Hansson March 12, 2014, 2:34 a.m. (March 12, 2014, 2:34 a.m.)

If you're ok with it, I would propose to go ahead and this point. I agree that tWR and tCCD should be high on the priority list. Any volunteers? :-)

Amin Farmahini March 12, 2014, 3:19 p.m. (March 12, 2014, 3:19 p.m.)

Sounds good to me. Regarding tWR and tCCD, I'll put them in my todo list for summer :).

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Status: Closed (submitted)