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. > >Signed-off-by: Nikolay Aleksandrov <niko...@cumulusnetworks.com> -- 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
