-
Notifications
You must be signed in to change notification settings - Fork 0
New mem ctrlr classes #19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
mbabaie
wants to merge
15
commits into
develop
Choose a base branch
from
new_memCtrlr_classes
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
dcb49ef
mem: building anew branch on top of develop
mbabaie c3f9c3e
mem: first version of 1st class (either DRAM or NVM)
mbabaie 019663e
mem: first class implementation done
mbabaie 3801551
mem: first class implementation done
mbabaie a01a020
mem: first version of second class (2 interfaces)
mbabaie 4e307ba
mem: splited mem ctrls and mem intrfcs in separate files
mbabaie b2698df
mem: applied almost all of the points discussed for base ctrlr
mbabaie 00b1934
mem: more updates on the structures and names and reusing
mbabaie 9ccd369
mem: cleaned the commented lines
mbabaie 80e18e9
mem: cleaned the commented lines
mbabaie 7ee12bb
mem: lates version
mbabaie 8f02f6b
mem: temporarily added trafGen script
mbabaie 1bffb46
Delete traffGen.py
mbabaie 3259752
mem: latest version with primary testing: identical results
mbabaie 5750029
mem: latest version with primary testing: identical results
mbabaie File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,90 @@ | ||
| # Copyright (c) 2012-2020 ARM Limited | ||
| # All rights reserved. | ||
| # | ||
| # The license below extends only to copyright in the software and shall | ||
| # not be construed as granting a license to any other intellectual | ||
| # property including but not limited to intellectual property relating | ||
| # to a hardware implementation of the functionality of the software | ||
| # licensed hereunder. You may use the software subject to the license | ||
| # terms below provided that you ensure that this notice is replicated | ||
| # unmodified and in its entirety in all distributions of the software, | ||
| # modified or unmodified, in source code or in binary form. | ||
| # | ||
| # Copyright (c) 2013 Amin Farmahini-Farahani | ||
| # Copyright (c) 2015 University of Kaiserslautern | ||
| # Copyright (c) 2015 The University of Bologna | ||
| # All rights reserved. | ||
| # | ||
| # Redistribution and use in source and binary forms, with or without | ||
| # modification, are permitted provided that the following conditions are | ||
| # met: redistributions of source code must retain the above copyright | ||
| # notice, this list of conditions and the following disclaimer; | ||
| # redistributions in binary form must reproduce the above copyright | ||
| # notice, this list of conditions and the following disclaimer in the | ||
| # documentation and/or other materials provided with the distribution; | ||
| # neither the name of the copyright holders nor the names of its | ||
| # contributors may be used to endorse or promote products derived from | ||
| # this software without specific prior written permission. | ||
| # | ||
| # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS | ||
| # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT | ||
| # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR | ||
| # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT | ||
| # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, | ||
| # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT | ||
| # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, | ||
| # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY | ||
| # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | ||
| # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | ||
| # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | ||
|
|
||
| from m5.params import * | ||
| from m5.proxy import * | ||
| from m5.objects.QoSMemCtrl import * | ||
|
|
||
| # Enum for memory scheduling algorithms, currently First-Come | ||
| # First-Served and a First-Row Hit then First-Come First-Served | ||
| class MemSched(Enum): vals = ['fcfs', 'frfcfs'] | ||
|
|
||
| # SimpleMemCtrl is a single-channel single-ported Memory controller model | ||
| # that aims to model the most important system-level performance | ||
| # effects of a memory controller, interfacing with media specific | ||
| # interfaces | ||
| class SimpleMemCtrl(QoSMemCtrl): | ||
| type = 'SimpleMemCtrl' | ||
| cxx_header = "mem/simple_mem_ctrl.hh" | ||
| cxx_class = 'gem5::memory::SimpleMemCtrl' | ||
|
|
||
| # single-ported on the system interface side, instantiate with a | ||
| # bus in front of the controller for multiple ports | ||
| port = ResponsePort("This port responds to memory requests") | ||
|
|
||
| # Interface to memory media | ||
| dram = Param.MemInterface(NULL, "Memory interface") | ||
|
|
||
| # read and write buffer depths are set in the interface | ||
| # the controller will read these values when instantiated | ||
|
|
||
| # threshold in percent for when to forcefully trigger writes and | ||
| # start emptying the write buffer | ||
| write_high_thresh_perc = Param.Percent(85, "Threshold to force writes") | ||
|
|
||
| # threshold in percentage for when to start writes if the read | ||
| # queue is empty | ||
| write_low_thresh_perc = Param.Percent(50, "Threshold to start writes") | ||
|
|
||
| # minimum write bursts to schedule before switching back to reads | ||
| min_writes_per_switch = Param.Unsigned(16, "Minimum write bursts before " | ||
| "switching to reads") | ||
|
|
||
| # scheduler, address map and page policy | ||
| mem_sched_policy = Param.MemSched('frfcfs', "Memory scheduling policy") | ||
|
|
||
| # pipeline latency of the controller and PHY, split into a | ||
| # frontend part and a backend part, with reads and writes serviced | ||
| # by the queues only seeing the frontend contribution, and reads | ||
| # serviced by the memory seeing the sum of the two | ||
| static_frontend_latency = Param.Latency("10ns", "Static frontend latency") | ||
| static_backend_latency = Param.Latency("10ns", "Static backend latency") | ||
|
|
||
| command_window = Param.Latency("10ns", "Static backend latency") |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs a comment which says it has two interfaces and a bit more detail.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in #23