Good point, I'll double-check that. Kent
From: Deucher, Alexander <[email protected]> Sent: Thursday, August 22, 2019 8:54 AM To: Russell, Kent <[email protected]>; [email protected] Subject: Re: [PATCH] drm/amdgpu: Fix Vega20 Average Power value Do we need an smu version check? Will AverageSocketPower report correctly on older versions of smu firmware? Assuming that is ok, patch is: Acked-by: Alex Deucher <[email protected]<mailto:[email protected]>> ________________________________ From: amd-gfx <[email protected]<mailto:[email protected]>> on behalf of Russell, Kent <[email protected]<mailto:[email protected]>> Sent: Thursday, August 22, 2019 8:19 AM To: [email protected]<mailto:[email protected]> <[email protected]<mailto:[email protected]>> Cc: Russell, Kent <[email protected]<mailto:[email protected]>> Subject: [PATCH] drm/amdgpu: Fix Vega20 Average Power value The SMU changed reading from CurrSocketPower to AverageSocketPower, so reflect this accordingly. This fixes the issue where Average Power Consumption was being reported as 0 from SMU 40.46-onward Change-Id: I471f93316820f1401cb497eefe29da68376a4bb9 Signed-off-by: Kent Russell <[email protected]<mailto:[email protected]>> --- drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c | 2 +- drivers/gpu/drm/amd/powerplay/vega20_ppt.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c index 0516c294b377..1820133f0ceb 100644 --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c @@ -2101,7 +2101,7 @@ static int vega20_get_gpu_power(struct pp_hwmgr *hwmgr, if (ret) return ret; - *query = metrics_table.CurrSocketPower << 8; + *query = metrics_table.AverageSocketPower << 8; return ret; } diff --git a/drivers/gpu/drm/amd/powerplay/vega20_ppt.c b/drivers/gpu/drm/amd/powerplay/vega20_ppt.c index e14363182691..17af12ee9e78 100644 --- a/drivers/gpu/drm/amd/powerplay/vega20_ppt.c +++ b/drivers/gpu/drm/amd/powerplay/vega20_ppt.c @@ -2927,7 +2927,7 @@ static int vega20_get_gpu_power(struct smu_context *smu, uint32_t *value) if (ret) return ret; - *value = metrics.CurrSocketPower << 8; + *value = metrics.AverageSocketPower << 8; return 0; } -- 2.17.1 _______________________________________________ amd-gfx mailing list [email protected]<mailto:[email protected]> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
