diff options
author | Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> | 2023-12-14 16:46:35 -0500 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2024-01-03 10:47:49 -0500 |
commit | b8a204fb1a97b39a7fcaefbf2c6c4d01aa4f3c57 (patch) | |
tree | 1a4375dac2aacbe1eb926ec5b1ac3e3ebd7d7ba7 /drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource_helpers.c | |
parent | 9ade4870b87b09e1f132ba92c1ab13a6769d1b0f (diff) |
drm/amd/display: Verify disallow bits were cleared for idle
[Why]
A hang was observed where a read-modify-write access occurred due to the
register for idle state being shared between DMCUB and driver.
dmcub read - idle allow / no commit
driver read - idle allow / no commit
driver write - idle disallow / no commit
dmcub write - idle allow / commit
Resulting in DMCUB re-entering IPS after a disable and keeping the allow
high.
[How]
Long term we need to split commit/allow into two registers or use shared
DRAM state, but short term we can reduce the repro rate by ensuring that
the disallow went through by bounding the expected worst case scenario.
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Reviewed-by: Hansen Dsouza <hansen.dsouza@amd.com>
Reviewed-by: Ovidiu Bunea <ovidiu.bunea@amd.com>
Acked-by: Rodrigo Siqueira <rodrigo.siqueira@amd.com>
Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource_helpers.c')
0 files changed, 0 insertions, 0 deletions