On 2018-07-17 11:32, Michal Soltys wrote: > On 07/17/2018 01:53 AM, Mahesh Bandewar (महेश बंडेवार) wrote: >> On Mon, Jul 16, 2018 at 2:24 PM, Jay Vosburgh >> <jay.vosbu...@canonical.com> wrote: >>> Mahesh Bandewar <mah...@bandewar.net> wrote: >>> >>>> From: Mahesh Bandewar <mahe...@google.com> >>>> >>>> Commit b89f04c61efe ("bonding: deliver link-local packets with >>>> skb->dev set to link that packets arrived on") changed the behavior >>>> of how link-local-multicast packets are processed. The change in >>>> the behavior broke some legacy use cases where these packets are >>>> expected to arrive on bonding master device also. >>>> >>>> This patch passes the packet to the stack with the link it arrived >>>> on as well as passes to the bonding-master device to preserve the >>>> legacy use case. >>> >>> Michal, can you test this? I'm travelling this week and won't >>> be able to run the patch. > > > Yes, will test today and report. >
The patch looks to be working fine - tested both passive bridge (stp_state == 0) and with in-kernel implementation active (stp_state == 1). No loops, no issues so far.