<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-2022-jp"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="blue" vlink="purple"> <div class="WordSection1"> Indeed, it allowed me to apply the profile even with no previous profile set (so it basically just applied all zeros). I’ll rebase my 2 patches on your patch once that is pushed to drm-next.<o:p></o:p> <o:p> </o:p> Kent<o:p></o:p> <o:p> </o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: Quan, Evan Sent: Tuesday, March 26, 2019 5:10 AM To: Russell, Kent <Kent.Russell@amd.com>; amd-gfx@lists.freedesktop.org Subject: RE: [PATCH 1/3] drm/amdgpu: Stop returning EINVAL during profile switch<o:p></o:p> </div> </div> <o:p> </o:p> That seems a bug. I will fix that.<o:p></o:p> It’s OK to store previous CUSTOM profile settings in driver and switch back to that with no need to pass in settings again. That makes sense. <o:p></o:p> But you may need to update the patch(to check whether there is previous custom profile settings).<o:p></o:p> <o:p> </o:p> Regards,<o:p></o:p> Evan<o:p></o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: amd-gfx <<a href="mailto:amd-gfx-bounces@lists.freedesktop.org">amd-gfx-bounces@lists.freedesktop.org</a>> On Behalf Of Russell, Kent Sent: 2019年3月26日 0:00 To: Quan, Evan <<a href="mailto:Evan.Quan@amd.com">Evan.Quan@amd.com</a>>; <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> Subject: Re: [PATCH 1/3] drm/amdgpu: Stop returning EINVAL during profile switch<o:p></o:p> </div> </div> <o:p> </o:p> I set CUSTOM to a weird value like "1 2 3 4 5 6" , then switched to VR profile, then switched back to CUSTOM (just with "echo 6 > profile), and it kept my custom values. Before this patch, it throws EINVAL but still switches it to CUSTOM with the saved values, on both vega10 and vega20. Kent KENT RUSSELL Sr. Software Engineer | Linux Compute Kernel 1 Commerce Valley Drive East Markham, ON L3T 7X6 O +(1) 289-695-2122 | Ext 72122<o:p></o:p> <div class="MsoNormal" align="center" style="text-align:center"> <hr size="2" width="98%" align="center"> </div> <div id="divRplyFwdMsg"> From: Quan, Evan Sent: Monday, March 25, 2019 10:14:03 AM To: Russell, Kent; <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> Cc: Russell, Kent Subject: RE: [PATCH 1/3] drm/amdgpu: Stop returning EINVAL during profile switch <o:p></o:p> <div> <o:p></o:p> </div> </div> <div> <div> How can you tell whether it was already applied before or 1st time to switch to compute mode? For the latter case, other settings/parameters do need. Regards, Evan > -----Original Message----- > From: amd-gfx <<a href="mailto:amd-gfx-bounces@lists.freedesktop.org">amd-gfx-bounces@lists.freedesktop.org</a>> On Behalf Of > Russell, Kent > Sent: 2019年3月25日 20:42 > To: <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> > Cc: Russell, Kent <<a href="mailto:Kent.Russell@amd.com">Kent.Russell@amd.com</a>> > Subject: [PATCH 1/3] drm/amdgpu: Stop returning EINVAL during profile > switch > > Don't return an error after switching to the CUSTOM profile, if there aren't > any CUSTOM values passed in. The CUSTOM profile is already applied, so if > no other arguments are passed, just return instead of throwing -EINVAL > when successful > > Change-Id: I114cc9783226ee9ebb146863897e951527a85e20 > Signed-off-by: Kent Russell <<a href="mailto:kent.russell@amd.com">kent.russell@amd.com</a>> > --- > drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 6 +++++- > drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c | 5 ++++- > 2 files changed, 9 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c > b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c > index ed6c638700f5..a98b927282ac 100644 > --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c > +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c > @@ -4911,7 +4911,11 @@ static int vega10_set_power_profile_mode(struct > pp_hwmgr *hwmgr, long *input, ui > 1<<hwmgr- > >power_profile_mode); > > if (hwmgr->power_profile_mode == > PP_SMC_POWER_PROFILE_CUSTOM) { > - if (size == 0 || size > 4) > + /* If size = 0, then just return, it was applied above */ > + if (size == 0) > + return 0; > + > + if (size != 4) > return -EINVAL; > > data->custom_profile_mode[0] = busy_set_point = input[0]; > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c > b/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c > index 664544e7fcdc..225b2102c82c 100644 > --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c > +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega20_hwmgr.c > @@ -3828,8 +3828,11 @@ static int vega20_set_power_profile_mode(struct > pp_hwmgr *hwmgr, long *input, ui > } > > if (hwmgr->power_profile_mode == > PP_SMC_POWER_PROFILE_CUSTOM) { > - if (size < 10) > + if (size < 10 && size != 0) > return -EINVAL; > + /* If size = 0, then just return, it was applied above */ > + if (size == 0) > + return 0; > > result = vega20_get_activity_monitor_coeff(hwmgr, > (uint8_t *)(&activity_monitor), > -- > 2.17.1 > > _______________________________________________ > amd-gfx mailing list > <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> > <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a><o:p></o:p> </div> </div> </div> </div> </div> </body> </html>