Skip to content

Conversation

@aktondak
Copy link
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 <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2026

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

Signed-off-by: Akshay Tondak <[email protected]>
@github-actions
Copy link
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
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