[LTP] [PATCH v4 1/2] net: Add tst_net_run helper
Petr Vorel
pvorel@suse.cz
Mon Nov 19 18:38:57 CET 2018
Hi Alexey,
> > tst_rhost_run with -s option in cleanup is only in 2 files:
> > testcases/network/mpls/mpls_lib.sh
> > testcases/network/tcp_cmds/sendfile/sendfile01
> > Maybe that's a mistake (unlike C functions where error in SAFE_* continue with
> > warning, in shell it exits during cleanup).
> For tst_test.sh we could add something like this to tst_brk()
> iff --git a/testcases/lib/tst_test.sh b/testcases/lib/tst_test.sh
> index 16081fa..e84cf7f 100644
> --- a/testcases/lib/tst_test.sh
> +++ b/testcases/lib/tst_test.sh
> @@ -41,6 +41,7 @@ trap "tst_brk TBROK 'test interrupted'" INT
> _tst_do_exit()
> {
> local ret=0
> + TST_DO_EXIT=1
> if [ -n "$TST_SETUP_STARTED" -a -n "$TST_CLEANUP" -a \
> -z "$TST_NO_CLEANUP" ]; then
> @@ -123,6 +124,11 @@ tst_brk()
> local res=$1
> shift
> + if [ "$TST_DO_EXIT" = 1 ]; then
> + tst_res TWARN "$@"
> + return
> + fi
> +
> tst_res "$res" "$@"
> _tst_do_exit
> }
Good idea! Could you please send a patch?
Kind regards,
Petr
More information about the ltp
mailing list