[PATCH] tests/kms_cursor_legacy: reduce CI debug output

[PATCH] tests/kms_cursor_legacy: reduce CI debug output

[Jeevan B]

Reduce CI output to avoid exceeding the runner's per-test limit.

Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/work_items/16960
Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/work_items/16961
Signed-off-by: Jeevan B <jeevan.b@intel.com>
---
 tests/kms_cursor_legacy.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
index 62339855e..9c6b63cbd 100644
--- a/tests/kms_cursor_legacy.c
+++ b/tests/kms_cursor_legacy.c
@@ -1888,7 +1888,8 @@ int igt_main()
 	};
 
 	igt_fixture() {
-		unsigned int debug_mask_if_ci = DRM_UT_KMS;
+		/* KMS debug logs can exceed the CI per-test output limit here. */
+		unsigned int debug_mask_if_ci = 0;
 		display.drm_fd = drm_open_driver_master(DRIVER_ANY);
 		kmstest_set_vt_graphics_mode();
 
-- 
2.43.0

RE: [PATCH] tests/kms_cursor_legacy: reduce CI debug output

[Samala, Pranay]

Hi Jeevan,

> -----Original Message-----
> From: igt-dev <igt-dev-bounces@lists.freedesktop.org> On Behalf Of Jeevan
> B
> Sent: Monday, October 26, 2026 3:15 PM
> To: igt-dev@lists.freedesktop.org
> Cc: B, Jeevan <jeevan.b@intel.com>
> Subject: [PATCH] tests/kms_cursor_legacy: reduce CI debug output
> 
> Reduce CI output to avoid exceeding the runner's per-test limit.
> 
> Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/work_items/16960
> Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/work_items/16961
> Signed-off-by: Jeevan B <jeevan.b@intel.com>
> ---
>  tests/kms_cursor_legacy.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c index
> 62339855e..9c6b63cbd 100644
> --- a/tests/kms_cursor_legacy.c
> +++ b/tests/kms_cursor_legacy.c
> @@ -1888,7 +1888,8 @@ int igt_main()
>  	};
> 
>  	igt_fixture() {
> -		unsigned int debug_mask_if_ci = DRM_UT_KMS;
> +		/* KMS debug logs can exceed the CI per-test output limit
> here. */
> +		unsigned int debug_mask_if_ci = 0;

1. The library's update_debug_mask_if_ci() log message will be wrong. 
It will always print "reducing the DRM debug mask to 0x4". 
With this patch it sets 0, so CI logs will claim KMS debug is on 
when it isn't. Fix it in the same patch/series.

2. Please mention for which subtests (if any specific) this log reduction is applicable 
in commit message.

3. Rather than turning DRM debug logging off completely, have you tried a
lower mask first, e.g., DRM_UT_CORE (0x1) or DRM_UT_DRIVER (0x2)?

	enum drm_debug_category {
		DRM_UT_CORE   = 1 << 0,
		DRM_UT_DRIVER = 1 << 1,
		DRM_UT_KMS    = 1 << 2,

If CORE or DRIVER keeps the output under the limit, we would still have some
useful context in the logs.

Regards,
Pranay

>  		display.drm_fd = drm_open_driver_master(DRIVER_ANY);
>  		kmstest_set_vt_graphics_mode();
> 
> --
> 2.43.0

[PATCH] tests/kms_cursor_legacy: reduce CI debug output

[Jeevan B]

Use DRM_UT_CORE instead of DRM_UT_KMS in CI to reduce log volume while
retaining diagnostics. This applies to all subtests, including the
reported cursorA-vs-flipA-atomic-transitions case.

Log the actual debug mask instead of the hard-coded 0x4.
Intel CI output-limit validation is still pending.

Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/work_items/16961
Signed-off-by: Jeevan B <jeevan.b@intel.com>
---
 lib/igt_sysfs.c           | 3 ++-
 tests/kms_cursor_legacy.c | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/lib/igt_sysfs.c b/lib/igt_sysfs.c
index 9952c1c0d..873015bdc 100644
--- a/lib/igt_sysfs.c
+++ b/lib/igt_sysfs.c
@@ -537,7 +537,8 @@ void update_debug_mask_if_ci(unsigned int debug_mask_if_ci)
 	const char *ci_run = getenv("IGT_CI_RUN");
 
 	if (ci_run && ci_run[0] == '1') {
-		igt_debug("Currently under CI execution, reducing the DRM debug mask to 0x4\n");
+		igt_debug("Currently under CI execution, reducing the DRM debug mask to 0x%x\n",
+			  debug_mask_if_ci);
 		igt_drm_debug_mask_update(debug_mask_if_ci);
 	}
 }
diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
index 62339855e..5d8bcae9e 100644
--- a/tests/kms_cursor_legacy.c
+++ b/tests/kms_cursor_legacy.c
@@ -1888,7 +1888,8 @@ int igt_main()
 	};
 
 	igt_fixture() {
-		unsigned int debug_mask_if_ci = DRM_UT_KMS;
+		/* Keep CORE diagnostics without the high-volume KMS debug logs. */
+		unsigned int debug_mask_if_ci = DRM_UT_CORE;
 		display.drm_fd = drm_open_driver_master(DRIVER_ANY);
 		kmstest_set_vt_graphics_mode();
 
-- 
2.43.0