On 1/7/24 11:51, Melissa Wen wrote:
On 01/06, LIPSKI, IVAN wrote:
[AMD Official Use Only - General]
@Siqueira, Rodrigo<mailto:rodrigo.sique...@amd.com>
________________________________
From: LIPSKI, IVAN <ivan.lip...@amd.com>
Sent: January 5, 2024 7:40 PM
To: amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>
Cc: rodrigo.siqui...@amd.com <rodrigo.siqui...@amd.com>; Choi, Nicholas <nicholas.c...@amd.com>; Deucher, Alexander
<alexander.deuc...@amd.com>; Koenig, Christian <christian.koe...@amd.com>; Wentland, Harry
<harry.wentl...@amd.com>; LIPSKI, IVAN <ivan.lip...@amd.com>; Melissa Wen <m...@igalia.com>; Mahfooz, Hamza
<hamza.mahf...@amd.com>
Subject: [PATCH 1/1] Revert "drm/amd/display: fix bandwidth validation failure on
DCN 2.1"
From: Ivan Lipski <ivlip...@amd.com>
This commit causes dmesg-warn on several IGT tests on DCN 3.1.6:
*ERROR* link_enc_cfg_validate: Invalid link encoder assignments - 0x1c
Affected IGT tests include:
amdgpu/[amd_assr|amd_plane|amd_hotplug]
kms_atomic
kms_color
kms_flip
kms_properties
kms_universal_plane
and some other tests
This reverts commit b7ebd39e2922f642c7ee63ade4a4a5a1ef675d84.
I'm not opposed to reverting this in the short term, but I don't see the
connection between doing a full validation and link encoder assignment
errors. It seems more like the change unveiled an underlying issue
rather than causing the error. I don't see those errors on DCN 2.1 and
3.0.1.
Yeah... I totally agree with you. I'll try to get one DCN 316 device to
validate this issue. For now, let's revert this change since it causes a
large regression in our CI.
Reviewed-by: Rodrigo Siqueira <rodrigo.sique...@amd.com>
Also applied to asdn.
Thanks
Siqueira
Unfortunately, I don't have a DCN 3.1.6 for debugging :( It'd deserve
further investigation.
Melissa
Cc: Melissa Wen <m...@igalia.com>
Cc: Hamza Mahfooz <hamza.mahf...@amd.com>
Signed-off-by: Ivan Lipski <ivlip...@amd.com>
---
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index d77fc79f3542..111c6f51f0ae 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -10752,7 +10752,7 @@ static int amdgpu_dm_atomic_check(struct drm_device
*dev,
DRM_DEBUG_DRIVER("drm_dp_mst_atomic_check()
failed\n");
goto fail;
}
- status = dc_validate_global_state(dc, dm_state->context, false);
+ status = dc_validate_global_state(dc, dm_state->context, true);
if (status != DC_OK) {
DRM_DEBUG_DRIVER("DC global validation failure: %s
(%d)",
dc_status_to_str(status), status);
--
2.34.1