<!DOCTYPE html><html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<p>Hi Pranay,</p>
<div class="moz-cite-prefix">On 8/12/2025 1:29 PM, Pranay Samala
wrote:<br>
</div>
<blockquote type="cite" cite="mid:20250812075909.2119126-1-pranay.samala@intel.com">
<pre wrap="" class="moz-quote-pre">To prevent disk usage issues during CI runs with multi-output,
this patch limits debug logging to DRM_UT_DRIVER for modeset
transition tests when more than two outputs are connected
v2: Include all modeset tests (Rama)
Access display structure and avoid temporary variable (Rama)
Signed-off-by: Pranay Samala <a class="moz-txt-link-rfc2396E" href="mailto:pranay.samala@intel.com"><pranay.samala@intel.com></a>
---
tests/kms_atomic_transition.c | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
index 8c6094987..1b81a01a3 100644
--- a/tests/kms_atomic_transition.c
+++ b/tests/kms_atomic_transition.c
@@ -1096,6 +1096,17 @@ static bool pipe_output_combo_valid(igt_display_t *display,
return ret;
}
+static int count_conn_outputs(igt_display_t *display)
+{
+ int conn_outputs = 0;
+
+ for (int i = 0; i < display->n_outputs; i++) {
+ if (igt_output_is_connected(&display->outputs[i]))
+ conn_outputs++;
+ }
+ return conn_outputs;
+}
+</pre>
</blockquote>
<p>This function might be useful in other places too. It would be
better to move it to a common library so we can reuse it easily in
future tests.</p>
<p>Change the function name to <span><span>igt_get_connected_output_count</span></span><span style="white-space: pre-wrap">(igt_display_t *display)</span></p>
<blockquote type="cite" cite="mid:20250812075909.2119126-1-pranay.samala@intel.com">
<pre wrap="" class="moz-quote-pre">
static int opt_handler(int opt, int opt_index, void *_data)
{
data_t *data = _data;
@@ -1210,6 +1221,9 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
}
for (i = 0; i < ARRAY_SIZE(transition_tests); i++) {
+ if (strstr(transition_tests[i].name, "modeset"))
+ update_debug_mask_if_ci(DRM_UT_DRIVER);
+
igt_describe(transition_tests[i].desc);
igt_subtest_with_dynamic_f("%s", transition_tests[i].name) {
pipe_count = 0;
@@ -1250,6 +1264,9 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
}
for (i = 0; i < ARRAY_SIZE(modeset_tests); i++) {
+ if (count_conn_outputs(&data.display) > 2)
+ update_debug_mask_if_ci(DRM_UT_DRIVER);
+
igt_describe_f("%s", modeset_tests[i].desc);
igt_subtest_with_dynamic_f("%s", modeset_tests[i].name) {
for (j = 1; j <= count; j++) {
</pre>
</blockquote>
</body>
</html>