[igt-dev] [PATCH i-g-t 1/3] i915/gem_ctx_persistence: Use "%u" for -1u conversion

Juha-Pekka Heikkila juhapekka.heikkila at gmail.com
Wed May 20 09:34:35 UTC 2020


Series

Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>


On 19.5.2020 9.34, Chris Wilson wrote:
> The debugfs modparams are more picky and refuse to do the implicit
> unsigned conversion.
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---
>   tests/i915/gem_ctx_persistence.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/i915/gem_ctx_persistence.c b/tests/i915/gem_ctx_persistence.c
> index ce9f02350..4a1ef0f27 100644
> --- a/tests/i915/gem_ctx_persistence.c
> +++ b/tests/i915/gem_ctx_persistence.c
> @@ -1127,7 +1127,7 @@ igt_main
>   		igt_require_gem(i915);
>   
>   		/* Restore the reset modparam if left clobbered */
> -		igt_assert(igt_params_set(i915, "reset", "%d", -1));
> +		igt_assert(igt_params_set(i915, "reset", "%u", -1));
>   
>   		enable_hangcheck(i915);
>   		igt_install_exit_handler(exit_handler);
> 



More information about the igt-dev mailing list