* [LTP] [PATCH v3 4/4] lib: reset SIGTERM to original disposition if timeout occured
@ 2021-05-08 5:51 Li Wang
2021-05-10 11:57 ` Cyril Hrubis
0 siblings, 1 reply; 4+ messages in thread
From: Li Wang @ 2021-05-08 5:51 UTC (permalink / raw)
To: ltp
To limit the "kill $sleep_pid; exit" action only effect on the
sleep proccess will make code easy to understand.
Also, to get rid of below redundant behavior:
./../../../testcases/lib/tst_test.sh: line 1: kill: (242089) - No such process
Signed-off-by: Li Wang <liwang@redhat.com>
---
testcases/lib/tst_test.sh | 1 +
1 file changed, 1 insertion(+)
diff --git a/testcases/lib/tst_test.sh b/testcases/lib/tst_test.sh
index b6ca0cb26..28c2052d6 100644
--- a/testcases/lib/tst_test.sh
+++ b/testcases/lib/tst_test.sh
@@ -472,6 +472,7 @@ _tst_timeout_process()
sleep_pid=$!
trap "kill $sleep_pid; exit" TERM
wait $sleep_pid
+ trap - TERM
_tst_kill_test
}
--
2.30.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [LTP] [PATCH v3 4/4] lib: reset SIGTERM to original disposition if timeout occured
2021-05-08 5:51 [LTP] [PATCH v3 4/4] lib: reset SIGTERM to original disposition if timeout occured Li Wang
@ 2021-05-10 11:57 ` Cyril Hrubis
2021-05-10 12:45 ` Li Wang
0 siblings, 1 reply; 4+ messages in thread
From: Cyril Hrubis @ 2021-05-10 11:57 UTC (permalink / raw)
To: ltp
Hi!
Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
The whole series looks good to me. I'm ok with merging these changes
beofre the release.
--
Cyril Hrubis
chrubis@suse.cz
^ permalink raw reply [flat|nested] 4+ messages in thread
* [LTP] [PATCH v3 4/4] lib: reset SIGTERM to original disposition if timeout occured
2021-05-10 11:57 ` Cyril Hrubis
@ 2021-05-10 12:45 ` Li Wang
2021-05-10 14:38 ` Petr Vorel
0 siblings, 1 reply; 4+ messages in thread
From: Li Wang @ 2021-05-10 12:45 UTC (permalink / raw)
To: ltp
On Mon, May 10, 2021 at 8:23 PM Cyril Hrubis <chrubis@suse.cz> wrote:
>
> Hi!
> Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
>
> The whole series looks good to me. I'm ok with merging these changes
> beofre the release.
Thanks for the review! Patchset tweaked with your suggestions and merged.
--
Regards,
Li Wang
^ permalink raw reply [flat|nested] 4+ messages in thread
* [LTP] [PATCH v3 4/4] lib: reset SIGTERM to original disposition if timeout occured
2021-05-10 12:45 ` Li Wang
@ 2021-05-10 14:38 ` Petr Vorel
0 siblings, 0 replies; 4+ messages in thread
From: Petr Vorel @ 2021-05-10 14:38 UTC (permalink / raw)
To: ltp
> On Mon, May 10, 2021 at 8:23 PM Cyril Hrubis <chrubis@suse.cz> wrote:
> > Hi!
> > Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
> > The whole series looks good to me. I'm ok with merging these changes
> > beofre the release.
> Thanks for the review! Patchset tweaked with your suggestions and merged.
Thanks!
Kind regards,
Petr
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-05-10 14:38 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-08 5:51 [LTP] [PATCH v3 4/4] lib: reset SIGTERM to original disposition if timeout occured Li Wang
2021-05-10 11:57 ` Cyril Hrubis
2021-05-10 12:45 ` Li Wang
2021-05-10 14:38 ` Petr Vorel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.