You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The for (i = 0; i < P_NUM_PLATFORMS; i++) in the variorum_print_gpu_utilization(void) API should be:
if (VARIORUM_WITH_NVIDIA_GPU || VARIORUM_WITH_AMD_GPU || VARIORUM_WITH_INTEL_GPU)
for (i = 0; i < P_NUM_PLATFORMS; i++)
{
#ifdef VARIORUM_WITH_INTEL_GPU
i = P_INTEL_GPU_IDX;
break;
#endif
#ifdef VARIORUM_WITH_NVIDIA_GPU
i = P_NVIDIA_GPU_IDX;
break;
#endif
#ifdef VARIORUM_WITH_AMD_GPU
i = P_AMD_GPU_IDX;
break;
#endif
}
or something similar
This is a GPU only API and should not be called on other (CPU) platforms. Results in an unimplemented error
The text was updated successfully, but these errors were encountered:
tpatki
changed the title
Variorum Print GPU utilization API should not call other platforms
Variorum should check for GPU build before calling GPU APIs (eg utilization or get_thermals etc)
Aug 15, 2023
The
for (i = 0; i < P_NUM_PLATFORMS; i++)
in thevariorum_print_gpu_utilization(void)
API should be:if (VARIORUM_WITH_NVIDIA_GPU || VARIORUM_WITH_AMD_GPU || VARIORUM_WITH_INTEL_GPU)
The text was updated successfully, but these errors were encountered: