[PATCH i-g-t] HAX/DONOT_MERGE: Add debug prints to HDR tests.

Bhanuprakash Modem bhanuprakash.modem at intel.com
Tue May 21 09:45:28 UTC 2024


Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
---
 lib/igt_kms.c   | 10 ++++++++--
 tests/kms_hdr.c | 21 +++++++++++++++++----
 2 files changed, 25 insertions(+), 6 deletions(-)

diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index f82102144..df6c8f8fc 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -6189,15 +6189,21 @@ bool igt_max_bpc_constraint(igt_display_t *display, enum pipe pipe,
 	for_each_connector_mode(output) {
 		igt_output_override_mode(output, &connector->modes[j__]);
 
+		kmstest_dump_mode(&connector->modes[j__]);
+
 		if (is_intel_device(display->drm_fd) &&
-		    !igt_check_bigjoiner_support(display))
+		    !igt_check_bigjoiner_support(display)) {
+			igt_info("Bigjoiner not supported, on selected combo, trying another mode\n");
 			continue;
+		}
 
 		igt_display_commit2(display, display->is_atomic ? COMMIT_ATOMIC : COMMIT_LEGACY);
 
 		if (!igt_check_output_bpc_equal(display->drm_fd, pipe,
-						output->name, bpc))
+						output->name, bpc)) {
+			igt_info("Can't use this mode to match the requested bpc-%d, trying another mode\n", bpc);
 			continue;
+		}
 
 		return true;
 	}
diff --git a/tests/kms_hdr.c b/tests/kms_hdr.c
index 99b78f7ef..cc5a08f8c 100644
--- a/tests/kms_hdr.c
+++ b/tests/kms_hdr.c
@@ -621,25 +621,37 @@ static void test_hdr(data_t *data, uint32_t flags)
 		 * set MAX_BPC property to 10bpc prior to setting
 		 * HDR metadata property. Therefore, checking.
 		 */
-		if (!has_max_bpc(output) || !has_hdr(output))
+		if (!has_max_bpc(output) || !has_hdr(output)) {
+			igt_info("Output %s: doesn't support IGT_CONNECTOR_MAX_BPC or IGT_CONNECTOR_HDR_OUTPUT_METADATA\n", output->name);
 			continue;
+		}
 
 		/* For negative test, panel should be non-hdr. */
-		if ((flags & TEST_INVALID_HDR) && is_panel_hdr(data, output))
+		if ((flags & TEST_INVALID_HDR) && is_panel_hdr(data, output)) {
+			igt_info("Output %s: Can't run negative test on HDR panel\n", output->name);
 			continue;
+		}
 
-		if ((flags & ~TEST_INVALID_HDR) && !is_panel_hdr(data, output))
+		if ((flags & ~TEST_INVALID_HDR) && !is_panel_hdr(data, output)) {
+			igt_info("Output %s: Can't run HDR tests on non-HDR panel\n", output->name);
 			continue;
+		}
 
-		if (igt_get_output_max_bpc(data->fd, output->name) < 10)
+		if (igt_get_output_max_bpc(data->fd, output->name) < 10) {
+			igt_info("Output %s: doesn't support 10 bpc\n", output->name);
 			continue;
+		}
 
 		for_each_pipe(display, pipe) {
+			igt_info("Using the combo: pipe-%s + %s\n", kmstest_pipe_name(pipe), output->name);
+
 			if (igt_pipe_connector_valid(pipe, output)) {
 				prepare_test(data, output, pipe);
 
 				if (is_intel_device(data->fd) &&
 				    !igt_max_bpc_constraint(display, pipe, output, 10)) {
+					igt_info("No suitable mode found to use 10 bpc.\n");
+
 					test_fini(data);
 					break;
 				}
@@ -661,6 +673,7 @@ static void test_hdr(data_t *data, uint32_t flags)
 				/* One pipe is enough */
 				break;
 			}
+			igt_info("pipe-%s-%s: Invalid combo\n", kmstest_pipe_name(pipe), output->name);
 		}
 	}
 }
-- 
2.43.2



More information about the igt-dev mailing list