Skip to content

[XRT-SMI] Extend watch mode (--watch) to all examine reports#9725

Merged
stsoe merged 2 commits intoXilinx:masterfrom
aktondak:watch_mode_extension
Apr 15, 2026
Merged

[XRT-SMI] Extend watch mode (--watch) to all examine reports#9725
stsoe merged 2 commits intoXilinx:masterfrom
aktondak:watch_mode_extension

Conversation

@aktondak
Copy link
Copy Markdown
Collaborator

Problem solved by the commit

This PR extends the existing watch mode functionality to generic xrt-smi examine reports. The extension supports refreshing the console output and also take in optionally the refresh interval.

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

https://jira.xilinx.com/browse/AIESW-25947

How problem was solved, alternative solutions (if any) and why they were rejected

Solved via extending the watch mode to all xrt-smi reports

Risks (if any) associated the changes in the commit

None. new addition.

What has been tested and how, request additional testing if necessary

Tested on Linux with --watch 2 to monitor output. Works as expected

Documentation impact (if any)

None

Signed-off-by: Akshay Tondak <aktondak@amd.com>
@github-actions
Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Copy link
Copy Markdown
Collaborator

@rchane rchane left a comment

Choose a reason for hiding this comment

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

Let's keep element in for now, as Alveo still needs it.

Signed-off-by: Akshay Tondak <aktondak@amd.com>
@github-actions
Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@aktondak aktondak changed the title Extend watch mode to all reports [XRT-SMI] Extend watch mode (--watch) to all examine reports Apr 14, 2026
@stsoe stsoe merged commit 2026f5d into Xilinx:master Apr 15, 2026
36 of 37 checks passed
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.

3 participants