[LTP] [PATCH 3/3] net/icmp: Unify log info messages

Alexey Kodanev alexey.kodanev@oracle.com
Mon Jun 18 16:29:17 CEST 2018


On 06/14/2018 04:15 PM, Petr Vorel wrote:
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
> New in v2.
> ---

Hi Petr,

>  testcases/network/stress/icmp/icmp-uni-basic.sh | 5 +----
>  testcases/network/stress/icmp/icmp-uni-vti.sh   | 2 +-
>  2 files changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/testcases/network/stress/icmp/icmp-uni-basic.sh b/testcases/network/stress/icmp/icmp-uni-basic.sh
> index fc84899dd..79581b46c 100755
> --- a/testcases/network/stress/icmp/icmp-uni-basic.sh
> +++ b/testcases/network/stress/icmp/icmp-uni-basic.sh
> @@ -25,10 +25,7 @@ TST_CLEANUP="tst_ipsec_cleanup"
>  
>  . ipsec_lib.sh
>  
> -# Test description
> -tst_resm TINFO "Sending ICMP messages with the following conditions"
> -tst_resm TINFO "- Version of IP is IPv${TST_IPV6:-4}"
> -tst_resm TINFO "- Size of packets are ( $IPSEC_SIZE_ARRAY )"
> +tst_resm TINFO "Sending ICMP messages">  
>  if [ -n "$IPSEC_MODE" -a -n "$IPSEC_PROTO" ]; then
>  	case $IPSEC_PROTO in

Probably better to fix the similar issue here with printing IPsec
protocol/mode type too, i.e. remove the above if block and print the
message in ipsec_lib.sh/tst_ipsec() instead, so it would be the same
as in tst_ipsec_setup_vti() now:

    tst_resm TINFO "Test vti$TST_IPV6 + IPsec[$IPSEC_PROTO/$IPSEC_MODE]"

> diff --git a/testcases/network/stress/icmp/icmp-uni-vti.sh b/testcases/network/stress/icmp/icmp-uni-vti.sh
> index 7fa211d7f..ea8b5c828 100755
> --- a/testcases/network/stress/icmp/icmp-uni-vti.sh
> +++ b/testcases/network/stress/icmp/icmp-uni-vti.sh
> @@ -26,7 +26,7 @@ do_test()
>  {
>  	PING_MAX="$IPSEC_REQUESTS"
>  
> -	tst_resm TINFO "Sending ICMP messages..."
> +	tst_resm TINFO "Sending ICMP messages"
>  	tst_ping $tst_vti $ip_rmt_tun $IPSEC_SIZE_ARRAY
>  }
>  

The rest looks fine to me.

Thanks,
Alexey


More information about the ltp mailing list