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

Add ttcluster apis to support Galaxy #8305

Closed
ubcheema opened this issue May 9, 2024 · 4 comments
Closed

Add ttcluster apis to support Galaxy #8305

ubcheema opened this issue May 9, 2024 · 4 comments
Assignees
Labels
feature-request External feature request Galaxy Galaxy Readiness Tickets for Project Javelin P1_critical

Comments

@ubcheema
Copy link
Contributor

ubcheema commented May 9, 2024

get_mmio_device_tunnel_depth():
tunnel depth = number of remote chips serviced by the tunnel.
If mmio chip is connected to only 1 device in the remote chip set, tunnel depth = number of chips in remote group. Resolves to 1 on N300, T3000 per mmio device.

If mmio chip is connected to > 1 chips in the remote chip set, tunnel depth = number of chips in remote chip set divided by tunnel count.

get_mmio_device_tunnel_count(): tunnel count = number of unique remote_device_ids connected to mmio eth ports.
If mmio decice has multiple ports to same remote chip, tunnel count = 1. e.g Should resolve to 1 on N300, T3000
If mmio chip has eth ports connected to multiple unique remote devices, tunnel count = num eth ports connected to unique remote devices. Should resolve to 2 for each gateway WH on TG.

@ubcheema ubcheema added the feature-request External feature request label May 9, 2024
@ubcheema ubcheema added P1_critical Galaxy Galaxy Readiness Tickets for Project Javelin labels May 9, 2024
aliuTT added a commit that referenced this issue May 12, 2024
aliuTT added a commit that referenced this issue May 12, 2024
aliuTT added a commit that referenced this issue May 12, 2024
aliuTT added a commit that referenced this issue May 12, 2024
aliuTT added a commit that referenced this issue May 13, 2024
@aliuTT
Copy link
Contributor

aliuTT commented May 13, 2024

APIs checked into galaxy/main

@aliuTT aliuTT closed this as completed May 13, 2024
aliuTT added a commit that referenced this issue May 14, 2024
aliuTT added a commit that referenced this issue May 14, 2024
@ubcheema ubcheema reopened this May 14, 2024
@ubcheema
Copy link
Contributor Author

These commits on galaxy/main cause segmentation fault at end of test:
I did not check which of the following two is the culprit. But removing these two commits makes the tests pass cleanly.
With the commits, test passes, but there is a segmentation fault.

commit a5f72e20c1cfa4e5eafff8fae970619245b60a99 (HEAD -> galaxy/main, origin/galaxy/main)
Author: Allan Liu <aliu@tenstorrent.com>
Date:   Mon May 13 14:49:20 2024 +0000

    #8305: cleanup, add print

commit 13d134f0eaa25074f38421fbf8ddabb6ae247a18
Author: Allan Liu <aliu@tenstorrent.com>
Date:   Sun May 12 19:31:23 2024 +0000

    #8305: add Galaxy cluster apis

@ubcheema
Copy link
Contributor Author

Doing a clean build gets rid of segmentation fault.
It was probably due to some stale build dependency.

@aliuTT
Copy link
Contributor

aliuTT commented May 15, 2024

Yea, this was the seg fault I mentioned to you on slack :( Incremental build doesn't pick up the board types I added to the cluster descriptor object used in tt silicon driver

aliuTT added a commit that referenced this issue May 15, 2024
aliuTT added a commit that referenced this issue May 15, 2024
ubcheema pushed a commit that referenced this issue May 15, 2024
ubcheema pushed a commit that referenced this issue May 15, 2024
ubcheema pushed a commit that referenced this issue May 17, 2024
ubcheema pushed a commit that referenced this issue May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request External feature request Galaxy Galaxy Readiness Tickets for Project Javelin P1_critical
Projects
None yet
Development

No branches or pull requests

2 participants