Re: [PATCH i-g-t v1 1/2] tests/kms_plane_scaling: Reduce debug log level in CI for scaling tests

["Ville Syrjälä" <ville.syrjala@linux.intel.com>]

On Thu, Nov 06, 2025 at 03:09:08PM +0200, Jani Nikula wrote:
> On Thu, 06 Nov 2025, Naladala Ramanaidu <ramanaidu.naladala@intel.com> wrote:
> > Reduce excessive logging in CI environments when multiple outputs
> > are connected. It prevents CI runners from exceeding disk usage
> > limits.
> 
> So how do we debug issues when there are no KMD debug logs? This can't
> be the go-to solution in the long run.

Yeah, this stuff is already causing problems. Eg.
igt@kms_atomic_transition@plane-toggle-modeset-transition keeps
failing randomly on tgl, but since the debugs are disabled the 
logs are completely useless.

I've thrown https://patchwork.freedesktop.org/series/155702 a few
times at trybot to get more logs, but since it's random no luck so 
far. If we didn't have this "disable debugs" crap in the first
place the bug would probably have been solved as soon as it
appeared.

> 
> Moreover, update_debug_mask_if_ci() doesn't even log the actual mask
> being switched to.
> 
> 
> BR,
> Jani.
> 
> 
> >
> > Signed-off-by: Naladala Ramanaidu <ramanaidu.naladala@intel.com>
> > ---
> >  tests/kms_plane_scaling.c | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/tests/kms_plane_scaling.c b/tests/kms_plane_scaling.c
> > index 63d35d76b..34c8f8a64 100644
> > --- a/tests/kms_plane_scaling.c
> > +++ b/tests/kms_plane_scaling.c
> > @@ -32,6 +32,7 @@
> >  
> >  #include "igt.h"
> >  #include "igt_vec.h"
> > +#include "igt_sysfs.h"
> >  #include <math.h>
> >  
> >  /**
> > @@ -1358,11 +1359,15 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
> >  		data.devid = is_intel_device(data.drm_fd) ?
> >  			intel_get_drm_devid(data.drm_fd) : 0;
> >  		igt_require(data.display.is_atomic);
> > +		igt_install_exit_handler(igt_drm_debug_mask_reset_exit_handler);
> >  	}
> >  
> >  	igt_subtest_group {
> >  		igt_output_t *output;
> >  
> > +		if (igt_get_connected_output_count(&data.display) > 1)
> > +			update_debug_mask_if_ci(DRM_UT_DRIVER | DRM_UT_CORE);
> > +
> >  		for (int index = 0; index < ARRAY_SIZE(scaler_with_pixel_format_tests); index++) {
> >  			igt_describe(scaler_with_pixel_format_tests[index].describe);
> >  			igt_subtest_with_dynamic(scaler_with_pixel_format_tests[index].name) {
> 
> -- 
> Jani Nikula, Intel

-- 
Ville Syrjälä
Intel