netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Linus Lüssing" <linus.luessing@web.de>
To: Amerigo Wang <amwang@redhat.com>
Cc: netdev@vger.kernel.org, Herbert Xu <herbert@gondor.hengli.com.au>,
	Stephen Hemminger <stephen@networkplumber.org>,
	"David S. Miller" <davem@davemloft.net>,
	Adam Baker <linux@baker-net.org.uk>,
	Cong Wang <amwang@redhat.com>
Subject: Re: [net-next,v5,2/3] bridge: only expire the mdb entry when query is received
Date: Tue, 18 Jun 2013 14:03:38 +0200	[thread overview]
Message-ID: <20130618120338.GA9063@Linus-Debian> (raw)
In-Reply-To: <1369209176-6841-2-git-send-email-amwang@redhat.com>

Hi Amerigo,

I just tested these three patches on top of 3.10-rc6 (thanks for
looking into these issues so far!) but unfortunately I'm still
having problems if there is no querier on the link:

We will never discover a multicast listener which sends its
startup MLD reports before being attached to the bridge port.

I had reported that issue two months ago to Herbert's patch which
changed the querier default [1].

I guess in such a scenario where neither our querier is activated
nor could we detect any querier, then the only safe thing we can
do is to disable the bridge multicast snooping, isn't it?

Cheers, Linus


[1]: http://permalink.gmane.org/gmane.linux.network/264442

  reply	other threads:[~2013-06-18 12:04 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-22  7:52 [Patch net-next v5 1/3] bridge: use the bridge IP addr as source addr for querier Cong Wang
2013-05-22  7:52 ` [Patch net-next v5 2/3] bridge: only expire the mdb entry when query is received Cong Wang
2013-06-18 12:03   ` Linus Lüssing [this message]
2013-06-19  4:44     ` [net-next,v5,2/3] " Herbert Xu
2013-06-21  7:31       ` Cong Wang
2013-06-21  7:34         ` Herbert Xu
2013-05-22  7:52 ` [Patch net-next v5 3/3] bridge: send query as soon as leave " Cong Wang
2013-05-22  7:55 ` [Patch net-next v5 1/3] bridge: use the bridge IP addr as source addr for querier Herbert Xu
2013-05-25 20:52 ` Adam Baker

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=20130618120338.GA9063@Linus-Debian \
    --to=linus.luessing@web.de \
    --cc=amwang@redhat.com \
    --cc=davem@davemloft.net \
    --cc=herbert@gondor.hengli.com.au \
    --cc=linux@baker-net.org.uk \
    --cc=netdev@vger.kernel.org \
    --cc=stephen@networkplumber.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).