diff options
| author | Lyude Paul <[email protected]> | 2022-11-14 22:17:52 +0000 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2022-11-17 05:18:25 +0000 |
| commit | 7cce4cd628bee0d0caff7518c377cf8f599aa38f (patch) | |
| tree | 552c5dbee4c9e61eda9f47c40cca35ff948423ee /drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | |
| parent | drm/amd/display: Align dcn314_smu logging with other DCNs (diff) | |
| download | kernel-7cce4cd628bee0d0caff7518c377cf8f599aa38f.tar.gz kernel-7cce4cd628bee0d0caff7518c377cf8f599aa38f.zip | |
drm/amdgpu/mst: Stop ignoring error codes and deadlocking
It appears that amdgpu makes the mistake of completely ignoring the return
values from the DP MST helpers, and instead just returns a simple
true/false. In this case, it seems to have come back to bite us because as
a result of simply returning false from
compute_mst_dsc_configs_for_state(), amdgpu had no way of telling when a
deadlock happened from these helpers. This could definitely result in some
kernel splats.
V2:
* Address Wayne's comments (fix another bunch of spots where we weren't
passing down return codes)
Signed-off-by: Lyude Paul <[email protected]>
Fixes: 8c20a1ed9b4f ("drm/amd/display: MST DSC compute fair share")
Cc: Harry Wentland <[email protected]>
Cc: <[email protected]> # v5.6+
Reviewed-by: Wayne Lin <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c')
0 files changed, 0 insertions, 0 deletions