[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