Re: [igt-dev] [PATCH] intel-ci: Drop kms_setmode@basic-clone-single-crtc from BAT

[Petri Latvala <petri.latvala@intel.com>]

On Tue, Apr 12, 2022 at 04:16:16PM +0530, Bhanuprakash Modem wrote:
> Remove subtest "kms_setmode@basic-clone-single-crtc" from the
> BAT test-list as most BAT systems are having single display
> only and this subtest requires a multi-display configuration.
> 
> Cc: Petri Latvala <petri.latvala@intel.com>
> Cc: Karthik B S <karthik.b.s@intel.com>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>

True, there's only fi-snb-2600 that has non-skips on this test. But
wouldn't it be better to request more multi-display configs in the BAT
farm?

From what I can see, removing this test from fast-feedback gains under
2 seconds of runtime on said fi-snb-2600, 0.001s on the other
platforms. What's the real gain here?


-- 
Petri Latvala


> ---
>  tests/intel-ci/fast-feedback.testlist | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> index f6329708..0f980319 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -133,7 +133,6 @@ igt@kms_psr@primary_page_flip
>  igt@kms_psr@cursor_plane_move
>  igt@kms_psr@sprite_plane_onoff
>  igt@kms_psr@primary_mmap_gtt
> -igt@kms_setmode@basic-clone-single-crtc
>  igt@i915_pm_backlight@basic-brightness
>  igt@i915_pm_rpm@basic-pci-d3-state
>  igt@i915_pm_rpm@basic-rte
> -- 
> 2.35.1
>