[igt-dev] [PATCH i-g-t 1/3] tests/gem_sync: Change duplicate subtest names

Zbigniew Kempczyński zbigniew.kempczynski at intel.com
Thu Oct 28 10:25:15 UTC 2021


On Thu, Oct 28, 2021 at 12:18:17PM +0200, Kamil Konieczny wrote:
> For legacy rings add "legacy-" prefix to subtest names to avoid
> duplicates.

For me this prefix is ok:

Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>

--
Zbigniew

> 
> Signed-off-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> ---
>  tests/i915/gem_sync.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/i915/gem_sync.c b/tests/i915/gem_sync.c
> index 6cb00c40..0093ca25 100644
> --- a/tests/i915/gem_sync.c
> +++ b/tests/i915/gem_sync.c
> @@ -1205,7 +1205,7 @@ igt_main
>  
>  	/* Legacy for selecting rings. */
>  	for_each_test(t, individual) {
> -		igt_subtest_with_dynamic_f("%s", t->name) {
> +		igt_subtest_with_dynamic_f("legacy-%s", t->name) {
>  			for (const struct intel_execution_ring *l = intel_execution_rings; l->name; l++) {
>  				igt_dynamic_f("%s", l->name) {
>  					t->func(fd, intel_ctx_0(fd), eb_ring(l),
> -- 
> 2.30.2
> 


More information about the igt-dev mailing list