Skip to content

[XRT-SMI] debug report addition#9882

Merged
stsoe merged 2 commits into
Xilinx:masterfrom
aktondak:debug_report
Jun 25, 2026
Merged

[XRT-SMI] debug report addition#9882
stsoe merged 2 commits into
Xilinx:masterfrom
aktondak:debug_report

Conversation

@aktondak

Copy link
Copy Markdown
Collaborator

Problem solved by the commit

This PR adds a new Debug report to xrt-smi examine --advanced.
Currently this report prints 3 debug fields but this can be extended further in future if and when we need to capture states of driver/firmware. Example hysteresis.

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

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

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

Solved via adding new report and adding device query for auto-coredump feature.

Risks (if any) associated the changes in the commit

None, new report

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

Tested on MDS platform. This is how the report looks like :

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

---------------------------
[0000:c5:00.1] : NPU Strix
---------------------------
Debug Configurations
  Firmware Log        : enabled (log level 4)
  Event Trace         : disabled
  Auto Coredump       : not supported

Documentation impact (if any)

None

Signed-off-by: Akshay Tondak <aktondak@amd.com>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

Comment thread src/runtime_src/core/tools/common/reports/ReportDebug.h Outdated
@aktondak aktondak changed the title debug report addition [XRT-SMI] debug report addition Jun 24, 2026
Signed-off-by: Akshay Tondak <aktondak@amd.com>
@github-actions

Copy link
Copy Markdown
Contributor

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

@stsoe stsoe merged commit 863d1f5 into Xilinx:master Jun 25, 2026
21 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.

4 participants