nfp: clean mc addresses in application firmware when closing port

When moving devices from one namespace to another, mc addresses are
cleaned in software while not removed from application firmware. Thus
the mc addresses are remained and will cause resource leak.

Now use `__dev_mc_unsync` to clean mc addresses when closing port.

Fixes: e20aa071cd95 ("nfp: fix schedule in atomic context when sync mc address")
Cc: [email protected]
Signed-off-by: Yinjun Zhang <[email protected]>
Acked-by: Simon Horman <[email protected]>
Signed-off-by: Louis Peens <[email protected]>
Reviewed-by: Jacob Keller <[email protected]>
Message-ID: <20230705052818[email protected]>
Signed-off-by: Jakub Kicinski <[email protected]>

diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_common.c b/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
index 49f2f081ebb54a19876964111dfd8c19da86a521..6b1fb57084347bee2ceb5f75b7eca5a29b020423 100644 (file)
--- a/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
+++ b/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
@@ -53,6 +53,8 @@
 #include "crypto/crypto.h"
 #include "crypto/fw.h"
 
+static int nfp_net_mc_unsync(struct net_device *netdev, const unsigned char *addr);
+
 /**
  * nfp_net_get_fw_version() - Read and parse the FW version
  * @fw_ver:    Output fw_version structure to read to
@@ -1084,6 +1086,9 @@ static int nfp_net_netdev_close(struct net_device *netdev)
 
        /* Step 2: Tell NFP
         */
+       if (nn->cap_w1 & NFP_NET_CFG_CTRL_MCAST_FILTER)
+               __dev_mc_unsync(netdev, nfp_net_mc_unsync);
+
        nfp_net_clear_config_and_disable(nn);
        nfp_port_configure(netdev, false);