Skip to content

Deployment checks#678

Open
manoj-freyr wants to merge 7 commits intomasterfrom
deployment-checks
Open

Deployment checks#678
manoj-freyr wants to merge 7 commits intomasterfrom
deployment-checks

Conversation

@manoj-freyr
Copy link
Collaborator

Display rocm version, OS, vbios and firmware vesions for the current running rvs environment.

@jkottiku
Copy link
Collaborator

OS and kernel versions also would be good add ons. Maybe can done as next upgrade.

info.node_id = node_id;
info.gpu_id = gpu_id;
info.device_id = dev_id;
auto ret = rsmi_dev_vbios_version_get(i, info.vbios_ver, 1024);// check returns TODO
Copy link
Collaborator

Choose a reason for hiding this comment

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

Check return value ?

@jkottiku jkottiku self-requested a review January 15, 2024 20:55
@manoj-freyr
Copy link
Collaborator Author

OS and kernel versions also would be good add ons. Maybe can done as next upgrade.

yeah can add later, but avoided as it is not directly correlated to ROCm

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