[igt-dev] [PATCH i-g-t v2 2/2] runner: Don't wait forever for processes to die

[Chris Wilson]

Quoting Petri Latvala (2019-12-03 11:15:37)
> While the originally written timeout for process killing (2 seconds)
> was way too short, waiting indefinitely is suboptimal as well. We're
> seeing cases where the test is stuck for possibly hours in
> uninterruptible sleep (IO). Wait a fairly longer selected time period
> of 2 minutes, because even making progress for that long means the
> machine is in bad enough state to require a good kicking and booting.
> 
> v2:
>  - Abort quicker if kernel is tainted (Chris)
>  - Correctly convert process-exists check with kill() to process-does-not-exist
> 
> Signed-off-by: Petri Latvala <petri.latvala at intel.com>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Arkadiusz Hiler <arkadiusz.hiler at intel.com>

Reads like it does what you say on the tin.
Reviewed-by Cc: Chris Wilson <chris at chris-wilson.co.uk>
-Chris