On Mon, Oct 12, 2015 at 10:51 AM, Ido Schimmel <ido...@mellanox.com> wrote: > Mon, Oct 12, 2015 at 02:41:09PM IDT, ra...@blackwall.org wrote: >>From: Nikolay Aleksandrov <niko...@cumulusnetworks.com> >> >>As Ido Schimmel pointed out the vlan_vid_del() loop in nbp_vlan_flush is >>unnecessary (and is actually a remnant of the old vlan code) so we can >>remove it and combine both br/nbp vlan_flush functions into one. > Just a small note to Scott and Vivien: > > One of the side effects of Nik's recent patchsets is that when VLANs are > flushed on a port the deletion is propagated to the driver via > switchdev ops, as __vlan_vid_del is called. > > Therefore there is no need to do internal bookkeeping and remove VLANs > yourself when port is removed from bridge.
Thanks for the heads-up. Seems like a nice result. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
