diff options
author | Petr Machata <petrm@mellanox.com> | 2018-10-17 10:53:27 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-10-18 02:45:08 +0200 |
commit | 045a5a99141d96e6254664783e8afb4e298ae2c9 (patch) | |
tree | e8a0213d96d325a6c29c497bd9d53ad799916d32 | |
parent | vxlan: Support marking RDSTs as offloaded (diff) | |
download | linux-045a5a99141d96e6254664783e8afb4e298ae2c9.tar.xz linux-045a5a99141d96e6254664783e8afb4e298ae2c9.zip |
vxlan: Notify for each remote of a removed FDB entry
When notifications are sent about FDB activity, and an FDB entry with
several remotes is removed, the notification is sent only for the first
destination. That makes it impossible to distinguish between the case
where only this first remote is removed, and the one where the FDB entry
is removed as a whole.
Therefore send one notification for each remote of a removed FDB entry.
Signed-off-by: Petr Machata <petrm@mellanox.com>
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/vxlan.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c index e98fc54379f8..1d74f90d6f5d 100644 --- a/drivers/net/vxlan.c +++ b/drivers/net/vxlan.c @@ -843,12 +843,15 @@ static void vxlan_fdb_free(struct rcu_head *head) static void vxlan_fdb_destroy(struct vxlan_dev *vxlan, struct vxlan_fdb *f, bool do_notify) { + struct vxlan_rdst *rd; + netdev_dbg(vxlan->dev, "delete %pM\n", f->eth_addr); --vxlan->addrcnt; if (do_notify) - vxlan_fdb_notify(vxlan, f, first_remote_rtnl(f), RTM_DELNEIGH); + list_for_each_entry(rd, &f->remotes, list) + vxlan_fdb_notify(vxlan, f, rd, RTM_DELNEIGH); hlist_del_rcu(&f->hlist); call_rcu(&f->rcu, vxlan_fdb_free); |