-
Notifications
You must be signed in to change notification settings - Fork 47
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
[DRAMsys 2.0] Terapool DRAMsys Merge Request #93
Closed
Closed
Changes from all commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
9db3fa3
[TeraPool] Configurations Changes for TeraPool merge into MemPool
yichao-zh 87300d9
[DRAMsys] add dram rtl model
20761d4
[DRAMsys] fix simulation bug
7d7776c
[DRAMsys] setting to add dramsys support
84013e2
[DRAMsys] fix bugs: stack overflow when reading from dramsys
645e364
[Software] Temp change for easy debug
yichao-zh f6ae1d4
[DMA] DMA bug fix and mempool trace bug fix
yichao-zh 429915b
[DRAM] Update DRAM lib with AXI reordering
yichao-zh d0ece15
[DRAM] Format codes
yichao-zh 390eb51
[DRAM] Merge SRAM and DRAM simulation in one RTL file
yichao-zh 4b9b88c
[Software] Update memcpy kernel
yichao-zh edc7de4
[Makefile] Update Makefile control simulation with dram var
yichao-zh 31ada7c
[DRAM] Delete old file
yichao-zh 922ff48
[Bender] Remove the deleted RTL file
yichao-zh b5f0b9d
[RTL] Change the AXI MUX to AXI Xbar to connect the DRAM
yichao-zh 6ffb02b
[DRAM] DRAM update to support interleaved address mapping
yichao-zh 72e25f7
[DRAM] Update DRAM model to support interleaved mode and fix write bugs
yichao-zh 711c095
[Hardware] Support the different interleave mode for DRAM access
yichao-zh 3d277f4
[Config] L2 address and size update
yichao-zh 6840ad8
[Kernel] memcpy kernel update
yichao-zh 7394a88
[DRAM] Non-Ideal PHY latency support
yichao-zh 7c3afcd
[DRAM] Python Script for DRAM Bandwidth Analysis
yichao-zh 8af0b71
[Format] Format the files for CI check
yichao-zh 53ec53e
[Format] Format and put liscenses to files for CI checking
yichao-zh 5e1c600
[Format] Format DRAM python script for CI check
yichao-zh addc7fc
[AXI] Update Auto Spliter Adding, Update Interleave SystemVerilog Wri…
yichao-zh 1a7a2b3
[HBM2E] Update DRAM HBM model to MICRON HBM2E-3600
yichao-zh d8a5278
[Env] Update some configurations, include the fifo size and DRAM conf…
yichao-zh 65efc1f
[Rebase] Rebase the DRAM work on top of main branch
yichao-zh 66cf7ca
[Config] Complete MinPool config for CI checking
yichao-zh 5ab178e
[memcpy] Reduce transfer size for MinPool CI check
yichao-zh 0a3f31a
[memcpy] Reduce transfer size for MinPool CI check
yichao-zh ed07d95
[memcpy] Remove unused dump from kernel
yichao-zh 4455f4b
[FIFO depth] The Fifo depth tune for support 8 outstanding transction…
yichao-zh 2108979
[DRAMsys] Remove the local version of DRAMsys hardware folder, add th…
yichao-zh 516b3e8
[Config] Move the dram related configurations to the config.mk
yichao-zh 1936ee1
[Makefile] Modify Makefile for updating submodule, patching dram conf…
yichao-zh 1511852
[hardware] hardware change for the new version dramsys support
yichao-zh f361a6b
[DRAM] Add the configuration files for HBM2 DRAM simulation, these fi…
yichao-zh 9528572
[tb] Change back the simulation clk period to 2ns, but 1ns will have …
yichao-zh e32c4e5
[Bender] Update bender to the correct RTL name, as DRAMsys updated th…
yichao-zh 0c1ac85
[software] Update memcpy kernel with reasonable transfer size and tur…
yichao-zh ef0c820
[config] Change simulation to SRAM as L2 for CI checking
yichao-zh 989c7dc
[CI test] Fix tb whitespace tailing and change the bender to compile …
yichao-zh 2d19b34
[CHANGELOG and README] Add changelog and readme for DRAM co-simulation
yichao-zh 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 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 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 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 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 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 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 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 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 |
---|---|---|
|
@@ -29,11 +29,15 @@ axi_data_width ?= 256 | |
# Read-only cache line width in AXI interconnect (in bits) | ||
ro_line_width ?= 256 | ||
|
||
# Number of DMA backends in each group | ||
dmas_per_group ?= 1 | ||
|
||
# Radix for hierarchical AXI interconnect | ||
axi_hier_radix ?= 2 | ||
|
||
# Number of AXI masters per group | ||
axi_masters_per_group ?= 1 | ||
|
||
# Number of DMA backends in each group | ||
dmas_per_group ?= 1 # Brust Length = 16 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In MinPool, the burst length will be 8, right? |
||
|
||
# L2 Banks/Channels | ||
l2_size ?= 4194304 # 400000 | ||
l2_banks ?= 4 |
This file contains 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 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
Submodule dram_rtl_sim
added at
15caf3
This file contains 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,15 @@ | ||
{ | ||
"simulation": { | ||
"addressmapping": "am_hbm2e_16Gb_pc_brc.json", | ||
"mcconfig": "mc_hbm2e_fr_fcfs_grp.json", | ||
"memspec": "ms_hbm2e_16Gb_3600.json", | ||
"simconfig": "simconfig_hbm2e.json", | ||
"simulationid": "hbm2e", | ||
"tracesetup": [ | ||
{ | ||
"clkMhz": 1800, | ||
"name": "HBM2E.stl" | ||
} | ||
] | ||
} | ||
} |
This file contains 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,47 @@ | ||
{ | ||
"addressmapping": { | ||
"BYTE_BIT": [ | ||
0, | ||
1, | ||
2 | ||
], | ||
"COLUMN_BIT": [ | ||
3, | ||
4, | ||
8, | ||
9, | ||
10, | ||
11, | ||
12 | ||
], | ||
"PSEUDOCHANNEL_BIT":[ | ||
5 | ||
], | ||
"BANK_BIT": [ | ||
16, | ||
17 | ||
], | ||
"BANKGROUP_BIT":[ | ||
6, | ||
7, | ||
13 | ||
], | ||
"ROW_BIT": [ | ||
14, | ||
15, | ||
18, | ||
19, | ||
20, | ||
21, | ||
22, | ||
23, | ||
24, | ||
25, | ||
26, | ||
27, | ||
28, | ||
29, | ||
30 | ||
] | ||
} | ||
} |
This file contains 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,20 @@ | ||
{ | ||
"mcconfig": { | ||
"PagePolicy": "Open", | ||
"Scheduler": "FrFcfsGrp", | ||
"SchedulerBuffer": "Bankwise", | ||
"RequestBufferSize": 128, | ||
"CmdMux": "Oldest", | ||
"RespQueue": "Fifo", | ||
"RefreshPolicy": "AllBank", | ||
"RefreshMaxPostponed": 8, | ||
"RefreshMaxPulledin": 8, | ||
"PowerDownPolicy": "NoPowerDown", | ||
"Arbiter": "Simple", | ||
"PhyDelayFw": 8, | ||
"PhyDelayBw": 9, | ||
"ThinkDelayFw": 12, | ||
"ThinkDelayBW": 12, | ||
"MaxActiveTransactions": 128 | ||
} | ||
} |
This file contains 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,48 @@ | ||
{ | ||
"memspec": { | ||
"memarchitecturespec": { | ||
"burstLength": 4, | ||
"dataRate": 2, | ||
"nbrOfBankGroups": 8, | ||
"nbrOfBanks": 32, | ||
"nbrOfColumns": 128, | ||
"nbrOfPseudoChannels": 2, | ||
"nbrOfRows": 32768, | ||
"width": 64, | ||
"nbrOfDevices": 1, | ||
"nbrOfChannels": 1 | ||
}, | ||
"memoryId": "Test MemPool-TeraPool with HBM2 upto 3600bps (16Gb, Single Channel)", | ||
"memoryType": "HBM2", | ||
"memtimingspec": { | ||
"CCDL": 4, | ||
"CCDS": 2, | ||
"CKE": 8, | ||
"DQSCK": 2, | ||
"FAW": 9, | ||
"PL": 2, | ||
"RAS": 30, | ||
"RC": 45, | ||
"RCDRD": 16, | ||
"RCDWR": 12, | ||
"REFI": 3900, | ||
"REFISB": 122, | ||
"RFC": 260, | ||
"RFCSB": 200, | ||
"RL": 41, | ||
"RP": 15, | ||
"RRDL": 2.22, | ||
"RRDS": 2.22, | ||
"RREFD": 8, | ||
"RTP": 4, | ||
"RTW": 18, | ||
"WL": 8, | ||
"WR": 41, | ||
"WTRL": 6, | ||
"WTRS": 4, | ||
"XP": 10, | ||
"XS": 270, | ||
"clkMhz": 1800 | ||
} | ||
} | ||
} |
This file contains 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,15 @@ | ||
{ | ||
"simconfig": { | ||
"AddressOffset": 0, | ||
"CheckTLM2Protocol": false, | ||
"DatabaseRecording": true, | ||
"Debug": false, | ||
"EnableWindowing": true, | ||
"PowerAnalysis": false, | ||
"SimulationName": "hbm2e", | ||
"SimulationProgressBar": true, | ||
"StoreMode": "Store", | ||
"UseMalloc": false, | ||
"WindowSize": 300 | ||
} | ||
} |
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.
I checked, and this change should not impact the backend too much, so if it actually helps with the DRAM's performance, we can keep it that way. But it would be interesting to know why we can't get full performance with smaller bursts on the DRAM.