On Fri, Mar 23, 2018 at 12:20 PM, Nikolay Aleksandrov <niko...@cumulusnetworks.com> wrote: > On 23/03/18 18:17, David Miller wrote: >> From: Chas Williams <3ch...@gmail.com> >> Date: Thu, 22 Mar 2018 11:34:06 -0400 >> >>> If the bridge is allowing multiple VLANs, some VLANs may have >>> different MTUs. Instead of choosing the minimum MTU for the >>> bridge interface, choose the maximum MTU of the bridge members. >>> With this the user only needs to set a larger MTU on the member >>> ports that are participating in the large MTU VLANS. >>> >>> Signed-off-by: Chas Williams <3ch...@gmail.com> >> >> Applied, thanks. >> > > Argh, this will break on builds without vlans because br->vlan_enabled > shouldn't > be accessed directly. I missed that when reviewing. > I'll send a follow up fix in a second that uses br_vlan_enabled(). >
Sorry I missed that one. Thanks for covering!