* [PATCH mptcp-next] selftests: mptcp: update output info of chk_rm_nr
@ 2022-02-16 2:51 Geliang Tang
2022-02-17 0:17 ` Mat Martineau
0 siblings, 1 reply; 2+ messages in thread
From: Geliang Tang @ 2022-02-16 2:51 UTC (permalink / raw)
To: mptcp; +Cc: Geliang Tang
This patch updated the output info of chk_rm_nr. Renamed 'sf' to 'rmsf',
which means 'remove subflow'. Added the display of whether the inverted
namespaces has been used to check the mib counters.
The new output looks like this:
002 remove multiple subflows syn[ ok ] - synack[ ok ] - ack[ ok ]
rm [ ok ] - rmsf [ ok ]
003 remove single address syn[ ok ] - synack[ ok ] - ack[ ok ]
add[ ok ] - echo [ ok ]
rm [ ok ] - rmsf [ ok ] invert
Signed-off-by: Geliang Tang <geliang.tang@suse.com>
---
tools/testing/selftests/net/mptcp/mptcp_join.sh | 17 +++++++++--------
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/tools/testing/selftests/net/mptcp/mptcp_join.sh
index c664565aa8b1..37b3d00b2dcb 100755
--- a/tools/testing/selftests/net/mptcp/mptcp_join.sh
+++ b/tools/testing/selftests/net/mptcp/mptcp_join.sh
@@ -1242,15 +1242,14 @@ chk_rm_nr()
local invert=${3:-""}
local count
local dump_stats
- local addr_ns
- local subflow_ns
+ local addr_ns=$ns1
+ local subflow_ns=$ns2
+ local extra_msg=""
- if [ -z $invert ]; then
- addr_ns=$ns1
- subflow_ns=$ns2
- elif [ $invert = "invert" ]; then
+ if [[ $invert = "invert" ]]; then
addr_ns=$ns2
subflow_ns=$ns1
+ extra_msg=" invert"
fi
printf "%-${nr_blank}s %s" " " "rm "
@@ -1264,7 +1263,7 @@ chk_rm_nr()
echo -n "[ ok ]"
fi
- echo -n " - sf "
+ echo -n " - rmsf "
count=`ip netns exec $subflow_ns nstat -as | grep MPTcpExtRmSubflow | awk '{print $2}'`
[ -z "$count" ] && count=0
if [ "$count" != "$rm_subflow_nr" ]; then
@@ -1272,9 +1271,11 @@ chk_rm_nr()
ret=1
dump_stats=1
else
- echo "[ ok ]"
+ echo -n "[ ok ]"
fi
+ echo "$extra_msg"
+
[ "${dump_stats}" = 1 ] && dump_stats
}
--
2.34.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH mptcp-next] selftests: mptcp: update output info of chk_rm_nr
2022-02-16 2:51 [PATCH mptcp-next] selftests: mptcp: update output info of chk_rm_nr Geliang Tang
@ 2022-02-17 0:17 ` Mat Martineau
0 siblings, 0 replies; 2+ messages in thread
From: Mat Martineau @ 2022-02-17 0:17 UTC (permalink / raw)
To: Geliang Tang; +Cc: mptcp
On Wed, 16 Feb 2022, Geliang Tang wrote:
> This patch updated the output info of chk_rm_nr. Renamed 'sf' to 'rmsf',
> which means 'remove subflow'. Added the display of whether the inverted
> namespaces has been used to check the mib counters.
>
> The new output looks like this:
>
> 002 remove multiple subflows syn[ ok ] - synack[ ok ] - ack[ ok ]
> rm [ ok ] - rmsf [ ok ]
> 003 remove single address syn[ ok ] - synack[ ok ] - ack[ ok ]
> add[ ok ] - echo [ ok ]
> rm [ ok ] - rmsf [ ok ] invert
>
> Signed-off-by: Geliang Tang <geliang.tang@suse.com>
> ---
> tools/testing/selftests/net/mptcp/mptcp_join.sh | 17 +++++++++--------
> 1 file changed, 9 insertions(+), 8 deletions(-)
>
> diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/tools/testing/selftests/net/mptcp/mptcp_join.sh
> index c664565aa8b1..37b3d00b2dcb 100755
> --- a/tools/testing/selftests/net/mptcp/mptcp_join.sh
> +++ b/tools/testing/selftests/net/mptcp/mptcp_join.sh
> @@ -1242,15 +1242,14 @@ chk_rm_nr()
> local invert=${3:-""}
> local count
> local dump_stats
> - local addr_ns
> - local subflow_ns
> + local addr_ns=$ns1
> + local subflow_ns=$ns2
> + local extra_msg=""
>
> - if [ -z $invert ]; then
> - addr_ns=$ns1
> - subflow_ns=$ns2
> - elif [ $invert = "invert" ]; then
> + if [[ $invert = "invert" ]]; then
> addr_ns=$ns2
> subflow_ns=$ns1
> + extra_msg=" invert"
> fi
>
> printf "%-${nr_blank}s %s" " " "rm "
> @@ -1264,7 +1263,7 @@ chk_rm_nr()
> echo -n "[ ok ]"
> fi
>
> - echo -n " - sf "
> + echo -n " - rmsf "
> count=`ip netns exec $subflow_ns nstat -as | grep MPTcpExtRmSubflow | awk '{print $2}'`
> [ -z "$count" ] && count=0
> if [ "$count" != "$rm_subflow_nr" ]; then
> @@ -1272,9 +1271,11 @@ chk_rm_nr()
> ret=1
> dump_stats=1
> else
> - echo "[ ok ]"
> + echo -n "[ ok ]"
> fi
>
> + echo "$extra_msg"
If the 'if' statement above went down the [fail] path, this will output
either an extra empty line or " invert" on its own line. But the output
formatting gets messy with failures anyway so it's not a big deal (also
happens in chk_rst_nr()), and it's still clear where the failure was.
Reviewed by: Mat Martineau <mathew.j.martineau@linux.intel.com>
> +
> [ "${dump_stats}" = 1 ] && dump_stats
> }
>
> --
> 2.34.1
>
>
>
--
Mat Martineau
Intel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-02-17 0:17 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-16 2:51 [PATCH mptcp-next] selftests: mptcp: update output info of chk_rm_nr Geliang Tang
2022-02-17 0:17 ` Mat Martineau
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.