Hi Lijo,

Thanks for the suggestion. I confirmed that the DPM tables are
indeed different on this hardware (GC 11.5.1, SMU 14.0.1):

pp_dpm_vclk: 800, 1472, 1605, 1766, 1962, 2208, 2523, 2944 MHz
pp_dpm_dclk: 800, 981, 1104, 1177, 1358, 1472, 1766, 2208 MHz

The original code was returning 0 for METRICS_AVERAGE_DCLK, so the
fall-through to VclkFrequency was an improvement but not fully
correct given the different DPM tables.

Since VcnClkLevelsEnabled applies to both VCLK and DCLK with the
same level count, I'll update patch 1 to match VclkFrequency to its
DPM level index and then report the DCLK frequency at that same
level for METRICS_AVERAGE_DCLK.

Will also check with the FW team about adding a DclkFrequency field
to SmuMetrics_t.

Will send v4 with the fix.

Regards,
Priya

Reply via email to