From: Jay Vosburgh <jay.vosburgh@canonical.com>
To: =?UTF-8?B?TWFoZXNoIEJhbmRld2FyICjgpK7gpLngpYfgpLYg4KSs4KSC4KSh4KWH4KS14KS+4KSwKQ==?=
<maheshb@google.com>
Cc: Michal Soltys <soltys@ziu.info>, linux-netdev <netdev@vger.kernel.org>
Subject: Re: [BUG] bonded interfaces drop bpdu (stp) frames
Date: Thu, 12 Jul 2018 10:51:58 -0400 [thread overview]
Message-ID: <24902.1531407118@nyx> (raw)
In-Reply-To: <CAF2d9jjK5DdoTgG2YYsZ+M5TBx-C1AcbT4VVmKrEMeJ7+PacnQ@mail.gmail.com>
Mahesh Bandewar (महेश बंडेवार) wrote:
>On Wed, Jul 11, 2018 at 3:23 PM, Michal Soltys <soltys@ziu.info> wrote:
>>
>> Hi,
>>
>> As weird as that sounds, this is what I observed today after bumping
>> kernel version. I have a setup where 2 bonds are attached to linux
>> bridge and physically are connected to two switches doing MSTP (and
>> linux bridge is just passing them).
>>
>> Initially I suspected some changes related to bridge code - but quick
>> peek at the code showed nothing suspicious - and the part of it that
>> explicitly passes stp frames if stp is not enabled has seen little
>> changes (e.g. per-port group_fwd_mask added recently). Furthermore - if
>> regular non-bonded interfaces are attached everything works fine.
>>
>> Just to be sure I detached the bond (802.3ad mode) and checked it with
>> simple tcpdump (ether proto \\stp) - and indeed no hello packets were
>> there (with them being present just fine on active enslaved interface,
>> or on the bond device in earlier kernels).
>>
>> If time permits I'll bisect tommorow to pinpoint the commit, but from
>> quick todays test - 4.9.x is working fine, while 4.16.16 (tested on
>> debian) and 4.17.3 (tested on archlinux) are failing.
>>
>> Unless this is already a known issue (or you have any suggestions what
>> could be responsible).
>>
>I believe these are link-local-multicast messages and sometime back a
>change went into to not pass those frames to the bonding master. This
>could be the side effect of that.
Mahesh, I suspect you're thinking of:
commit b89f04c61efe3b7756434d693b9203cc0cce002e
Author: Chonggang Li <chonggangli@google.com>
Date: Sun Apr 16 12:02:18 2017 -0700
bonding: deliver link-local packets with skb->dev set to link that packets arrived on
Michal, are you able to revert this patch and test?
-J
---
-Jay Vosburgh, jay.vosburgh@canonical.com
next prev parent reply other threads:[~2018-07-12 15:01 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-11 22:23 [BUG] bonded interfaces drop bpdu (stp) frames Michal Soltys
2018-07-11 23:22 ` Mahesh Bandewar (महेश बंडेवार)
2018-07-12 14:51 ` Jay Vosburgh [this message]
2018-07-12 16:37 ` Michal Soltys
2018-07-12 18:03 ` Jay Vosburgh
2018-07-12 21:26 ` Mahesh Bandewar (महेश बंडेवार)
2018-07-12 22:03 ` Jay Vosburgh
2018-07-12 23:14 ` Michal Soltys
2018-07-13 0:15 ` Mahesh Bandewar (महेश बंडेवार)
2018-07-14 15:48 ` Michal Soltys
2018-07-12 22:14 ` Michal Soltys
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=24902.1531407118@nyx \
--to=jay.vosburgh@canonical.com \
--cc=maheshb@google.com \
--cc=netdev@vger.kernel.org \
--cc=soltys@ziu.info \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.