[igt-dev] [PATCH i-g-t 3/4] intel/kms_big_fb.c: remove linear-addfb-size*-overflow docs

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


Hi Mauro,
On 2023-10-09 at 12:01:40 +0200, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab at kernel.org>
> 
> As documented at the comment that registers the subtests:
> 
> 	/*
> 	 * Skip linear as it doesn't hit the overflow we want
> 	 * on account of the tile height being effectively one,
> 	 * and thus the kenrnel rounding up to the next tile
> 	 * height won't do anything.
> 	 */
> 
> There are no linear tests for addfb-size-overflow and
> addfb-size-offset-overflow. Remove their documentation
> as well.
> 
> Signed-off-by: Mauro Carvalho Chehab <mchehab at kernel.org>

As these are kms tests adding Bhanu and Swati to +cc.

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

> ---
>  tests/intel/kms_big_fb.c | 16 ----------------
>  1 file changed, 16 deletions(-)
> 
> diff --git a/tests/intel/kms_big_fb.c b/tests/intel/kms_big_fb.c
> index 611e60896c8e..2c7b24fcab77 100644
> --- a/tests/intel/kms_big_fb.c
> +++ b/tests/intel/kms_big_fb.c
> @@ -736,14 +736,6 @@ static void test_scanout(data_t *data)
>  }
>  
>  /**
> - * SUBTEST: linear-addfb-size-overflow
> - * Description: Sanity check if addfb ioctl fails correctly for (Linear) modifier
> - *              with small bo.
> - * Driver requirement: i915, xe
> - * Functionality: big_fbs, kms_gem_interop
> - * Mega feature: General Display Features
> - * Test category: functionality test
> - *
>   * SUBTEST: %s-addfb-size-overflow
>   * Description: Sanity check if addfb ioctl fails correctly for (%arg[1]) modifier
>   *              with small bo.
> @@ -804,14 +796,6 @@ test_size_overflow(data_t *data)
>  }
>  
>  /**
> - * SUBTEST: linear-addfb-size-offset-overflow
> - * Description: Sanity check if addfb ioctl fails correctly for (Linear) modifier
> - *              and offsets with small bo
> - * Driver requirement: i915, xe
> - * Functionality: big_fbs, kms_gem_interop
> - * Mega feature: General Display Features
> - * Test category: functionality test
> - *
>   * SUBTEST: %s-addfb-size-offset-overflow
>   * Description: Sanity check if addfb ioctl fails correctly for (%arg[1]) modifier
>   *              and offsets with small bo
> -- 
> 2.41.0
> 


More information about the igt-dev mailing list