<div dir="ltr"><div><div><div><div><div>Hello Asher, </div>Thanks for the information, is it open-source tests? Can I reproduce it? </div>Also, it seems like Radeon Instinct MI25 was released before Radeon RX Vega, is it possible that they have different PowerPlay subsystems? </div>On my Vega56, all these registers from vega20 are working very well. </div>Seems like we should split implementation somehow. </div>Regards, </div> <div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Oct 26, 2022 at 3:51 PM Song, Asher <<a href="mailto:Asher.Song@amd.com">Asher.Song@amd.com</a>> wrote: </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="msg7722690196034941173"> <div style="overflow-wrap: break-word;" lang="EN-US"> <div class="m_7722690196034941173WordSection1"> [AMD Official Use Only - General] +<a id="m_7722690196034941173OWAAM2C3FD9A9EF544EB3A89BBBB72AF67D08" href="mailto:Xiaohu.Meng@amd.com" target="_blank">@Meng, Xiaohu</a> Hi Zhuravlev, The information of test card is as following: Lspci name: [AMD/ATI] Vega 10 [Radeon Instinct MI25 MxGPU] (rev 05) Firmware: ATOM BIOS: 113-D0531800-C04 When run test scripts compute-rocm-*/utils/run_rsmitst.sh, the below test cases fail. [ FAILED ] 4 tests, listed below: [ FAILED ] rsmitstReadOnly.TestOverdriveRead [ FAILED ] rsmitstReadWrite.FanReadWrite [ FAILED ] rsmitstReadWrite.TestOverdriveReadWrite [ FAILED ] rsmitstReadWrite.TestPowerCapReadWrite Best Regards, Asher <div style="border-color:rgb(225,225,225) currentcolor currentcolor;border-style:solid none none;border-width:1pt medium medium;padding:3pt 0in 0in"> From: Yury Zhuravlev <<a href="mailto:stalkerg@gmail.com" target="_blank">stalkerg@gmail.com</a>> Sent: Wednesday, October 26, 2022 9:31 AM To: Song, Asher <<a href="mailto:Asher.Song@amd.com" target="_blank">Asher.Song@amd.com</a>> Cc: Deucher, Alexander <<a href="mailto:Alexander.Deucher@amd.com" target="_blank">Alexander.Deucher@amd.com</a>>; Chen, Guchun <<a href="mailto:Guchun.Chen@amd.com" target="_blank">Guchun.Chen@amd.com</a>>; Quan, Evan <<a href="mailto:Evan.Quan@amd.com" target="_blank">Evan.Quan@amd.com</a>>; <a href="mailto:amd-gfx@lists.freedesktop.org" target="_blank">amd-gfx@lists.freedesktop.org</a> Subject: Re: [PATCH] drm/amdgpu: Revert "drm/amdgpu: getting fan speed pwm for vega10 properly" </div> <div> <div> <div> Hello, Can you write here your card name and firmware version? Without this patch, my fan sensors are broken, and it's sensitive to my pc case with water cooling. My card is: Sapphire Pulse Radeon RX Vega 56 vd6546 SA lspci name: [AMD/ATI] Vega 10 XL/XT [Radeon RX Vega 56/64] (rev c3) Firmware: ATOM BIOS: 113-376XL-UO2 This patch is 100% working correct on my machine, and I tested it last 2 months. </div> <div> Regards, </div> </div> </div> <div> <div> On Fri, Oct 14, 2022 at 1:15 PM Asher Song <<a href="mailto:Asher.Song@amd.com" target="_blank">Asher.Song@amd.com</a>> wrote: </div> <blockquote style="border-color:currentcolor currentcolor currentcolor rgb(204,204,204);border-style:none none none solid;border-width:medium medium medium 1pt;padding:0in 0in 0in 6pt;margin:5pt 0in 5pt 4.8pt"> This reverts commit fe01cb24b81c0091d7e5668874d51ce913e44a97. Unfortunately, that commit causes fan monitors can't be read and written properly. Signed-off-by: Asher Song <<a href="mailto:Asher.Song@amd.com" target="_blank">Asher.Song@amd.com</a>> --- .../amd/pm/powerplay/hwmgr/vega10_thermal.c | 25 ++++++++++--------- 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_thermal.c b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_thermal.c index 190af79f3236..dad3e3741a4e 100644 --- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_thermal.c +++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_thermal.c @@ -67,21 +67,22 @@ int vega10_fan_ctrl_get_fan_speed_info(struct pp_hwmgr *hwmgr, int vega10_fan_ctrl_get_fan_speed_pwm(struct pp_hwmgr *hwmgr, uint32_t *speed) { - struct amdgpu_device *adev = hwmgr->adev; - uint32_t duty100, duty; - uint64_t tmp64; + uint32_t current_rpm; + uint32_t percent = 0; - duty100 = REG_GET_FIELD(RREG32_SOC15(THM, 0, mmCG_FDO_CTRL1), - CG_FDO_CTRL1, FMAX_DUTY100); - duty = REG_GET_FIELD(RREG32_SOC15(THM, 0, mmCG_THERMAL_STATUS), - CG_THERMAL_STATUS, FDO_PWM_DUTY); + if (hwmgr->thermal_controller.fanInfo.bNoFan) + return 0; - if (!duty100) - return -EINVAL; + if (vega10_get_current_rpm(hwmgr, &current_rpm)) + return -1; + + if (hwmgr->thermal_controller. + advanceFanControlParameters.usMaxFanRPM != 0) + percent = current_rpm * 255 / + hwmgr->thermal_controller. + advanceFanControlParameters.usMaxFanRPM; - tmp64 = (uint64_t)duty * 255; - do_div(tmp64, duty100); - *speed = MIN((uint32_t)tmp64, 255); + *speed = MIN(percent, 255); return 0; } -- 2.25.1 </blockquote> </div> </div> </div> </div></blockquote></div>