ROCm / rocminfo

ROCm Application for Reporting System Info
Other
32 stars 30 forks source link

Fix rocminfo when run within docker environments. #48

Closed iotamudelta closed 2 years ago

iotamudelta commented 2 years ago

Currently, rocminfo will fail when executed inside a docker container due to a missing ROCk module. This has impacts on rocprofiler use. However, a missing ROCk module is expected within a docker environment.

Fix this by searching for the "docker" string within the self-cgroups. If it is found, we do not error on a missing ROCk module.

fxkamd commented 2 years ago

All the whitespace patches should be squashed into the original patch.

As an alternative idea, instead of looking for amdgpu in lspci, can the container see the directory /sys/module/amdgpu?

iotamudelta commented 2 years ago

@fxkamd that directory is visible from within the container environment

iotamudelta commented 2 years ago

Abandon this solution in favor of the superior solution suggested by @fxkamd and implemented in #49