[igt-dev] [PATCH i-g-t] i915/gem_exec_suspend: Fix cached/uncached test names

Petri Latvala petri.latvala at intel.com
Tue Jan 12 06:41:23 UTC 2021


On Mon, Jan 11, 2021 at 08:57:26PM -0500, Ashutosh Dixit wrote:
> 9e7c581519 ("i915/gem_exec_suspend: Remove legacy ring abi") botched up
> cached/uncached test names. Fix these:
> 
> Before:
> 
> uncached-
> cached-
> uncached--S3
> cached--S3
> uncached--S4
> cached--S4
> 
> After:
> 
> uncached
> cached
> uncached-S3
> cached-S3
> uncached-S4
> cached-S4
> 
> Fixes: 9e7c581519 ("i915/gem_exec_suspend: Remove legacy ring abi")
> Signed-off-by: Ashutosh Dixit <ashutosh.dixit at intel.com>
> ---
>  tests/i915/gem_exec_suspend.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
> index a31dd6625..b6558a144 100644
> --- a/tests/i915/gem_exec_suspend.c
> +++ b/tests/i915/gem_exec_suspend.c
> @@ -297,7 +297,7 @@ igt_main
>  		run_test(fd, ALL_ENGINES, HIBERNATE);
>  
>  	for (m = modes; m->suffix; m++) {
> -		igt_subtest_with_dynamic_f("uncached-%s", m->suffix) {
> +		igt_subtest_with_dynamic_f("uncached%s", m->suffix) {
>  			__for_each_physical_engine(fd, e) {
>  				if (!gem_class_can_store_dword(fd, e->class))
>  					continue;
> @@ -306,7 +306,7 @@ igt_main
>  			}
>  		}
>  
> -		igt_subtest_with_dynamic_f("cached-%s", m->suffix) {
> +		igt_subtest_with_dynamic_f("cached%s", m->suffix) {
>  			__for_each_physical_engine(fd, e) {
>  				if (!gem_class_can_store_dword(fd, e->class))
>  					continue;


Reviewed-by: Petri Latvala <petri.latvala at intel.com>


More information about the igt-dev mailing list