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.

Sign up for GitHub

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

Jump to bottom

driverInitialized check not valid #102

Open
mrusme opened this issue Mar 27, 2022 · 2 comments · May be fixed by #104
Open

driverInitialized check not valid #102

mrusme opened this issue Mar 27, 2022 · 2 comments · May be fixed by #104
Labels
enhancement

Comments

@mrusme
Copy link

mrusme commented Mar 27, 2022

This check is not valid since one can have the GPU module baked into the kernel. In that case, lsmod//proc/modules won't report it.
@littlewu2508
Copy link

ROCm/rocminfo#42 (comment) may help

littlewu2508 added a commit to littlewu2508/rocm_smi_lib that referenced this issue Apr 2, 2022
@littlewu2508
Check /sys/module/amdgpu for ROCk instead of lsmod
dc2a4b5 
Closes: ROCm#102

Signed-off-by: YiyangWu <xgreenlandforwyy@gmail.com>
@littlewu2508 littlewu2508 linked a pull request Apr 2, 2022 that will close this issue
Open
@dmitrii-galantsev
Copy link
Collaborator

Fixed as of 9f6614e. Please reopen if you disagree :)

littlewu2508 added a commit to littlewu2508/rocm_smi_lib that referenced this issue Jan 1, 2024
@littlewu2508
Just check /sys/module/amdgpu if initstate not present
f95cde2 
In case amdgpu is builtin

Closes: ROCm#102

Signed-off-by: YiyangWu <xgreenlandforwyy@gmail.com>
littlewu2508 added a commit to littlewu2508/rocm_smi_lib that referenced this issue Jan 1, 2024
@littlewu2508
Just check /sys/module/amdgpu if initstate not present
fe96886 
In case amdgpu is builtin

Closes: ROCm#102

Signed-off-by: YiyangWu <xgreenlandforwyy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check /sys/module/amdgpu for ROCk instead of lsmod littlewu2508/rocm_smi_lib
3 participants
@mrusme @dmitrii-galantsev @littlewu2508