[PATCH i-g-t v2] tests/kms_atomic_transition: Change debug mask for multi-output modeset tests

[Naladala, Ramanaidu]

Hi Pranay,

On 8/12/2025 1:29 PM, Pranay Samala wrote:
> 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<pranay.samala at intel.com>
> ---
>   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;
> +}
> +

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.

Change the function name to igt_get_connected_output_count(igt_display_t 
*display)

>   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++) {
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/igt-dev/attachments/20250818/905d3746/attachment-0001.htm>