Skip to content

XRT submodule update and npu3 archive addition #844

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
Merged
merged 16 commits into from
Nov 19, 2025
Merged

XRT submodule update and npu3 archive addition #844

merged 16 commits into from
Nov 19, 2025

Conversation

@aktondak
Copy link
Contributor

@aktondak aktondak commented Oct 31, 2025

This PR updates the XRT submodule to include the latest changes for event tracing and firmware logging.
This PR also adds the npu3 specific archive to linux driver stack for running xrt-smi tests and accessing npu3 specific configurations.

maxzhen
maxzhen requested changes Oct 31, 2025
View reviewed changes
Copy link
Collaborator

@maxzhen maxzhen left a comment

Choose a reason for hiding this comment

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

Can we move all vtd links to info.json and download them in build.sh? This way we have all of them in a central location and can enjoy the cache mechanism that is already built-in. They are not changing frequently and don't need to be downloaded every time you go through cmake.

@aktondak
downloading method update
f621fe2 
Signed-off-by: Akshay Tondak <[email protected]>
@aktondak
Copy link
Contributor Author

aktondak commented Oct 31, 2025

Can we move all vtd links to info.json and download them in build.sh? This way we have all of them in a central location and can enjoy the cache mechanism that is already built-in. They are not changing frequently and don't need to be downloaded every time you go through cmake.

That sounds like a good idea. I have updated the PR to download archive binaries exactly similar to how firmware binaries are downloaded and installed. Edits the info.json to capture vtd archive details.
I am not sure on the caching mechanism though. The firmware binary are removed and reinstalled on each build and so is the case with archives now. But the flow is more uniform now

@aktondak aktondak requested a review from maxzhen October 31, 2025 17:30
@aktondak
Copy link
Contributor Author

aktondak commented Nov 5, 2025

retest this please

@aktondak aktondak closed this Nov 5, 2025
@aktondak aktondak reopened this Nov 5, 2025
@aktondak
Copy link
Contributor Author

aktondak commented Nov 5, 2025

retest this please

@aktondak aktondak closed this Nov 6, 2025
@aktondak aktondak reopened this Nov 6, 2025
@aktondak
Copy link
Contributor Author

aktondak commented Nov 6, 2025

retest this please

@aktondak aktondak closed this Nov 7, 2025
@aktondak aktondak reopened this Nov 7, 2025
@aktondak
Copy link
Contributor Author

aktondak commented Nov 7, 2025

retest this please

1 similar comment
@aktondak
Copy link
Contributor Author

aktondak commented Nov 7, 2025

retest this please

@aktondak aktondak closed this Nov 7, 2025
@aktondak aktondak reopened this Nov 7, 2025
@maxzhen
Copy link
Collaborator

maxzhen commented Nov 7, 2025

retest this please

@amd-gevadlam amd-gevadlam reopened this Nov 7, 2025
@amd-gevadlam amd-gevadlam reopened this Nov 7, 2025
@amd-gevadlam amd-gevadlam reopened this Nov 7, 2025
@aktondak
update vtd commit
dd73acb 
Signed-off-by: Akshay Tondak <[email protected]>
@aktondak
Copy link
Contributor Author

aktondak commented Nov 7, 2025

retest this please

@aktondak aktondak mentioned this pull request Nov 11, 2025
Merged
NishadSaraf and others added 7 commits November 13, 2025 08:58
@NishadSaraf @aktondak
Implement queries to get DPT status (amd#857)
7e176f8 
* Implement queries to get DPT status

Implement queries to get DPT status.

Signed-off-by: Nishad Saraf <[email protected]>

* Apply suggestion from @Copilot

Co-authored-by: Copilot <[email protected]>

---------

Signed-off-by: Nishad Saraf <[email protected]>
Co-authored-by: Copilot <[email protected]>
@chvamshi-xilinx @aktondak
fix multi threaded issues in ve2 (amd#855)
7426ffc 
Co-authored-by: ch vamshi krishna <[email protected]>
@ManojTakasi @aktondak
Update the module name and git hash (amd#858)
163b344 
Signed-off-by: Manoj Takasi <[email protected]>
@aktondak
update xrt
004a626 
Signed-off-by: Akshay Tondak <[email protected]>
@aktondak
update
d5987eb 
Signed-off-by: Akshay Tondak <[email protected]>
@aktondak
update xrt
28f7835 
Signed-off-by: Akshay Tondak <[email protected]>
@aktondak
Merge branch 'amd:main' into xrt_update
f4db718
@aktondak
Copy link
Contributor Author

aktondak commented Nov 13, 2025

retest this please

1 similar comment
@amd-gevadlam
Copy link

amd-gevadlam commented Nov 13, 2025

retest this please

@aktondak
update xrt
7b6e515 
Signed-off-by: Akshay Tondak <[email protected]>
@aktondak
update vtd to latest
0645533 
Signed-off-by: Akshay Tondak <[email protected]>
@maxzhen maxzhen merged commit 188c1f3 into amd:main Nov 19, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maxzhen maxzhen maxzhen approved these changes

@NishadSaraf NishadSaraf Awaiting requested review from NishadSaraf

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@aktondak @maxzhen @amd-gevadlam @NishadSaraf @chvamshi-xilinx @ManojTakasi