From: Nikolay Aleksandrov <nikolay@nvidia.com>
To: "Linus Lüssing" <linus.luessing@c0d3.blue>, netdev@vger.kernel.org
Cc: Roopa Prabhu <roopa@nvidia.com>, Jakub Kicinski <kuba@kernel.org>,
"David S . Miller" <davem@davemloft.net>,
bridge@lists.linux-foundation.org,
b.a.t.m.a.n@lists.open-mesh.org, linux-kernel@vger.kernel.org
Subject: Re: [net-next v2 01/11] net: bridge: mcast: rename multicast router lists and timers
Date: Tue, 11 May 2021 12:12:39 +0300 [thread overview]
Message-ID: <5c126c23-9ef2-7d7e-7574-538287f604ef@nvidia.com> (raw)
In-Reply-To: <20210509194509.10849-2-linus.luessing@c0d3.blue>
On 09/05/2021 22:44, Linus Lüssing wrote:
> In preparation for the upcoming split of multicast router state into
> their IPv4 and IPv6 variants, rename the affected variable to the IPv4
> version first to avoid some renames in later commits.
>
> Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue>
> ---
> net/bridge/br_forward.c | 2 +-
> net/bridge/br_mdb.c | 6 ++---
> net/bridge/br_multicast.c | 48 +++++++++++++++++++--------------------
> net/bridge/br_private.h | 10 ++++----
> 4 files changed, 33 insertions(+), 33 deletions(-)
>
Acked-by: Nikolay Aleksandrov <nikolay@nvidia.com>
> diff --git a/net/bridge/br_forward.c b/net/bridge/br_forward.c
> index 6e9b049..3b67184 100644
> --- a/net/bridge/br_forward.c
> +++ b/net/bridge/br_forward.c
> @@ -290,7 +290,7 @@ void br_multicast_flood(struct net_bridge_mdb_entry *mdst,
> struct net_bridge_port *port, *lport, *rport;
>
> lport = p ? p->key.port : NULL;
> - rport = hlist_entry_safe(rp, struct net_bridge_port, rlist);
> + rport = hlist_entry_safe(rp, struct net_bridge_port, ip4_rlist);
>
> if ((unsigned long)lport > (unsigned long)rport) {
> port = lport;
> diff --git a/net/bridge/br_mdb.c b/net/bridge/br_mdb.c
> index 95fa4af..d61def8 100644
> --- a/net/bridge/br_mdb.c
> +++ b/net/bridge/br_mdb.c
> @@ -23,14 +23,14 @@ static int br_rports_fill_info(struct sk_buff *skb, struct netlink_callback *cb,
> struct net_bridge_port *p;
> struct nlattr *nest, *port_nest;
>
> - if (!br->multicast_router || hlist_empty(&br->router_list))
> + if (!br->multicast_router || hlist_empty(&br->ip4_mc_router_list))
> return 0;
>
> nest = nla_nest_start_noflag(skb, MDBA_ROUTER);
> if (nest == NULL)
> return -EMSGSIZE;
>
> - hlist_for_each_entry_rcu(p, &br->router_list, rlist) {
> + hlist_for_each_entry_rcu(p, &br->ip4_mc_router_list, ip4_rlist) {
> if (!p)
> continue;
> port_nest = nla_nest_start_noflag(skb, MDBA_ROUTER_PORT);
> @@ -38,7 +38,7 @@ static int br_rports_fill_info(struct sk_buff *skb, struct netlink_callback *cb,
> goto fail;
> if (nla_put_nohdr(skb, sizeof(u32), &p->dev->ifindex) ||
> nla_put_u32(skb, MDBA_ROUTER_PATTR_TIMER,
> - br_timer_value(&p->multicast_router_timer)) ||
> + br_timer_value(&p->ip4_mc_router_timer)) ||
> nla_put_u8(skb, MDBA_ROUTER_PATTR_TYPE,
> p->multicast_router)) {
> nla_nest_cancel(skb, port_nest);
> diff --git a/net/bridge/br_multicast.c b/net/bridge/br_multicast.c
> index 226bb05..6fe93a3 100644
> --- a/net/bridge/br_multicast.c
> +++ b/net/bridge/br_multicast.c
> @@ -1357,13 +1357,13 @@ static int br_ip6_multicast_add_group(struct net_bridge *br,
> static void br_multicast_router_expired(struct timer_list *t)
> {
> struct net_bridge_port *port =
> - from_timer(port, t, multicast_router_timer);
> + from_timer(port, t, ip4_mc_router_timer);
> struct net_bridge *br = port->br;
>
> spin_lock(&br->multicast_lock);
> if (port->multicast_router == MDB_RTR_TYPE_DISABLED ||
> port->multicast_router == MDB_RTR_TYPE_PERM ||
> - timer_pending(&port->multicast_router_timer))
> + timer_pending(&port->ip4_mc_router_timer))
> goto out;
>
> __del_port_router(port);
> @@ -1386,12 +1386,12 @@ static void br_mc_router_state_change(struct net_bridge *p,
>
> static void br_multicast_local_router_expired(struct timer_list *t)
> {
> - struct net_bridge *br = from_timer(br, t, multicast_router_timer);
> + struct net_bridge *br = from_timer(br, t, ip4_mc_router_timer);
>
> spin_lock(&br->multicast_lock);
> if (br->multicast_router == MDB_RTR_TYPE_DISABLED ||
> br->multicast_router == MDB_RTR_TYPE_PERM ||
> - timer_pending(&br->multicast_router_timer))
> + timer_pending(&br->ip4_mc_router_timer))
> goto out;
>
> br_mc_router_state_change(br, false);
> @@ -1613,7 +1613,7 @@ int br_multicast_add_port(struct net_bridge_port *port)
> port->multicast_router = MDB_RTR_TYPE_TEMP_QUERY;
> port->multicast_eht_hosts_limit = BR_MCAST_DEFAULT_EHT_HOSTS_LIMIT;
>
> - timer_setup(&port->multicast_router_timer,
> + timer_setup(&port->ip4_mc_router_timer,
> br_multicast_router_expired, 0);
> timer_setup(&port->ip4_own_query.timer,
> br_ip4_multicast_port_query_expired, 0);
> @@ -1649,7 +1649,7 @@ void br_multicast_del_port(struct net_bridge_port *port)
> hlist_move_list(&br->mcast_gc_list, &deleted_head);
> spin_unlock_bh(&br->multicast_lock);
> br_multicast_gc(&deleted_head);
> - del_timer_sync(&port->multicast_router_timer);
> + del_timer_sync(&port->ip4_mc_router_timer);
> free_percpu(port->mcast_stats);
> }
>
> @@ -1674,7 +1674,7 @@ static void __br_multicast_enable_port(struct net_bridge_port *port)
> br_multicast_enable(&port->ip6_own_query);
> #endif
> if (port->multicast_router == MDB_RTR_TYPE_PERM &&
> - hlist_unhashed(&port->rlist))
> + hlist_unhashed(&port->ip4_rlist))
> br_multicast_add_router(br, port);
> }
>
> @@ -1700,7 +1700,7 @@ void br_multicast_disable_port(struct net_bridge_port *port)
>
> __del_port_router(port);
>
> - del_timer(&port->multicast_router_timer);
> + del_timer(&port->ip4_mc_router_timer);
> del_timer(&port->ip4_own_query.timer);
> #if IS_ENABLED(CONFIG_IPV6)
> del_timer(&port->ip6_own_query.timer);
> @@ -2666,19 +2666,19 @@ static void br_multicast_add_router(struct net_bridge *br,
> struct net_bridge_port *p;
> struct hlist_node *slot = NULL;
>
> - if (!hlist_unhashed(&port->rlist))
> + if (!hlist_unhashed(&port->ip4_rlist))
> return;
>
> - hlist_for_each_entry(p, &br->router_list, rlist) {
> + hlist_for_each_entry(p, &br->ip4_mc_router_list, ip4_rlist) {
> if ((unsigned long) port >= (unsigned long) p)
> break;
> - slot = &p->rlist;
> + slot = &p->ip4_rlist;
> }
>
> if (slot)
> - hlist_add_behind_rcu(&port->rlist, slot);
> + hlist_add_behind_rcu(&port->ip4_rlist, slot);
> else
> - hlist_add_head_rcu(&port->rlist, &br->router_list);
> + hlist_add_head_rcu(&port->ip4_rlist, &br->ip4_mc_router_list);
> br_rtr_notify(br->dev, port, RTM_NEWMDB);
> br_port_mc_router_state_change(port, true);
> }
> @@ -2690,9 +2690,9 @@ static void br_multicast_mark_router(struct net_bridge *br,
>
> if (!port) {
> if (br->multicast_router == MDB_RTR_TYPE_TEMP_QUERY) {
> - if (!timer_pending(&br->multicast_router_timer))
> + if (!timer_pending(&br->ip4_mc_router_timer))
> br_mc_router_state_change(br, true);
> - mod_timer(&br->multicast_router_timer,
> + mod_timer(&br->ip4_mc_router_timer,
> now + br->multicast_querier_interval);
> }
> return;
> @@ -2704,7 +2704,7 @@ static void br_multicast_mark_router(struct net_bridge *br,
>
> br_multicast_add_router(br, port);
>
> - mod_timer(&port->multicast_router_timer,
> + mod_timer(&port->ip4_mc_router_timer,
> now + br->multicast_querier_interval);
> }
>
> @@ -3316,7 +3316,7 @@ void br_multicast_init(struct net_bridge *br)
> br_opt_toggle(br, BROPT_HAS_IPV6_ADDR, true);
>
> spin_lock_init(&br->multicast_lock);
> - timer_setup(&br->multicast_router_timer,
> + timer_setup(&br->ip4_mc_router_timer,
> br_multicast_local_router_expired, 0);
> timer_setup(&br->ip4_other_query.timer,
> br_ip4_multicast_querier_expired, 0);
> @@ -3416,7 +3416,7 @@ void br_multicast_open(struct net_bridge *br)
>
> void br_multicast_stop(struct net_bridge *br)
> {
> - del_timer_sync(&br->multicast_router_timer);
> + del_timer_sync(&br->ip4_mc_router_timer);
> del_timer_sync(&br->ip4_other_query.timer);
> del_timer_sync(&br->ip4_own_query.timer);
> #if IS_ENABLED(CONFIG_IPV6)
> @@ -3453,7 +3453,7 @@ int br_multicast_set_router(struct net_bridge *br, unsigned long val)
> case MDB_RTR_TYPE_DISABLED:
> case MDB_RTR_TYPE_PERM:
> br_mc_router_state_change(br, val == MDB_RTR_TYPE_PERM);
> - del_timer(&br->multicast_router_timer);
> + del_timer(&br->ip4_mc_router_timer);
> br->multicast_router = val;
> err = 0;
> break;
> @@ -3472,9 +3472,9 @@ int br_multicast_set_router(struct net_bridge *br, unsigned long val)
>
> static void __del_port_router(struct net_bridge_port *p)
> {
> - if (hlist_unhashed(&p->rlist))
> + if (hlist_unhashed(&p->ip4_rlist))
> return;
> - hlist_del_init_rcu(&p->rlist);
> + hlist_del_init_rcu(&p->ip4_rlist);
> br_rtr_notify(p->br->dev, p, RTM_DELMDB);
> br_port_mc_router_state_change(p, false);
>
> @@ -3493,7 +3493,7 @@ int br_multicast_set_port_router(struct net_bridge_port *p, unsigned long val)
> if (p->multicast_router == val) {
> /* Refresh the temp router port timer */
> if (p->multicast_router == MDB_RTR_TYPE_TEMP)
> - mod_timer(&p->multicast_router_timer,
> + mod_timer(&p->ip4_mc_router_timer,
> now + br->multicast_querier_interval);
> err = 0;
> goto unlock;
> @@ -3502,7 +3502,7 @@ int br_multicast_set_port_router(struct net_bridge_port *p, unsigned long val)
> case MDB_RTR_TYPE_DISABLED:
> p->multicast_router = MDB_RTR_TYPE_DISABLED;
> __del_port_router(p);
> - del_timer(&p->multicast_router_timer);
> + del_timer(&p->ip4_mc_router_timer);
> break;
> case MDB_RTR_TYPE_TEMP_QUERY:
> p->multicast_router = MDB_RTR_TYPE_TEMP_QUERY;
> @@ -3510,7 +3510,7 @@ int br_multicast_set_port_router(struct net_bridge_port *p, unsigned long val)
> break;
> case MDB_RTR_TYPE_PERM:
> p->multicast_router = MDB_RTR_TYPE_PERM;
> - del_timer(&p->multicast_router_timer);
> + del_timer(&p->ip4_mc_router_timer);
> br_multicast_add_router(br, p);
> break;
> case MDB_RTR_TYPE_TEMP:
> diff --git a/net/bridge/br_private.h b/net/bridge/br_private.h
> index 7ce8a77..26e91d2 100644
> --- a/net/bridge/br_private.h
> +++ b/net/bridge/br_private.h
> @@ -307,6 +307,8 @@ struct net_bridge_port {
>
> #ifdef CONFIG_BRIDGE_IGMP_SNOOPING
> struct bridge_mcast_own_query ip4_own_query;
> + struct timer_list ip4_mc_router_timer;
> + struct hlist_node ip4_rlist;
> #if IS_ENABLED(CONFIG_IPV6)
> struct bridge_mcast_own_query ip6_own_query;
> #endif /* IS_ENABLED(CONFIG_IPV6) */
> @@ -314,9 +316,7 @@ struct net_bridge_port {
> u32 multicast_eht_hosts_cnt;
> unsigned char multicast_router;
> struct bridge_mcast_stats __percpu *mcast_stats;
> - struct timer_list multicast_router_timer;
> struct hlist_head mglist;
> - struct hlist_node rlist;
> #endif
>
> #ifdef CONFIG_SYSFS
> @@ -449,9 +449,9 @@ struct net_bridge {
>
> struct hlist_head mcast_gc_list;
> struct hlist_head mdb_list;
> - struct hlist_head router_list;
>
> - struct timer_list multicast_router_timer;
> + struct hlist_head ip4_mc_router_list;
> + struct timer_list ip4_mc_router_timer;
> struct bridge_mcast_other_query ip4_other_query;
> struct bridge_mcast_own_query ip4_own_query;
> struct bridge_mcast_querier ip4_querier;
> @@ -868,7 +868,7 @@ static inline bool br_multicast_is_router(struct net_bridge *br)
> {
> return br->multicast_router == 2 ||
> (br->multicast_router == 1 &&
> - timer_pending(&br->multicast_router_timer));
> + timer_pending(&br->ip4_mc_router_timer));
> }
>
> static inline bool
>
next prev parent reply other threads:[~2021-05-11 9:12 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-09 19:44 [PATCH net-next v2 00/11] net: bridge: split IPv4/v6 mc router state and export for batman-adv Linus Lüssing
2021-05-09 19:44 ` [net-next v2 01/11] net: bridge: mcast: rename multicast router lists and timers Linus Lüssing
2021-05-11 9:12 ` Nikolay Aleksandrov [this message]
2021-05-09 19:45 ` [net-next v2 02/11] net: bridge: mcast: add wrappers for router node retrieval Linus Lüssing
2021-05-11 9:12 ` Nikolay Aleksandrov
2021-05-09 19:45 ` [net-next v2 03/11] net: bridge: mcast: prepare mdb netlink for mcast router split Linus Lüssing
2021-05-11 9:13 ` Nikolay Aleksandrov
2021-05-09 19:45 ` [net-next v2 04/11] net: bridge: mcast: prepare query reception " Linus Lüssing
2021-05-11 9:13 ` Nikolay Aleksandrov
2021-05-09 19:45 ` [net-next v2 05/11] net: bridge: mcast: prepare is-router function " Linus Lüssing
2021-05-11 9:16 ` Nikolay Aleksandrov
2021-05-09 19:45 ` [net-next v2 06/11] net: bridge: mcast: prepare expiry functions " Linus Lüssing
2021-05-11 9:16 ` Nikolay Aleksandrov
2021-05-09 19:45 ` [net-next v2 07/11] net: bridge: mcast: prepare add-router function " Linus Lüssing
2021-05-09 19:45 ` [net-next v2 08/11] net: bridge: mcast: split router port del+notify " Linus Lüssing
2021-05-11 9:19 ` Nikolay Aleksandrov
2021-05-09 19:45 ` [net-next v2 09/11] net: bridge: mcast: split multicast router state for IPv4 and IPv6 Linus Lüssing
2021-05-11 9:29 ` Nikolay Aleksandrov
2021-05-11 9:33 ` Nikolay Aleksandrov
2021-05-11 11:28 ` Linus Lüssing
2021-05-09 19:45 ` [net-next v2 10/11] net: bridge: mcast: add ip4+ip6 mcast router timers to mdb netlink Linus Lüssing
2021-05-11 9:30 ` Nikolay Aleksandrov
2021-05-09 19:45 ` [net-next v2 11/11] net: bridge: mcast: export multicast router presence adjacent to a port Linus Lüssing
2021-05-11 9:23 ` Nikolay Aleksandrov
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=5c126c23-9ef2-7d7e-7574-538287f604ef@nvidia.com \
--to=nikolay@nvidia.com \
--cc=b.a.t.m.a.n@lists.open-mesh.org \
--cc=bridge@lists.linux-foundation.org \
--cc=davem@davemloft.net \
--cc=kuba@kernel.org \
--cc=linus.luessing@c0d3.blue \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=roopa@nvidia.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).