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

Russell, Kent Kent.Russell at amd.com
Tue Oct 2 11:43:21 UTC 2018


I don't do any parsing of the names of the Power Profiles in the SMI, this was just for the smi_lib which expects specific string formats. I had no issues with it, since it wouldn't muck with the SMI as it currently exists. But it will help if I decide to try to parse these in the SMI tool in the future.

 Kent

-----Original Message-----
From: Kuehling, Felix 
Sent: Monday, October 01, 2018 4:34 PM
To: Deucher, Alexander <Alexander.Deucher at amd.com>; Quan, Evan <Evan.Quan at amd.com>; Alex Deucher <alexdeucher at gmail.com>; amd-gfx at lists.freedesktop.org; Russell, Kent <Kent.Russell at amd.com>
Subject: Re: [PATCH] drm/amdgpu/vega20: make power profile output more consistent

On 2018-10-01 04:04 PM, Deucher, Alexander wrote:
>> -----Original Message-----
>> From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of 
>> Felix Kuehling
>> Sent: Monday, October 1, 2018 3:49 PM
>> To: Quan, Evan <Evan.Quan at amd.com>; Alex Deucher 
>> <alexdeucher at gmail.com>; amd-gfx at lists.freedesktop.org; Russell, Kent 
>> <Kent.Russell at amd.com>
>> Cc: Deucher, Alexander <Alexander.Deucher at amd.com>
>> Subject: Re: [PATCH] drm/amdgpu/vega20: make power profile output 
>> more consistent
>>
>> [+Kent]
>>
>> This may break the rocm-smi tool, which parses the power profile output.
>>
> This was at the request of rocm-smi. Since the current rocm-smi couldn't parse the new format.  This makes it more similar to previous asics.

Thanks! I missed that this was for Vega20.

Regards,
  Felix

>
> Alex
>
>> 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
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx



More information about the amd-gfx mailing list