-
Notifications
You must be signed in to change notification settings - Fork 87
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
TT-Metal VCS Simulator #11101
Merged
Merged
TT-Metal VCS Simulator #11101
Conversation
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
vtangTT
requested review from
ttmchiou,
TT-billteng,
tt-rkim,
abhullar-tt,
pgkeller,
aliuTT,
tt-aho,
tt-dma,
tt-asaigal,
ubcheema,
davorchap and
cfjchu
as code owners
August 5, 2024 22:23
vtangTT
force-pushed
the
vtangTT/vcs_simulator
branch
from
August 5, 2024 22:25
70ca92b
to
b8748af
Compare
vtangTT
force-pushed
the
vtangTT/vcs_simulator
branch
from
August 5, 2024 23:10
b8748af
to
99bec9d
Compare
TT-billteng
reviewed
Aug 6, 2024
TT-billteng
reviewed
Aug 6, 2024
TT-billteng
reviewed
Aug 6, 2024
TT-billteng
reviewed
Aug 6, 2024
vtangTT
force-pushed
the
vtangTT/vcs_simulator
branch
from
August 6, 2024 22:01
99bec9d
to
e5978bd
Compare
TT-billteng
approved these changes
Aug 7, 2024
abhullar-tt
approved these changes
Aug 8, 2024
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.
Working simulator 🚀
- bump umd to pick up simulator infra changes - soc + core descriptors for VCS simulator - CPM package for Google flatbuffer, NNG and libuv - apptainer .def for `aus` machines - TT_METAL_SIMULATOR_EN env var to target VCS simulator - remove everything versim related
vtangTT
force-pushed
the
vtangTT/vcs_simulator
branch
from
August 8, 2024 19:35
fac740c
to
eb6bb0b
Compare
tt-rkim
approved these changes
Aug 8, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Ticket
#11099
UMD PR: tenstorrent/tt-umd#25
Problem description
Connect tt-metal to VCS simulator. Will allow us to start development on future chips (i.e. quasar) without silicon.
What's changed
Instead of building VCS libraries into metal, send the DPI read/write calls over the network using
NNG
+Flatbuffers
to a remote VCS instance (kicked off bylibuv
in a separate process intt_SimulationDevice
. Remote VCS will then send read responses back, completely isolating metal from anything VCS related.Metal changes include:
NNG
,Flatbuffers
for networking andlibuv
for process management (all used at UMD level)Checklist