[igt-dev] [PATCH i-g-t 4/4] intel/kms_frontbuffer_tracking.c: remove fbc-slowdraw test

Kamil Konieczny kamil.konieczny at linux.intel.com
Mon Oct 9 10:30:51 UTC 2023


Hi Mauro,

please correct subject:

intel/kms_frontbuffer_tracking.c: remove fbc-slowdraw test
----------------------------------------------------- ^^^^
s/test/docs/

I will also add Bhanu and Swati to cc.

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

On 2023-10-09 at 12:01:41 +0200, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab at kernel.org>
> 
> The slowdraw test is defined to run only for two features:
> 
> 	if ((t.feature & FEATURE_PSR) || (t.feature & FEATURE_DRRS))
> 		igt_subtest_f("%s-slowdraw", feature_str(t.feature)) {
> 			igt_require(igt_draw_supports_method(drm.fd, t.method));
> 			slow_draw_subtest(&t);
> 		}
> 
> As it won't be reported for FBC, remove documentation for it.
> 
> Signed-off-by: Mauro Carvalho Chehab <mchehab at kernel.org>
> ---
>  tests/intel/kms_frontbuffer_tracking.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/tests/intel/kms_frontbuffer_tracking.c b/tests/intel/kms_frontbuffer_tracking.c
> index 6526e1c3cffa..f90d09f9f06d 100644
> --- a/tests/intel/kms_frontbuffer_tracking.c
> +++ b/tests/intel/kms_frontbuffer_tracking.c
> @@ -3025,13 +3025,6 @@ static bool tiling_is_valid(int feature_flags, enum tiling_type tiling)
>   * Mega feature: General Display Features
>   * Test category: functionality test
>   *
> - * SUBTEST: fbc-slowdraw
> - * Description: Sleep a little bit between drawing operations with FBC
> - * Driver requirement: i915, xe
> - * Functionality: fbc, fbt, kms_core
> - * Mega feature: General Display Features
> - * Test category: functionality test
> - *
>   * SUBTEST: psr-slowdraw
>   * Description: Sleep a little bit between drawing operations with PSR
>   * Driver requirement: i915, xe
> -- 
> 2.41.0
> 


More information about the igt-dev mailing list