[PATCH i-g-t v3 1/2] tests/kms_atomic_transition: Update debug mask for multi-output modeset tests

[Naladala, Ramanaidu]

Hi Pranay,

On 8/18/2025 9:46 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)
> v3: Move count_conn_outputs() to lib for future use (Rama)
Minor suggestion: Add ( . ) on end of sentence. This you can fix while 
merge.
>
> Signed-off-by: Pranay Samala <pranay.samala at intel.com>
> ---
>   tests/kms_atomic_transition.c | 6 ++++++
>   1 file changed, 6 insertions(+)
>
> diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
> index 8c6094987..a13830965 100644
> --- a/tests/kms_atomic_transition.c
> +++ b/tests/kms_atomic_transition.c
> @@ -1210,6 +1210,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 +1253,9 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
>   	}
>   
>   	for (i = 0; i < ARRAY_SIZE(modeset_tests); i++) {
> +		if (igt_get_connected_output_count(&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++) {
LGTM,
Reviewed-by: Naladala Ramanaidu <ramanaidu.naladala at intel.com>