From: Alexey Kodanev <aleksei.kodanev@bell-sw.com>
To: Petr Vorel <petr.vorel@gmail.com>, ltp@lists.linux.it
Subject: Re: [LTP] [PATCH 1/2] tst_net.sh: Fix for disabled IPv6
Date: Tue, 25 Jan 2022 11:32:17 +0300 [thread overview]
Message-ID: <29051e9c-f4c2-9840-7f84-8c9f49efba93@bell-sw.com> (raw)
In-Reply-To: <20220124220334.154003-1-petr.vorel@gmail.com>
Hi Petr,
On 25.01.2022 01:03, Petr Vorel wrote:
> From: Petr Vorel <pvorel@suse.cz>
>
> Tests failed in tst_init_iface even IPv4 only test was run.
> Allow to init interfaces at least for IPv4.
>
> Tests which use TST_IPV6=6 needs to be fixed (unless they use
> just tst_ipaddr_un()).
>
> Store result into TST_NET_{L,R}HOST_IPV6 in order to cache the result.
>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
> Changes v1->v2:
> * cache variables with $TST_NET_{L,R}HOST_IPV6 (mdoucha)
> NOTE: although I'd prefer to cache the result internally and use
> function for detection, tst_net.sh uses variables for most of the
> things, thus keep it this way.
>
> Kind regards,
> Petr
>
> testcases/lib/tst_net.sh | 72 ++++++++++++++++++++++++++++++++++------
> 1 file changed, 61 insertions(+), 11 deletions(-)
>
> diff --git a/testcases/lib/tst_net.sh b/testcases/lib/tst_net.sh
> index 4dc0fca92..d1ae3042a 100644
> --- a/testcases/lib/tst_net.sh
> +++ b/testcases/lib/tst_net.sh
> @@ -24,7 +24,9 @@ TST_IPV6_FLAG=${TST_IPV6_FLAG:-}
> tst_net_parse_args()
> {
> case $1 in
> - 6) TST_IPV6=6 TST_IPVER=6 TST_IPV6_FLAG="-6";;
> + 6) tst_net_require_ipv6
> + TST_IPV6=6 TST_IPVER=6 TST_IPV6_FLAG="-6"
> + ;;
> *) [ "$TST_PARSE_ARGS_CALLER" ] && $TST_PARSE_ARGS_CALLER "$1" "$2";;
> esac
> }
> @@ -100,6 +102,30 @@ tst_brk_()
> [ -z "$TST_USE_LEGACY_API" ] && tst_brk $@ || tst_brkm $@
> }
>
> +tst_net_detect_ipv6()
> +{
> + local type="${1:-lhost}"
> + local cmd='[ -f /proc/net/if_inet6 ]'
> +
> + if [ "$type" = "lhost" ]; then
> + $cmd
> + return $?
> + fi
> +
> + tst_rhost_run -c "$cmd"
> + return $?
> +}
> +
> +tst_net_require_ipv6()
> +{
> + local err="IPv6 not supported on:"
> + local missing
> +
> + [ "$TST_NET_LHOST_IPV6" = 1 ] || missing=" lhost"
> + [ "$TST_NET_RHOST_IPV6" = 1 ] || missing="$missing rhost"
> + [ -z "$missing" ] || tst_brk_ TCONF "$err$missing"
> +}
> +
> init_ltp_netspace()
> {
> local pid
> @@ -517,7 +543,9 @@ tst_init_iface()
> ip link set $iface down || return $?
> ip route flush dev $iface || return $?
> ip addr flush dev $iface || return $?
> - sysctl -qw net.ipv6.conf.$iface.accept_dad=0 || return $?
> + if [ "$TST_NET_LHOST_IPV6" = 1 ]; then
> + sysctl -qw net.ipv6.conf.$iface.accept_dad=0 || return $?
> + fi
> ip link set $iface up
> return $?
> fi
> @@ -529,7 +557,9 @@ tst_init_iface()
> tst_rhost_run -c "ip link set $iface down" || return $?
> tst_rhost_run -c "ip route flush dev $iface" || return $?
> tst_rhost_run -c "ip addr flush dev $iface" || return $?
> - tst_rhost_run -c "sysctl -qw net.ipv6.conf.$iface.accept_dad=0" || return $?
> + if [ "$TST_NET_RHOST_IPV6" = 1 ]; then
> + tst_rhost_run -c "sysctl -qw net.ipv6.conf.$iface.accept_dad=0" || return $?
> + fi
> tst_rhost_run -c "ip link set $iface up"
> }
>
> @@ -606,7 +636,9 @@ tst_restore_ipaddr()
> local ret=0
> local backup_tst_ipv6=$TST_IPV6
> TST_IPV6= tst_add_ipaddr $type $link_num || ret=$?
> - TST_IPV6=6 tst_add_ipaddr $type $link_num || ret=$?
> + if [ "$TST_NET_LHOST_IPV6" = 1 ]; then
> + TST_IPV6=6 tst_add_ipaddr $type $link_num || ret=$?
> + fi
> TST_IPV6=$backup_tst_ipv6
>
> return $ret
> @@ -971,8 +1003,16 @@ IPV6_RHOST="${IPV6_RHOST:-fd00:1:1:1::1/64}"
> if [ -z "$_tst_net_parse_variables" ]; then
> eval $(tst_net_ip_prefix $IPV4_LHOST || echo "exit $?")
> eval $(tst_net_ip_prefix -r $IPV4_RHOST || echo "exit $?")
> - eval $(tst_net_ip_prefix $IPV6_LHOST || echo "exit $?")
> - eval $(tst_net_ip_prefix -r $IPV6_RHOST || echo "exit $?")
> +
> + tst_net_detect_ipv6 && TST_NET_LHOST_IPV6=1
> + tst_net_detect_ipv6 rhost && TST_NET_RHOST_IPV6=1
> +
> + if [ "$TST_NET_LHOST_IPV6" = 1 ]; then
> + eval $(tst_net_ip_prefix $IPV6_LHOST || echo "exit $?")
> + fi
> + if [ "$TST_NET_RHOST_IPV6" = 1 ]; then
> + eval $(tst_net_ip_prefix -r $IPV6_RHOST || echo "exit $?")
> + fi
Do we really need to keep around and check two variables? If at least
one machine doesn't have IPv6, it's not longer necessary to setup the
other.
> fi
>
> [ -n "$TST_USE_NETNS" -a "$TST_INIT_NETNS" != "no" ] && init_ltp_netspace
> @@ -981,19 +1021,29 @@ if [ -z "$_tst_net_parse_variables" ]; then
> eval $(tst_net_iface_prefix $IPV4_LHOST || echo "exit $?")
> eval $(tst_rhost_run -c 'tst_net_iface_prefix -r '$IPV4_RHOST \
> || echo "exit $?")
> - eval $(tst_net_iface_prefix $IPV6_LHOST || echo "exit $?")
> - eval $(tst_rhost_run -c 'tst_net_iface_prefix -r '$IPV6_RHOST \
> - || echo "exit $?")
> +
> + if [ "$TST_NET_LHOST_IPV6" = 1 ]; then
> + eval $(tst_net_iface_prefix $IPV6_LHOST || echo "exit $?")
> + fi
> +
> + if [ "$TST_NET_RHOST_IPV6" = 1 ]; then
> + eval $(tst_rhost_run -c 'tst_net_iface_prefix -r '$IPV6_RHOST \
> + || echo "exit $?")
> + fi
>
> eval $(tst_net_vars $IPV4_LHOST/$IPV4_LPREFIX \
> $IPV4_RHOST/$IPV4_RPREFIX || echo "exit $?")
> - eval $(tst_net_vars $IPV6_LHOST/$IPV6_LPREFIX \
> - $IPV6_RHOST/$IPV6_RPREFIX || echo "exit $?")
> +
> + if [ "$TST_NET_LHOST_IPV6" = 1 ]; then
> + eval $(tst_net_vars $IPV6_LHOST/$IPV6_LPREFIX \
> + $IPV6_RHOST/$IPV6_RPREFIX || echo "exit $?")
> + fi
>
> tst_res_ TINFO "Network config (local -- remote):"
> tst_res_ TINFO "$LHOST_IFACES -- $RHOST_IFACES"
> tst_res_ TINFO "$IPV4_LHOST/$IPV4_LPREFIX -- $IPV4_RHOST/$IPV4_RPREFIX"
> tst_res_ TINFO "$IPV6_LHOST/$IPV6_LPREFIX -- $IPV6_RHOST/$IPV6_RPREFIX"
> +
> export _tst_net_parse_variables="yes"
> fi
>
--
Mailing list info: https://lists.linux.it/listinfo/ltp
next prev parent reply other threads:[~2022-01-25 8:32 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-24 22:03 [LTP] [PATCH 1/2] tst_net.sh: Fix for disabled IPv6 Petr Vorel
2022-01-24 22:03 ` [LTP] [PATCH 2/2] broken_ip-nexthdr.sh: Check IPv6 support before forcing it Petr Vorel
2022-01-24 22:11 ` [LTP] [PATCH 1/2] tst_net.sh: Fix for disabled IPv6 Petr Vorel
2022-01-25 8:32 ` Alexey Kodanev [this message]
2022-01-25 9:08 ` Petr Vorel
2022-02-02 10:32 ` Petr Vorel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=29051e9c-f4c2-9840-7f84-8c9f49efba93@bell-sw.com \
--to=aleksei.kodanev@bell-sw.com \
--cc=ltp@lists.linux.it \
--cc=petr.vorel@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).