ipv6: Remove in6addr_any alternatives.

Some code defines the IPv6 wildcard address as a local variable and
use it with memcmp() or ipv6_addr_equal().

Let's use in6addr_any and ipv6_addr_any() instead.

Signed-off-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Reviewed-by: Mark Bloch <mbloch@nvidia.com>
Reviewed-by: David Ahern <dsahern@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en/tc_tun_encap.c b/drivers/net/ethernet/mellanox/mlx5/core/en/tc_tun_encap.c
index a108e73c9f6698fca00be5ebeddbcbae08564030..20c2d2ecaf9356da8d253e8e33a9b45add25bf73 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/en/tc_tun_encap.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en/tc_tun_encap.c
@@ -98,7 +98,6 @@ int mlx5e_tc_set_attr_rx_tun(struct mlx5e_tc_flow *flow,
 #if IS_ENABLED(CONFIG_INET) && IS_ENABLED(CONFIG_IPV6)
        else if (ip_version == 6) {
                int ipv6_size = MLX5_FLD_SZ_BYTES(ipv6_layout, ipv6);
-               struct in6_addr zerov6 = {};
 
                daddr = MLX5_ADDR_OF(fte_match_param, spec->match_value,
                                     outer_headers.dst_ipv4_dst_ipv6.ipv6_layout.ipv6);
@@ -106,8 +105,8 @@ int mlx5e_tc_set_attr_rx_tun(struct mlx5e_tc_flow *flow,
                                     outer_headers.src_ipv4_src_ipv6.ipv6_layout.ipv6);
                memcpy(&tun_attr->dst_ip.v6, daddr, ipv6_size);
                memcpy(&tun_attr->src_ip.v6, saddr, ipv6_size);
-               if (!memcmp(&tun_attr->dst_ip.v6, &zerov6, sizeof(zerov6)) ||
-                   !memcmp(&tun_attr->src_ip.v6, &zerov6, sizeof(zerov6)))
+               if (ipv6_addr_any(&tun_attr->dst_ip.v6) ||
+                   ipv6_addr_any(&tun_attr->src_ip.v6))
                        return 0;
        }
 #endif

diff --git a/include/net/ip6_fib.h b/include/net/ip6_fib.h
index 79570cb4ea9c8e85275c1e8198f1777eef3a9b4e..05e6f756feafeb1ebd41122f7bd793190a978929 100644 (file)
--- a/include/net/ip6_fib.h
+++ b/include/net/ip6_fib.h
@@ -469,13 +469,10 @@ void rt6_get_prefsrc(const struct rt6_info *rt, struct in6_addr *addr)
        rcu_read_lock();
 
        from = rcu_dereference(rt->from);
-       if (from) {
+       if (from)
                *addr = from->fib6_prefsrc.addr;
-       } else {
-               struct in6_addr in6_zero = {};
-
-               *addr = in6_zero;
-       }
+       else
+               *addr = in6addr_any;
 
        rcu_read_unlock();
 }

diff --git a/include/trace/events/fib.h b/include/trace/events/fib.h
index c2300c407f58325768032b4b7bcd7e83714f7564..76297ecd4935c97ffe97a620c8f398e7d835be50 100644 (file)
--- a/include/trace/events/fib.h
+++ b/include/trace/events/fib.h
@@ -36,7 +36,6 @@ TRACE_EVENT(fib_table_lookup,
        ),
 
        TP_fast_assign(
-               struct in6_addr in6_zero = {};
                struct net_device *dev;
                struct in6_addr *in6;
                __be32 *p32;
@@ -74,7 +73,7 @@ TRACE_EVENT(fib_table_lookup,
                                *p32 = nhc->nhc_gw.ipv4;
 
                                in6 = (struct in6_addr *)__entry->gw6;
-                               *in6 = in6_zero;
+                               *in6 = in6addr_any;
                        } else if (nhc->nhc_gw_family == AF_INET6) {
                                p32 = (__be32 *) __entry->gw4;
                                *p32 = 0;
@@ -87,7 +86,7 @@ TRACE_EVENT(fib_table_lookup,
                        *p32 = 0;
 
                        in6 = (struct in6_addr *)__entry->gw6;
-                       *in6 = in6_zero;
+                       *in6 = in6addr_any;
                }
        ),
 

diff --git a/include/trace/events/fib6.h b/include/trace/events/fib6.h
index 6e821eb7945033dc6ec89964e3b30fa3027966d0..4d3e607b3cdecc3583d4f06c59eca71e9fca365a 100644 (file)
--- a/include/trace/events/fib6.h
+++ b/include/trace/events/fib6.h
@@ -68,11 +68,8 @@ TRACE_EVENT(fib6_table_lookup,
                        strcpy(__entry->name, "-");
                }
                if (res->f6i == net->ipv6.fib6_null_entry) {
-                       struct in6_addr in6_zero = {};
-
                        in6 = (struct in6_addr *)__entry->gw;
-                       *in6 = in6_zero;
-
+                       *in6 = in6addr_any;
                } else if (res->nh) {
                        in6 = (struct in6_addr *)__entry->gw;
                        *in6 = res->nh->fib_nh_gw6;

diff --git a/net/ethtool/ioctl.c b/net/ethtool/ioctl.c
index 646b3e490c71a748f9612081971f36e7e449fc1a..59adc4e6e9eec118e6a2ae0425b6ed58b09f7c10 100644 (file)
--- a/net/ethtool/ioctl.c
+++ b/net/ethtool/ioctl.c
@@ -27,6 +27,7 @@
 #include <linux/net.h>
 #include <linux/pm_runtime.h>
 #include <net/devlink.h>
+#include <net/ipv6.h>
 #include <net/xdp_sock_drv.h>
 #include <net/flow_offload.h>
 #include <linux/ethtool_netlink.h>
@@ -3127,7 +3128,6 @@ struct ethtool_rx_flow_rule *
 ethtool_rx_flow_rule_create(const struct ethtool_rx_flow_spec_input *input)
 {
        const struct ethtool_rx_flow_spec *fs = input->fs;
-       static struct in6_addr zero_addr = {};
        struct ethtool_rx_flow_match *match;
        struct ethtool_rx_flow_rule *flow;
        struct flow_action_entry *act;
@@ -3233,20 +3233,20 @@ ethtool_rx_flow_rule_create(const struct ethtool_rx_flow_spec_input *input)
 
                v6_spec = &fs->h_u.tcp_ip6_spec;
                v6_m_spec = &fs->m_u.tcp_ip6_spec;
-               if (memcmp(v6_m_spec->ip6src, &zero_addr, sizeof(zero_addr))) {
+               if (!ipv6_addr_any((struct in6_addr *)v6_m_spec->ip6src)) {
                        memcpy(&match->key.ipv6.src, v6_spec->ip6src,
                               sizeof(match->key.ipv6.src));
                        memcpy(&match->mask.ipv6.src, v6_m_spec->ip6src,
                               sizeof(match->mask.ipv6.src));
                }
-               if (memcmp(v6_m_spec->ip6dst, &zero_addr, sizeof(zero_addr))) {
+               if (!ipv6_addr_any((struct in6_addr *)v6_m_spec->ip6dst)) {
                        memcpy(&match->key.ipv6.dst, v6_spec->ip6dst,
                               sizeof(match->key.ipv6.dst));
                        memcpy(&match->mask.ipv6.dst, v6_m_spec->ip6dst,
                               sizeof(match->mask.ipv6.dst));
                }
-               if (memcmp(v6_m_spec->ip6src, &zero_addr, sizeof(zero_addr)) ||
-                   memcmp(v6_m_spec->ip6dst, &zero_addr, sizeof(zero_addr))) {
+               if (!ipv6_addr_any((struct in6_addr *)v6_m_spec->ip6src) ||
+                   !ipv6_addr_any((struct in6_addr *)v6_m_spec->ip6dst)) {
                        match->dissector.used_keys |=
                                BIT(FLOW_DISSECTOR_KEY_IPV6_ADDRS);
                        match->dissector.offset[FLOW_DISSECTOR_KEY_IPV6_ADDRS] =

diff --git a/net/ipv4/inet_hashtables.c b/net/ipv4/inet_hashtables.c
index 6edae3886885565985f9f45bae65ef2f4498f22f..e7391bf310a75cd5599192e46f2a9577962e82ce 100644 (file)
--- a/net/ipv4/inet_hashtables.c
+++ b/net/ipv4/inet_hashtables.c
@@ -826,13 +826,11 @@ bool inet_bind2_bucket_match_addr_any(const struct inet_bind2_bucket *tb, const
                                      unsigned short port, int l3mdev, const struct sock *sk)
 {
 #if IS_ENABLED(CONFIG_IPV6)
-       struct in6_addr addr_any = {};
-
        if (sk->sk_family != tb->family) {
                if (sk->sk_family == AF_INET)
                        return net_eq(ib2_net(tb), net) && tb->port == port &&
                                tb->l3mdev == l3mdev &&
-                               ipv6_addr_equal(&tb->v6_rcv_saddr, &addr_any);
+                               ipv6_addr_any(&tb->v6_rcv_saddr);
 
                return false;
        }
@@ -840,7 +838,7 @@ bool inet_bind2_bucket_match_addr_any(const struct inet_bind2_bucket *tb, const
        if (sk->sk_family == AF_INET6)
                return net_eq(ib2_net(tb), net) && tb->port == port &&
                        tb->l3mdev == l3mdev &&
-                       ipv6_addr_equal(&tb->v6_rcv_saddr, &addr_any);
+                       ipv6_addr_any(&tb->v6_rcv_saddr);
        else
 #endif
                return net_eq(ib2_net(tb), net) && tb->port == port &&
@@ -866,11 +864,10 @@ inet_bhash2_addr_any_hashbucket(const struct sock *sk, const struct net *net, in
 {
        struct inet_hashinfo *hinfo = tcp_or_dccp_get_hashinfo(sk);
        u32 hash;
-#if IS_ENABLED(CONFIG_IPV6)
-       struct in6_addr addr_any = {};
 
+#if IS_ENABLED(CONFIG_IPV6)
        if (sk->sk_family == AF_INET6)
-               hash = ipv6_portaddr_hash(net, &addr_any, port);
+               hash = ipv6_portaddr_hash(net, &in6addr_any, port);
        else
 #endif
                hash = ipv4_portaddr_hash(net, 0, port);