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

Deucher, Alexander Alexander.Deucher at amd.com
Mon Oct 1 20:04:44 UTC 2018


> -----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.

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