Skip to content

app: include die id in info command. #21

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.

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

Merged
merged 1 commit into from
Mar 26, 2025
Merged

Conversation

jlrivasp
Copy link
Contributor

@jlrivasp jlrivasp commented Mar 25, 2025

This patch updates the current die value shown in the info command. If a die name is defined in a record, the name will be shown, otherwise the die id numeric value will be shown in the info command.

For Crash Log records that don't have any die value defined, the info command will keep showing an empty string.

@jlrivasp jlrivasp requested a review from psurply March 25, 2025 07:47
This patch updates the current die value shown in the `info` command.
If a die name is defined in a record, the name will be shown, otherwise
the die id numeric value will be shown in the info command.

 For Crash Log records that don't have any die value defined, the `info`
 command will keep showing an empty string.

Signed-off-by: Rivas Paz, Jose L <jose.l.rivas.paz@intel.com>
Copy link
Contributor

@psurply psurply left a comment

Choose a reason for hiding this comment

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

Thank you!

@psurply psurply merged commit 9812742 into intel:main Mar 26, 2025
5 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.

2 participants