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

Allow more host memory channels for Blackhole #93

Closed
pjanevskiTT opened this issue Oct 1, 2024 · 2 comments
Closed

Allow more host memory channels for Blackhole #93

pjanevskiTT opened this issue Oct 1, 2024 · 2 comments
Labels
Blackhole Blackhole specific issue
Milestone
Blackhole support...

Comments

@pjanevskiTT
Copy link
Contributor

We should allow more than one host memory channel for Blackhole. Since we are considering moving to IOMMU (#49), this issue should be low prirority, unless tt-metal really needs it for Blackhole

For reference, we are forcing single host memory channel here https://github.com/tenstorrent/tt-umd/blob/main/device/tt_silicon_driver.cpp#L519
@pjanevskiTT pjanevskiTT added the Blackhole Blackhole specific issue label Oct 1, 2024
@abhullar-tt
Copy link
Contributor

Metal uses multiple host memory channels to support fast-dispatch on multiple chips

@pjanevskiTT pjanevskiTT added P1 Major issues with workarounds available, fix soon P0 Critical issues, needs immediate attention and removed P1 Major issues with workarounds available, fix soon labels Nov 5, 2024
@broskoTT broskoTT removed the P0 Critical issues, needs immediate attention label Nov 26, 2024
@broskoTT broskoTT added this to the Blackhole support in UMD milestone Dec 4, 2024
@broskoTT
Copy link
Contributor

Fixed by #393

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Blackhole Blackhole specific issue
Projects
None yet
Milestone
Blackhole support in UMD
Development

No branches or pull requests
3 participants
@abhullar-tt @broskoTT @pjanevskiTT