<!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/5/2025 2:11 PM, Pranay Samala
wrote:<br>
</div>
<blockquote type="cite" cite="mid:20250805084131.1512658-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
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 | 20 ++++++++++++++++++++
1 file changed, 20 insertions(+)
diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
index 8c6094987..2bb2cc63b 100644
--- a/tests/kms_atomic_transition.c
+++ b/tests/kms_atomic_transition.c
@@ -1096,6 +1096,19 @@ 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;
+ igt_output_t *output;
+
+ for (int i = 0; i < display->n_outputs; i++) {
+ output = &display->outputs[i];
+ if (igt_output_is_connected(output))
+ conn_outputs++;</pre>
</blockquote>
<p>Use direct access to display->outputs[i] instead of a
temporary variable.</p>
<pre wrap="" class="moz-quote-pre">if (igt_output_is_connected(&display->outputs[i])</pre>
<blockquote type="cite" cite="mid:20250805084131.1512658-1-pranay.samala@intel.com">
<pre wrap="" class="moz-quote-pre">
+ }
+ return conn_outputs;
+}
+
static int opt_handler(int opt, int opt_index, void *_data)
{
data_t *data = _data;
@@ -1210,6 +1223,10 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
}
for (i = 0; i < ARRAY_SIZE(transition_tests); i++) {
+ if (strcmp(transition_tests[i].name, "plane-all-modeset-transition") == 0 ||
+ strcmp(transition_tests[i].name, "plane-all-modeset-transition-fencing") == 0)
+ update_debug_mask_if_ci(DRM_UT_DRIVER);</pre>
</blockquote>
imho, All modeset-related tests or on more display config should
reduce the log level, not just the two explicitly listed. change if
condition as below:<br>
<span>
<div class="scriptor-paragraph"><span>if</span><span> </span><span>(</span><span>strstr</span><span>(</span><span>transition_tests</span><span>[</span><span>i</span><span>]</span><span>.name</span><span>,</span><span>
</span><span>"modeset"</span><span>)</span><span> </span><span>|</span><span>|</span></div>
<div class="scriptor-paragraph"><span> </span><span>count_conn_outputs</span><span>(</span><span>&</span><span>data</span><span>.display</span><span>)</span><span>
</span><span>></span><span> </span><span>2</span><span>)</span></div>
</span><br>
<blockquote type="cite" cite="mid:20250805084131.1512658-1-pranay.samala@intel.com">
<pre wrap="" class="moz-quote-pre">
+
igt_describe(transition_tests[i].desc);
igt_subtest_with_dynamic_f("%s", transition_tests[i].name) {
pipe_count = 0;
@@ -1250,6 +1267,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>