mlxsw: spectrum_switchdev: Release multicast groups during fini
authorIdo Schimmel <idosch@mellanox.com>
Fri, 4 Aug 2017 12:12:30 +0000 (14:12 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 4 Aug 2017 18:15:07 +0000 (11:15 -0700)
commit852cfeed0ebe93a1c1e0c153c04a57221f72bd7e
tree71a66a1f9bfd1283f9e4b8b83ed14216f73fa587
parent17b334a876fe121abbd6634ace83ca58deea4bc5
mlxsw: spectrum_switchdev: Release multicast groups during fini

Each multicast group (MID) stores a bitmap of ports to which a packet
should be forwarded to in case an MDB entry associated with the MID is
hit.

Since the initial introduction of IGMP snooping in commit 3a49b4fde2a1
("mlxsw: Adding layer 2 multicast support") the driver didn't correctly
free these multicast groups upon ungraceful situations such as the
removal of the upper bridge device or module removal.

The correct way to fix this is to associate each MID with the bridge
ports member in it and then drop the reference in case the bridge port
is destroyed, but this will result in a lot more code and will be fixed
in net-next.

For now, upon module removal, traverse the MID list and release each
one.

Fixes: 3a49b4fde2a1 ("mlxsw: Adding layer 2 multicast support")
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c