<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=utf-8"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="#0563C1" vlink="#954F72" style="word-wrap:break-word"> <p style="font-family:Arial;font-size:11pt;color:#0078D7;margin:5pt;" align="Left"> [AMD Official Use Only - Internal Distribution Only]<br> </p> <br> <div> <div class="WordSection1"> <p class="MsoNormal">+ Alex<o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <p class="MsoNormal">Hi all,<o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <p class="MsoNormal">Recently, many users reported the issue to us that fine grain not enabled. Actually, most of them are just caused by not switching to “manual” mode.<o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <p class="MsoNormal"> if (!(smu_dpm_ctx->dpm_level == AMD_DPM_FORCED_LEVEL_MANUAL)) {<br> - dev_warn(smu->adev->dev, "Fine grain is not enabled!\n");<br> + dev_warn(smu->adev->dev,<br> + "pp_od_clk_voltage is not accessible if power_dpm_force_perfomance_level is not in manual mode!\n");<o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <p class="MsoNormal">We have to need reminder in the warning message to tell the user where they are. <o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <p class="MsoNormal">Any objection for this patch? I found Navi series actually didn’t need this operation to update max/min clock levels. Would you clarify whether dGPU still needs this before we move the prints into amdgpu_pm.c?<o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <p class="MsoNormal">However, in APU fine grain design, patch looks good for me.<o:p></o:p></p> <p class="MsoNormal">Acked-by: Huang Rui <<a href="mailto:ray.huang@amd.com">ray.huang@amd.com</a>><o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <p class="MsoNormal">Thanks,<o:p></o:p></p> <p class="MsoNormal">Ray<o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> <p class="MsoNormal"><b>From:</b> Wang, Kevin(Yang) <Kevin1.Wang@amd.com> <br> <b>Sent:</b> Wednesday, January 20, 2021 12:10 PM<br> <b>To:</b> Du, Xiaojian <Xiaojian.Du@amd.com>; amd-gfx@lists.freedesktop.org<br> <b>Cc:</b> Huang, Ray <Ray.Huang@amd.com>; Quan, Evan <Evan.Quan@amd.com>; Lazar, Lijo <Lijo.Lazar@amd.com><br> <b>Subject:</b> Re: [PATCH] drm/amd/pm: make the error log more clear for fine grain tuning function<o:p></o:p></p> </div> </div> <p class="MsoNormal"><o:p> </o:p></p> <p style="margin:5.0pt"><span style="font-family:"Arial",sans-serif;color:#0078D7">[AMD Official Use Only - Internal Distribution Only]<o:p></o:p></span></p> <p class="MsoNormal"><o:p> </o:p></p> <div> <div> <p class="MsoNormal"><span style="font-size:12.0pt;color:black"><o:p> </o:p></span></p> </div> <div class="MsoNormal" align="center" style="text-align:center"> <hr size="2" width="98%" align="center"> </div> <div id="divRplyFwdMsg"> <p class="MsoNormal"><b><span style="color:black">From:</span></b><span style="color:black"> Du, Xiaojian <<a href="mailto:Xiaojian.Du@amd.com">Xiaojian.Du@amd.com</a>><br> <b>Sent:</b> Wednesday, January 20, 2021 11:48 AM<br> <b>To:</b> <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> <<a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a>><br> <b>Cc:</b> Huang, Ray <<a href="mailto:Ray.Huang@amd.com">Ray.Huang@amd.com</a>>; Quan, Evan <<a href="mailto:Evan.Quan@amd.com">Evan.Quan@amd.com</a>>; Wang, Kevin(Yang) <<a href="mailto:Kevin1.Wang@amd.com">Kevin1.Wang@amd.com</a>>; Lazar, Lijo <<a href="mailto:Lijo.Lazar@amd.com">Lijo.Lazar@amd.com</a>>; Du, Xiaojian <<a href="mailto:Xiaojian.Du@amd.com">Xiaojian.Du@amd.com</a>>; Du, Xiaojian <<a href="mailto:Xiaojian.Du@amd.com">Xiaojian.Du@amd.com</a>><br> <b>Subject:</b> [PATCH] drm/amd/pm: make the error log more clear for fine grain tuning function</span> <o:p></o:p></p> <div> <p class="MsoNormal"> <o:p></o:p></p> </div> </div> <div> <div> <p class="MsoNormal">From: Xiaojian Du <<a href="mailto:xiaojian.du@amd.com">xiaojian.du@amd.com</a>><br> <br> From: Xiaojian Du <<a href="mailto:Xiaojian.Du@amd.com">Xiaojian.Du@amd.com</a>><br> <br> This patch is to make the error log more clear for fine grian tuning<br> function, it covers Raven/Raven2/Picasso/Renoir/Vangogh.<br> The fine grain tuning function uses the sysfs file -- pp_od_clk_voltage,<br> but only when another sysfs file -- power_dpm_force_performance_level is<br> switched to "manual" mode, it is allowd to access "pp_od_clk_voltage".<br> <br> Signed-off-by: Xiaojian Du <<a href="mailto:Xiaojian.Du@amd.com">Xiaojian.Du@amd.com</a>><br> ---<br> drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c | 2 +-<br> drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c | 3 ++-<br> drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c | 3 ++-<br> 3 files changed, 5 insertions(+), 3 deletions(-)<br> <br> diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c<br> index 88322781e447..ed05a30d1139 100644<br> --- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c<br> +++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c<br> @@ -1487,7 +1487,7 @@ static int smu10_set_fine_grain_clk_vol(struct pp_hwmgr *hwmgr,<br> }<br> <br> if (!smu10_data->fine_grain_enabled) {<br> - pr_err("Fine grain not started\n");<br> + pr_err("pp_od_clk_voltage is not accessible if power_dpm_force_perfomance_level is not in manual mode!\n");<o:p></o:p></p> </div> <div> <p class="MsoNormal">[kevin]:<o:p></o:p></p> </div> <div> <p class="MsoNormal">for above codes, the old one looks better for me, i prefer to keep current design.<o:p></o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal">Best Regards,<o:p></o:p></p> </div> <div> <p class="MsoNormal">Kevin<br> return -EINVAL;<br> }<br> <br> diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c<br> index 6d3c556dbe6b..a847fa66797e 100644<br> --- a/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c<br> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c<br> @@ -1452,7 +1452,8 @@ static int vangogh_od_edit_dpm_table(struct smu_context *smu, enum PP_OD_DPM_TAB<br> struct smu_dpm_context *smu_dpm_ctx = &(smu->smu_dpm);<br> <br> if (!(smu_dpm_ctx->dpm_level == AMD_DPM_FORCED_LEVEL_MANUAL)) {<br> - dev_warn(smu->adev->dev, "Fine grain is not enabled!\n");<br> + dev_warn(smu->adev->dev,<br> + "pp_od_clk_voltage is not accessible if power_dpm_force_perfomance_level is not in manual mode!\n");<br> return -EINVAL;<br> }<br> <br> diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c<br> index ab15570305f7..4ce8fb1d5ce9 100644<br> --- a/drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c<br> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c<br> @@ -350,7 +350,8 @@ static int renoir_od_edit_dpm_table(struct smu_context *smu,<br> struct smu_dpm_context *smu_dpm_ctx = &(smu->smu_dpm);<br> <br> if (!(smu_dpm_ctx->dpm_level == AMD_DPM_FORCED_LEVEL_MANUAL)) {<br> - dev_warn(smu->adev->dev, "Fine grain is not enabled!\n");<br> + dev_warn(smu->adev->dev,<br> + "pp_od_clk_voltage is not accessible if power_dpm_force_perfomance_level is not in manual mode!\n");<br> return -EINVAL;<o:p></o:p></p> </div> <div> <p class="MsoNormal"><span style="font-family:"Segoe UI",sans-serif;color:#323130">[Kevin]:</span><o:p></o:p></p> </div> <p class="MsoNormal"><span style="font-family:"Segoe UI",sans-serif;color:#323130">Just tell the User what's going on, not why.</span><o:p></o:p></p> </div> <div> <p class="MsoNormal" style="text-align:justify"><span style="font-family:"Segoe UI",sans-serif">and we'd better make a function to check manual mode , the<span style="color:#323130">n embed it t</span>o every sysfs node in amdgpu_pm.c</span><o:p></o:p></p> <p class="MsoNormal" style="text-align:justify">using a unify interface to return result to user.<o:p></o:p></p> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal">Best Regards,<o:p></o:p></p> </div> <div> <p class="MsoNormal" style="margin-bottom:12.0pt">Kevin<br> }<br> <br> -- <br> 2.17.1<o:p></o:p></p> </div> </div> </div> </div> </div> </body> </html>