diff options
| author | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2019-07-19 15:01:52 +0300 |
|---|---|---|
| committer | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2019-07-22 13:50:53 +0300 |
| commit | f3b3f93724cdb5142ff090f894b5bc356118c19e (patch) | |
| tree | 07ac4d7ac2e026a8c90ff9ae9f1f42f292b3506c /lib | |
| parent | ee5c833550eef5424fd3546433a0ea67869d24c5 (diff) | |
runner: Make sure that we are closing watchdogs on signals
There are few short windows of opportunity when watchdogs are primed but
there is no signal handling in place, so the process may exit without
proper shutdown sequence.
This patch rearranges the existing code so that we set up the signalfd
and BLOCK the signals before setting up watchdogs and UNBLOCK only after
the watchdogs are closed properly.
If igt_runner exits due to signal, non-zero status code is returned.
v2: more error handling and minor touch ups (Simon)
Cc: Petri Latvala <petri.latvala@intel.com>
Cc: Simon Ser <simon.ser@intel.com>
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Reviewed-by: Simon Ser <simon.ser@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions
