[igt-dev] [PATCH i-g-t 1/2] runner: Actually ping watchdogs every interval

Chris Wilson chris at chris-wilson.co.uk
Tue Dec 3 11:39:20 UTC 2019


Quoting Petri Latvala (2019-12-03 10:44:56)
> The split to timeout intervals was made to accomodate for watchdogs
> that cannot use a timeout as high as we wanted. Actually using that
> feature requires us to ping the watchdog every interval even though we
> handle actual timeouting after all intervals are used up.
> 
> Signed-off-by: Petri Latvala <petri.latvala at intel.com>
> Cc: Arkadiusz Hiler <arkadiusz.hiler at intel.com>

Doesn't look harmful. The implication is that we ping the watchdog for
some cases where we did not previously, but for as long as the runner
remains in controls, that should not be an issue.

Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris


More information about the igt-dev mailing list