drm/amd/pm: fix the deadlock issue observed on SI
The adev->pm.mutx is already held at the beginning of amdgpu_dpm_compute_clocks/amdgpu_dpm_enable_uvd/amdgpu_dpm_enable_vce. But on their calling path, amdgpu_display_bandwidth_update will be called and thus its sub functions amdgpu_dpm_get_sclk/mclk. They will then try to acquire the same adev->pm.mutex and deadlock will occur. By placing amdgpu_display_bandwidth_update outside of adev->pm.mutex protection(considering logically they do not need such protection) and restructuring the call flow accordingly, we can eliminate the deadlock issue. This comes with no real logics change. Fixes: 3712e7a4 ("drm/amd/pm: unified lock protections in amdgpu_dpm.c") Reported-by:Paul Menzel <pmenzel@molgen.mpg.de> Reported-by:
Arthur Marsh <arthur.marsh@internode.on.net> Link: https://lore.kernel.org/all/9e689fea-6c69-f4b0-8dee-32c4cf7d8f9c@molgen.mpg.de/ BugLink: drm/amd#1957 Signed-off-by:
Evan Quan <evan.quan@amd.com> Reviewed-by:
Lijo Lazar <lijo.lazar@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
Showing
- drivers/gpu/drm/amd/pm/amdgpu_dpm.c 39 additions, 0 deletionsdrivers/gpu/drm/amd/pm/amdgpu_dpm.c
- drivers/gpu/drm/amd/pm/legacy-dpm/legacy_dpm.c 0 additions, 10 deletionsdrivers/gpu/drm/amd/pm/legacy-dpm/legacy_dpm.c
- drivers/gpu/drm/amd/pm/legacy-dpm/si_dpm.c 0 additions, 35 deletionsdrivers/gpu/drm/amd/pm/legacy-dpm/si_dpm.c
- drivers/gpu/drm/amd/pm/powerplay/amd_powerplay.c 0 additions, 10 deletionsdrivers/gpu/drm/amd/pm/powerplay/amd_powerplay.c
Loading
Please register or sign in to comment