Re: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"
On 10/26/2018 8:34 AM, Alex Deucher wrote: > On Thu, Oct 25, 2018 at 10:52 PM Agrawal, Akshu wrote: >> >> >> >> On 10/26/2018 8:09 AM, Alex Deucher wrote: >>> On Thu, Oct 25, 2018 at 12:27 PM S, Shirish wrote: This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. Reason for revert: This patch sends msg PPSMC_MSG_DisableLowMemoryPstate(0x002e) in wrong of sequence to SMU which is before PPSMC_MSG_UVDPowerON (0x0008). This leads to SMU failing to service the request as it is dependent on UVD to be powered ON, since it accesses UVD registers. >>> >>> Does this patch that is being reverted actually break something or is >>> it ok to leave as a workaround? It supposedly fixed display issues at >>> 4k with video. Reverting it will bring that back won't it? >>> >>> Alex >>> >> Yes Alex, it will break 4k video as there will be underrun. But we are >> working on patches that will Disable memory NBPstate only for 4k videos. >> We can have this patch in and will be posting couple of patches to fix >> 4k videos display issues. > > Can we land them all together? Otherwise, we'll have a regressed > state until the later fixes land. > > Alex > Agreed, will push them as a series. Thanks, Akshu ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"
On Thu, Oct 25, 2018 at 10:52 PM Agrawal, Akshu wrote: > > > > On 10/26/2018 8:09 AM, Alex Deucher wrote: > > On Thu, Oct 25, 2018 at 12:27 PM S, Shirish wrote: > >> > >> This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. > >> > >> Reason for revert: > >> This patch sends msg PPSMC_MSG_DisableLowMemoryPstate(0x002e) > >> in wrong of sequence to SMU which is before PPSMC_MSG_UVDPowerON (0x0008). > >> This leads to SMU failing to service the request as it is > >> dependent on UVD to be powered ON, since it accesses UVD > >> registers. > > > > Does this patch that is being reverted actually break something or is > > it ok to leave as a workaround? It supposedly fixed display issues at > > 4k with video. Reverting it will bring that back won't it? > > > > Alex > > > Yes Alex, it will break 4k video as there will be underrun. But we are > working on patches that will Disable memory NBPstate only for 4k videos. > We can have this patch in and will be posting couple of patches to fix > 4k videos display issues. Can we land them all together? Otherwise, we'll have a regressed state until the later fixes land. Alex ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"
On 10/26/2018 8:09 AM, Alex Deucher wrote: > On Thu, Oct 25, 2018 at 12:27 PM S, Shirish wrote: >> >> This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. >> >> Reason for revert: >> This patch sends msg PPSMC_MSG_DisableLowMemoryPstate(0x002e) >> in wrong of sequence to SMU which is before PPSMC_MSG_UVDPowerON (0x0008). >> This leads to SMU failing to service the request as it is >> dependent on UVD to be powered ON, since it accesses UVD >> registers. > > Does this patch that is being reverted actually break something or is > it ok to leave as a workaround? It supposedly fixed display issues at > 4k with video. Reverting it will bring that back won't it? > > Alex > Yes Alex, it will break 4k video as there will be underrun. But we are working on patches that will Disable memory NBPstate only for 4k videos. We can have this patch in and will be posting couple of patches to fix 4k videos display issues. Thanks, Akshu ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"
On Thu, Oct 25, 2018 at 12:27 PM S, Shirish wrote: > > This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. > > Reason for revert: > This patch sends msg PPSMC_MSG_DisableLowMemoryPstate(0x002e) > in wrong of sequence to SMU which is before PPSMC_MSG_UVDPowerON (0x0008). > This leads to SMU failing to service the request as it is > dependent on UVD to be powered ON, since it accesses UVD > registers. Does this patch that is being reverted actually break something or is it ok to leave as a workaround? It supposedly fixed display issues at 4k with video. Reverting it will bring that back won't it? Alex > > This msg should ideally be sent only when the UVD is about to decode > a 4k video. > > Signed-off-by: Shirish S > --- > drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c | 5 + > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c > b/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c > index fef111d..53cf787 100644 > --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c > +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c > @@ -1228,17 +1228,14 @@ static int smu8_dpm_force_dpm_level(struct pp_hwmgr > *hwmgr, > > static int smu8_dpm_powerdown_uvd(struct pp_hwmgr *hwmgr) > { > - if (PP_CAP(PHM_PlatformCaps_UVDPowerGating)) { > - smu8_nbdpm_pstate_enable_disable(hwmgr, true, true); > + if (PP_CAP(PHM_PlatformCaps_UVDPowerGating)) > return smum_send_msg_to_smc(hwmgr, PPSMC_MSG_UVDPowerOFF); > - } > return 0; > } > > static int smu8_dpm_powerup_uvd(struct pp_hwmgr *hwmgr) > { > if (PP_CAP(PHM_PlatformCaps_UVDPowerGating)) { > - smu8_nbdpm_pstate_enable_disable(hwmgr, false, true); > return smum_send_msg_to_smc_with_parameter( > hwmgr, > PPSMC_MSG_UVDPowerON, > -- > 2.7.4 > > ___ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
RE: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"
Patch is Acked-by: Rex Zhu Regards Rex > -Original Message- > From: S, Shirish > Sent: Friday, October 26, 2018 12:28 AM > To: Zhu, Rex ; Agrawal, Akshu > ; Deucher, Alexander > > Cc: amd-gfx@lists.freedesktop.org; S, Shirish > Subject: [PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on > On/OFF of UVD" > > This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. > > Reason for revert: > This patch sends msg PPSMC_MSG_Disabl eLowMemoryPstate(0x002e) > in wrong of sequence to SMU which is before PPSMC_MSG_UVDPowerON > (0x0008). > This leads to SMU failing to service the request as it is dependent on UVD to > be powered ON, since it accesses UVD registers. > > This msg should ideally be sent only when the UVD is about to decode a 4k > video. > > Signed-off-by: Shirish S > --- > drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c | 5 + > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c > b/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c > index fef111d..53cf787 100644 > --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c > +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c > @@ -1228,17 +1228,14 @@ static int smu8_dpm_force_dpm_level(struct > pp_hwmgr *hwmgr, > > static int smu8_dpm_powerdown_uvd(struct pp_hwmgr *hwmgr) { > - if (PP_CAP(PHM_PlatformCaps_UVDPowerGating)) { > - smu8_nbdpm_pstate_enable_disable(hwmgr, true, true); > + if (PP_CAP(PHM_PlatformCaps_UVDPowerGating)) > return smum_send_msg_to_smc(hwmgr, > PPSMC_MSG_UVDPowerOFF); > - } > return 0; > } > > static int smu8_dpm_powerup_uvd(struct pp_hwmgr *hwmgr) { > if (PP_CAP(PHM_PlatformCaps_UVDPowerGating)) { > - smu8_nbdpm_pstate_enable_disable(hwmgr, false, true); > return smum_send_msg_to_smc_with_parameter( > hwmgr, > PPSMC_MSG_UVDPowerON, > -- > 2.7.4 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
[PATCH] Revert "drm/amd/powerplay: Enable/Disable NBPSTATE on On/OFF of UVD"
This reverts commit dbd8299c32f6f413f6cfe322fe0308f3cfc577e8. Reason for revert: This patch sends msg PPSMC_MSG_DisableLowMemoryPstate(0x002e) in wrong of sequence to SMU which is before PPSMC_MSG_UVDPowerON (0x0008). This leads to SMU failing to service the request as it is dependent on UVD to be powered ON, since it accesses UVD registers. This msg should ideally be sent only when the UVD is about to decode a 4k video. Signed-off-by: Shirish S --- drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c index fef111d..53cf787 100644 --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu8_hwmgr.c @@ -1228,17 +1228,14 @@ static int smu8_dpm_force_dpm_level(struct pp_hwmgr *hwmgr, static int smu8_dpm_powerdown_uvd(struct pp_hwmgr *hwmgr) { - if (PP_CAP(PHM_PlatformCaps_UVDPowerGating)) { - smu8_nbdpm_pstate_enable_disable(hwmgr, true, true); + if (PP_CAP(PHM_PlatformCaps_UVDPowerGating)) return smum_send_msg_to_smc(hwmgr, PPSMC_MSG_UVDPowerOFF); - } return 0; } static int smu8_dpm_powerup_uvd(struct pp_hwmgr *hwmgr) { if (PP_CAP(PHM_PlatformCaps_UVDPowerGating)) { - smu8_nbdpm_pstate_enable_disable(hwmgr, false, true); return smum_send_msg_to_smc_with_parameter( hwmgr, PPSMC_MSG_UVDPowerON, -- 2.7.4 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx