Skip to content
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

Ensure PCI id is lower case so it matches the case of lspci. #149

Merged
merged 1 commit into from
Sep 27, 2024
Merged

Ensure PCI id is lower case so it matches the case of lspci. #149

merged 1 commit into from
Sep 27, 2024

Conversation

veldenb
Copy link
Contributor

@veldenb veldenb commented Sep 27, 2024

For me this fixes #131.
nvidia-smi outputs PCI id's as 00000000:0B:00.0 while lspci lists the PCI-id as 0b:00.0 (capital vs lower B).
This fix ensures the case matches so the extension can pick the right PCI-id for the stats to be loaded.

@ljuzig ljuzig merged commit 7cb3dec into AstraExt:main Sep 27, 2024
@ljuzig
Copy link
Collaborator

ljuzig commented Sep 27, 2024

Nice catch! Thank you!

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.

GPU stats broken for Ubuntu 24.04 + Nvidia
2 participants