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

#0: Upgrade WH and T3000 WH KMD and FW versions to v1.27.1 and v80.10.0.0 respectively #7999

Merged
merged 2 commits into from
Jul 19, 2024

Conversation

tt-rkim
Copy link
Collaborator

@tt-rkim tt-rkim commented May 1, 2024

@tt-rkim tt-rkim assigned tapspatel and unassigned tapspatel May 1, 2024
@tt-rkim tt-rkim requested a review from tapspatel May 1, 2024 12:21
INSTALLING.md Outdated
@@ -15,8 +15,8 @@ Note the current compatability matrix:
| Device | OS | Python | Driver (TT-KMD) | Firmware (TT-Flash) | TT-SMI | TT-Topology |
|---------------------|-----------------|----------|--------------------|--------------------------------------------|-----------------------|--------------------------------|
| Grayskull | Ubuntu 20.04 | 3.8.10 | v1.26 | fw_pack-80.4.0.0_acec1267.tar.gz (v4.0.0) | v2.1.0 or above | N/A |
Copy link
Contributor

Choose a reason for hiding this comment

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

@tt-rkim as we're here, can we update Grayskull Firmware to fw_pack-80.8.0.0.fwbundle?

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe v1.27.1 should be the right version for Grayskull too (that's referenced in the main TT-KMD repo) so we should update that too

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We actually don't have 1.27.1 on our Grayskull machines for CI yet, still 1.26. So there's a bit of a mismatch. We hope to upgrade soon.

Despite this, do you want to tell users this for GS anyway?

Copy link
Member

Choose a reason for hiding this comment

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

The differences between 1.26 and 1.27.1 are almost entirely in the hwmon section of code. From a metalium perspective the two should be identical, so it's up to you on if you advocate for the upgrade or not. Syseng's take is we would recommend upgrading just on general principal, but this isn't a critical update.

INSTALLING.md Outdated
| Wormhole | Ubuntu 20.04 | 3.8.10 | v1.26 | fw_pack-80.8.11.0.tar.gz (v80.8.11.0) | v2.1.0 or above | N/A |
| T3000 (Wormhole) | Ubuntu 20.04 | 3.8.10 | v1.26 | fw_pack-80.8.11.0.tar.gz (v80.8.11.0) | v2.1.0 or above | v1.0.2 or above, `mesh` config |
| Wormhole | Ubuntu 20.04 | 3.8.10 | v1.27.1 | fw_pack-80.8.12.0.tar.gz (v80.8.12.0) | v2.1.0 or above | N/A |
| T3000 (Wormhole) | Ubuntu 20.04 | 3.8.10 | v1.27.1 | fw_pack-80.8.12.0.tar.gz (v80.8.12.0) | v2.1.0 or above | v1.0.2 or above, `mesh` config |
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we provide the fw as a link fw_pack-80.8.12.0.fwbundle? Just because it's in the patches folder so some users might miss that

Copy link
Contributor

@adifrancescoTT adifrancescoTT left a comment

Choose a reason for hiding this comment

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

Left a couple of comments, it would be great to update the grayskull fw ref too. I believe @warthog9 is in the loop on this but tagging him for visibility just in case. Thanks @tt-rkim !

@tt-rkim tt-rkim changed the title #0: Upgrade WH and T3000 WH KMD and FW versions to v1.27.1 and v80.8.12.0 respectively #0: Upgrade WH and T3000 WH KMD and FW versions to v1.27.1 and v80.10.0.0 respectively Jul 17, 2024
@tt-rkim tt-rkim merged commit 501ebfb into main Jul 19, 2024
5 checks passed
@tt-rkim tt-rkim deleted the rkim/0-upgrade-wh-127-812 branch July 19, 2024 15:26
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.

4 participants