<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 2, 2021 at 6:02 AM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Also timeout requires to run a test cleanup (e.g. zram01.sh).<br>
Thus send first SIGINT, but keep also SIGKILL for safety reasons<br>
(after 5 sec to give some time to the cleanup function and<br>
_tst_check_security_modules()).<br>
<br>
Signed-off-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
---<br>
Originally posted in <br>
<a href="https://patchwork.ozlabs.org/project/ltp/patch/20210202101942.31328-1-pvorel@suse.cz/" rel="noreferrer" target="_blank">https://patchwork.ozlabs.org/project/ltp/patch/20210202101942.31328-1-pvorel@suse.cz/</a><br>
<br>
* renamed function<br>
* use signal names instead of numbers in kill parameters<br>
<br>
 testcases/lib/tst_test.sh | 13 ++++++++++---<br>
 1 file changed, 10 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/testcases/lib/tst_test.sh b/testcases/lib/tst_test.sh<br>
index 58056e28b..097f672a1 100644<br>
--- a/testcases/lib/tst_test.sh<br>
+++ b/testcases/lib/tst_test.sh<br>
@@ -21,7 +21,7 @@ export TST_LIB_LOADED=1<br>
 . tst_security.sh<br>
<br>
 # default trap function<br>
-trap "tst_brk TBROK 'test interrupted'" INT<br>
+trap "tst_brk TBROK 'test interrupted or timed out'" INT<br>
<br>
 _tst_cleanup_timer()<br>
 {<br>
@@ -442,6 +442,14 @@ _tst_multiply_timeout()<br>
        return 0<br>
 }<br>
<br>
+_tst_run_timer()<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Hmm, this name is not good than before, or rename to _tst_kill_timer_pid(), _tst_stop_timer()?</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
+{<br>
+       tst_res TBROK "test killed, timeout! If you are running on slow machine, try exporting LTP_TIMEOUT_MUL > 1"<br>
+       kill -INT -$pid<br>
+       sleep 5<br>
+       kill -KILL -$pid<br>
+}<br>
+<br>
 _tst_setup_timer()<br>
 {<br>
        TST_TIMEOUT=${TST_TIMEOUT:-300}<br>
@@ -465,8 +473,7 @@ _tst_setup_timer()<br>
        tst_res TINFO "timeout per run is ${h}h ${m}m ${s}s"<br>
<br>
        _tst_cleanup_timer<br>
-<br>
-       sleep $sec && tst_res TBROK "test killed, timeout! If you are running on slow machine, try exporting LTP_TIMEOUT_MUL > 1" && kill -9 -$pid &<br>
+       sleep $sec && _tst_run_timer &<br>
<br>
        _tst_setup_timer_pid=$!<br>
 }<br>
-- <br>
2.30.1<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>