bridge: defer switchdev fdb del call in fdb_del_external_learn

Since spinlock is held here, defer the switchdev operation. Also, ensure
that defered switchdev ops are processed before port master device
is unlinked.

Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

2 files changed, 7 insertions, 1 deletions

diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c
index f5e7da0fe93b..c88bd8e8937e 100644
--- a/net/bridge/br_fdb.c
+++ b/net/bridge/br_fdb.c
@@ -134,7 +134,10 @@ static void fdb_del_hw_addr(struct net_bridge *br, const unsigned char *addr)
 static void fdb_del_external_learn(struct net_bridge_fdb_entry *f)
 {
         struct switchdev_obj_port_fdb fdb = {
-                .obj.id = SWITCHDEV_OBJ_ID_PORT_FDB,
+                .obj = {
+                        .id = SWITCHDEV_OBJ_ID_PORT_FDB,
+                        .flags = SWITCHDEV_F_DEFER,
+                },
                 .vid = f->vlan_id,
         };
 
diff --git a/net/bridge/br_if.c b/net/bridge/br_if.c
index 45e4757c6fd2..ec02f5869a78 100644
--- a/net/bridge/br_if.c
+++ b/net/bridge/br_if.c
@@ -24,6 +24,7 @@
 #include <linux/slab.h>
 #include <net/sock.h>
 #include <linux/if_vlan.h>
+#include <net/switchdev.h>
 
 #include "br_private.h"
 
@@ -250,6 +251,8 @@ static void del_nbp(struct net_bridge_port *p)
 
         nbp_vlan_flush(p);
         br_fdb_delete_by_port(br, p, 0, 1);
+        switchdev_deferred_process();
+
         nbp_update_port_count(br);
 
         netdev_upper_dev_unlink(dev, br->dev);