search

LLNL / variorum

Vendor-neutral library for exposing power and performance features across diverse architectures
https://variorum.readthedocs.io
MIT License
67 stars 26 forks source link

Verify correct frequency JSON API for Intel 4F #535

Closed slabasan closed 5 months ago

slabasan commented 5 months ago

https://github.com/LLNL/variorum/blob/dev/src/variorum/Intel/config_intel.c#L226

Should this be referencing intel_cpu_fm_06_4f_get_clocks_json instead of intel_cpu_fm_06_3f_get_clocks_json?

        g_platform[idx].variorum_get_thermals_json =
            intel_cpu_fm_06_4f_get_thermals_json;
        g_platform[idx].variorum_get_frequency_json =
            intel_cpu_fm_06_3f_get_clocks_json;