[igt-dev] [PATCH i-g-t v4 10/11] i915_test_config.json: exclude gem_concurrent_blit.c

Kamil Konieczny kamil.konieczny at linux.intel.com
Thu May 25 21:06:44 UTC 2023


Hi Mauro,

On 2023-05-25 at 11:05:34 +0200, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab at kernel.org>
> 
> This is actually a subset of gem_concurrent_all.c, which can't
> be described, as it containd 400k+ tests on it. It also has an
> insane number of tests:
> 
> 	$ ./build/tests/gem_concurrent_all --list |wc -l
> 	428400
> 	$ ./build/tests/gem_concurrent_blit --list |wc -l
> 	57120
> 
> It makes no sense to document all of them. So, exclude it as
> well, as otherwise adding a logic to check for missing documentation
> on i915 will complain with:
> 
> 	...
> 	Warning: Missing documentation for igt at gem_concurrent_blit@4KiB-tiny-wc-blt-read-read-bcs-forked
> 	Warning: Missing documentation for igt at gem_concurrent_blit@4KiB-tiny-wc-blt-read-read-bcs-interruptible
> 	Warning: Missing documentation for igt at gem_concurrent_blit@4KiB-tiny-wc-blt-read-read-rcs
> 	...
> 
> Signed-off-by: Mauro Carvalho Chehab <mchehab at kernel.org>
> ---
>  tests/i915/i915_test_config.json | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/i915/i915_test_config.json b/tests/i915/i915_test_config.json
> index 812cb73b9f52..ae8317b6aefe 100644
> --- a/tests/i915/i915_test_config.json
> +++ b/tests/i915/i915_test_config.json
> @@ -11,7 +11,7 @@
>                 "../prime_vgem.c", "../sw_sync.c", "../syncobj_basic.c",
>                 "../syncobj_timeline.c", "../syncobj_wait.c",
>                 "../vgem_basic.c", "../vgem_slow.c" ],
> -    "exclude_files": [ "kms_*", "gem_concurrent_all.c" ],
> +    "exclude_files": [ "kms_*", "gem_concurrent_all.c", "gem_concurrent_blit.c" ],

Looks good,

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

--
Kamil

>      "fields": {
>  	"Mega feature": {
>                  "_properties_": {
> -- 
> 2.40.1
> 


More information about the igt-dev mailing list