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

RoCC Interface and Basic Example Implementation #2183

Open
wants to merge 69 commits into
base: devel
Choose a base branch
from
Open

RoCC Interface and Basic Example Implementation #2183

wants to merge 69 commits into from

Conversation

ATWeatherly
Copy link

This is the RoCC Interface for attaching arbitrary accelerators to a RISC-V host and one implementation of a basic accelerator using this RoCC interface.

1 - Targeting origin devel

2 - Merged with origin devel right before this request

3 - All original tests still pass locally

atweath and others added 21 commits July 25, 2023 13:42
Basic RoCC Interface Example
2c11a0f
@ATWeatherly
Update basic_rocc_vanadis.py
61cd5f3
Restore LSQ and Change Test File
fd1c3f4
Merge branch 'vanadis-rocc' of https://github.com/ATWeatherly/sst-ele…
d31e036 
…ments into vanadis-rocc
@ATWeatherly
Update vanadis.h
cb35f7d 
removed analog include
@ATWeatherly
Delete vroccanalog.h
4ca7353
@ATWeatherly
Update Makefile.am
836b54b 
removed analog from makefile
@ATWeatherly
Update vbasicrocc.h
57e9cc0 
removed std::out print statements
@ATWeatherly
Update vbasiclsq.h
72945d1 
removed std::out print statements
Print Statement Cleanup
31397e2 
removed a few std::cout's
Removed analog naming
7e43946 
also made RoCC Subcomponent loading optional
Update vbasicrocc.h
d69de7d 
make RoCC Response functional rather than meta-data
Restoring Original Binaries
33a82d3
Merge branch 'vanadis-rocc' of https://github.com/ATWeatherly/sst-ele…
906e1fa 
…ments into vanadis-rocc
Cleanup & New SST Gold for RoCC Test
0bb21ec
Update .gitignore
3bf9d10
@ATWeatherly
Merge pull request #1 from ATWeatherly/vanadis-rocc
2824bbe 
Vanadis Basic RoCC Accelerator Interface
Merge remote-tracking branch 'origin/devel' into vanadis-rocc
eb630b1
made RoCC not required
fb0a857 
it would seg fault if you didn't load a rocc subcomponent
Fixed hello-world-cpp Binary
fa73d9e
@ATWeatherly
Merge pull request #2 from ATWeatherly/vanadis-rocc
53af2d0 
Vanadis rocc
@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

@hughes-c hughes-c self-assigned this Jul 30, 2023
@hughes-c hughes-c self-requested a review July 30, 2023 20:39
@hughes-c hughes-c added this to the SST V13.1.0 milestone Jul 30, 2023
@sst-autotester
Merge pull request sstsimulator#2204 from sstsimulator/devel
1e8894c 
Automatically Merged using SST Master Branch Merger
@sst-autotester
Merge pull request sstsimulator#2206 from sstsimulator/devel
72f5867 
Automatically Merged using SST Master Branch Merger
@sst-autotester
Merge pull request sstsimulator#2212 from sstsimulator/devel
c83f878 
Automatically Merged using SST Master Branch Merger
@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

@ATWeatherly
Copy link
Author

ATWeatherly commented Apr 12, 2024 •
edited
Loading

Here is a model of a fully functional MVM accelerator using this RoCC interface:
https://github.com/PlatinumCD/sst-elements/tree/basic_rocc/src/sst/elements/golem

See vroccanalog.h for interface usage

@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

@ATWeatherly
Copy link
Author

Interface can now be used to attach up to 4 accelerators that adhere to the RoCC specification. Python test driver updated to show how to attach multiple accelerators

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hughes-c hughes-c Awaiting requested review from hughes-c

Assignees

@hughes-c hughes-c

Labels
Enhancement SST-vanadis
Projects
None yet
Milestone
Future
Development

Successfully merging this pull request may close these issues.
3 participants
@ATWeatherly @sst-autotester @hughes-c