Skip to content

[XRT-SMI] Context health update to move to Option Options#9525

Merged
maxzhen merged 2 commits intoXilinx:masterfrom
aktondak:context_health_update
Jan 7, 2026
Merged

[XRT-SMI] Context health update to move to Option Options#9525
maxzhen merged 2 commits intoXilinx:masterfrom
aktondak:context_health_update

Conversation

@aktondak
Copy link
Copy Markdown
Collaborator

@aktondak aktondak commented Jan 5, 2026

Problem solved by the commit

This PR updates the context health feature within xrt-smi to function as a standalone option rather than a separate report.
Also fixes sudo requirements for xrt-smi features that need to be run in sudo mode

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

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

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

The problem was solved via creating a new Option option --context-health (similar to firmware-log and event-trace). Since context health itself has some sub options, the help printing is now much simpler and convenient for the end user. This is more inlined with other xrt-smi options since now there is no requirement for manually parsing the "ctx-id" and "pid" string and the flow is aligned with other fearures.

Risks (if any) associated the changes in the commit

None

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

Tested on linux :

aktondak@xsjstrix45:/proj/rdi/staff/aktondak/xdna/xdna-driver/xrt$ xrt-smi examine --advanced --context-health --help
-------------------------------------------------------------------------
                    DISCLAIMER  (xrt-smi --advanced)                     
You are running a developer command that may change system configuration.
                Continue only if you understand the risks.               
-------------------------------------------------------------------------

COMMAND: examine

DESCRIPTION: Display context health information

USAGE:  examine --context-health [-h] [--watch] [-d arg] [--ctx-id arg] [--pid arg]

OPTIONS:
  --context-health   - Display context health information
  -d, --device       - The Bus:Device.Function (e.g., 0000:d8:00.0) device of interest
  -h, --help         - Help to use this sub-command
  --watch            - Continuously monitor context health
  --ctx-id           - Comma-separated list of context IDs to filter (e.g., 1,2,3)
  --pid              - Comma-separated list of PIDs to filter (e.g., 1234,5678)

Documentation impact (if any)

None

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

github-actions Bot commented Jan 5, 2026

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

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

github-actions Bot commented Jan 5, 2026

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

@maxzhen maxzhen merged commit 20a196a into Xilinx:master Jan 7, 2026
89 of 103 checks passed
@aktondak aktondak deleted the context_health_update branch January 16, 2026 00:03
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