[PATCH] drm/i915/selftests: Use preemption timeout on cleanup

[Andi Shyti]

Hi Janusz,

...

>  	for_each_gt(gt, i915, i) {
> +		struct intel_engine_cs *engine;
> +		unsigned long timeout_ms = 0;
> +		unsigned int id;
> +
>  		if (intel_gt_is_wedged(gt))
>  			ret = -EIO;
>  
> +		for_each_engine(engine, gt, id) {
> +			if (engine->props.preempt_timeout_ms > timeout_ms)
> +				timeout_ms = engine->props.preempt_timeout_ms;
> +		}


the brackets are not really required here.

> +
>  		cond_resched();
>  
> -		if (intel_gt_wait_for_idle(gt, HZ * 3) == -ETIME) {
> +		if (intel_gt_wait_for_idle(gt, HZ * timeout_ms / 500) == -ETIME) {

where is this 500 coming from?

Thanks,
Andi

>  			pr_err("%pS timed out, cancelling all further testing.\n",
>  			       __builtin_return_address(0));
>  
> -- 
> 2.47.1