[PATCH] drm/amdgpu/vega20: make power profile output more consistent

[Felix Kuehling]

[+Kent]

This may break the rocm-smi tool, which parses the power profile output.

Regards,
  Felix


On 2018-09-27 09:15 PM, Quan, Evan wrote:
> Reviewed-by: Evan Quan <evan.quan at amd.com>
>
>> -----Original Message-----
>> From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of Alex
>> Deucher
>> Sent: 2018年9月27日 21:46
>> To: amd-gfx at lists.freedesktop.org
>> Cc: Deucher, Alexander <Alexander.Deucher at amd.com>
>> Subject: [PATCH] drm/amdgpu/vega20: make power profile output more
>> consistent
>>
>> Make the profile name line match previous generations more closely.
>>
>> E.g.,
>> 0 3D_FULL_SCREEN :
>> vs:
>> 0(3D_FULL_SCREEN )
>>
>> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
>> ---
>>  drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c
>> b/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c
>> index 2a554f9edcda..75945fc50b3a 100644
>> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c
>> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c
>> @@ -3165,7 +3165,7 @@ static int vega20_get_power_profile_mode(struct
>> pp_hwmgr *hwmgr, char *buf)
>>  				"[GetPowerProfile] Failed to get activity
>> monitor!",
>>  				return result);
>>
>> -		size += sprintf(buf + size, "%2d(%14s%s)\n",
>> +		size += sprintf(buf + size, "%2d %14s%s:\n",
>>  			i, profile_name[i], (i == hwmgr-
>>> power_profile_mode) ? "*" : " ");
>>  		size += sprintf(buf + size,
>> "%19s %d(%13s) %7d %7d %7d %7d %7d %7d %7d %7d %7d\n",
>> --
>> 2.13.6
>>
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx