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.

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

Implement wait for Blackhole DRAM training #542

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

pjanevskiTT
Copy link
Contributor

Issue

#478

Note: the PR is probably going to timeout on Blackhole CI until the fw from the syseng is released, so reviewers can do the review without me checking this in, to be ready when the fw is released.

Description

Implement waiting for DRAM training on Blackhole. Format of the data provided by syseng to sw is described in https://tenstorrent.atlassian.net/browse/SYS-1241. This PR integrates the wait in our code according to the proposed ticket design.

List of the changes

  • Add function to wait for DRAM cores
  • Implement the proposed design from the ticket
  • Add functions to TTDevice that expose DRAM training info

Testing

CI

API Changes

/

@pjanevskiTT pjanevskiTT requested a review from broskoTT February 25, 2025 18:06
@pjanevskiTT pjanevskiTT self-assigned this Feb 25, 2025
@@ -94,4 +95,56 @@ void LocalChip::wait_eth_cores_training(const uint32_t timeout_ms) {
}
}

void LocalChip::wait_dram_cores_training(const uint32_t timeout_ms) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Blackhole CI timed out as expected, don't merge the PR until the CI passes

@pjanevskiTT pjanevskiTT force-pushed the pjanevski/bh_dram_training branch from 0f2df6f to 9bc2211 Compare February 28, 2025 13:17
@pjanevskiTT pjanevskiTT force-pushed the pjanevski/bh_dram_training branch from 47891d5 to e7bc2ba Compare March 3, 2025 10:16
@pjanevskiTT pjanevskiTT force-pushed the pjanevski/bh_dram_training branch from e7bc2ba to b206855 Compare March 3, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants