Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Adds external TL scratchpad support #380

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

Adds external TL scratchpad support #380

wants to merge 26 commits into from

Conversation

richardyrh
Copy link
Collaborator

This allows the scratchpad to be instantiated outside of Gemmini, which will be interfaced with using TileLink instead of the SyncMem interface. This does not remove the option of having a fully private scratchpad like before.

Some key changes:

  • Added an instruction to move data out to scratchpad instead of L2
    • Reservation station changed to resolve new hazards
    • TLB skips translation if destination is scratchpad (since it's a separate memory space)
  • Added Tilelink option for external scratchpad (default off)
    • A few scratchpad IOs were changed from valid-only to ready/valid to accommodate this
  • Incorporated Vikram's recoded FP numbers change

richardyrh and others added 26 commits January 5, 2024 04:24
@richardyrh
move scratchpad to outside with TL interface, add mvout_spad instruct…
dffcfd1 
…ion that writes scaled results to spad
@richardyrh
mmio gemmini rocc command
8863f6f
@hansungk
Set MMIO base address to within smem range
b2f9cbc
@richardyrh
32b mmio registers and fix unitialized regs
81aee08
@richardyrh
single rw tilelink node for external scratchpad
abfdff4
@richardyrh
bug fixes for unified mem node
e39a63d
@richardyrh
add if clause for integer normalizations to stop errors under fp configs
3fdd55c
@richardyrh
initialize double buffer selection register
19a12b2
@richardyrh
back-to-back spad access and other mem node fixes
c97cef9
@richardyrh
bump gemmini-rocc-tests
b97b09a
@richardyrh
add spad compute loop fsm and instruction support
5a10e96
@richardyrh
move ext scratchpad sram instantiation outside
a92e231
@richardyrh
transfer size fix
5de43ae
@richardyrh
bump gemmini-rocc-tests
4db2e1f
@richardyrh
data drop fix, isa change for configurable fsm stages
493c45a
@richardyrh
re-enable rocc interface
6acc664
@richardyrh
fsm changes for dma, change mmio reg offset
5bcbead
@richardyrh
Merge remote-tracking branch 'origin/dev' into ext-spad
8a9c423
@richardyrh
move mmio out to tile
38d1020
@richardyrh
add recoded arithmetic from sirius
0328277
@vikramjain236 @richardyrh
incorporate sirius recoding skips in the gemmini mesh
2916cde
@hansungk
Add write queue valid check
dd8b74e
@hansungk
Increase scratchpad write queue size to 32
3106a4c 
temporary workaround of the queue backpressure problem for the flash
attention kernel
@richardyrh
dma no longer forces spad write
40498de
@richardyrh
update loop tracking
041342d
@richardyrh
Merge remote-tracking branch 'origin/master' into ext-spad
b7c63d1
@richardyrh
Copy link
Collaborator Author

Requires ucb-bar/gemmini-rocc-tests#51

This was referenced Feb 6, 2025
Open
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@richardyrh @hansungk @vikramjain236