From: Andrew Lunn <andrew@lunn•ch>
To: David Miller <davem@davemloft•net>
Cc: netdev@vger•kernel.org
Subject: Re: [net-next RFC] net: ipv4: Send IGMP messages from highest scoped address
Date: Tue, 18 Aug 2015 15:36:41 +0200 [thread overview]
Message-ID: <20150818133641.GB4381@lunn.ch> (raw)
In-Reply-To: <20150817.213158.1038537149649115817.davem@davemloft.net>
On Mon, Aug 17, 2015 at 09:31:58PM -0700, David Miller wrote:
> From: Andrew Lunn <andrew@lunn•ch>
> Date: Mon, 17 Aug 2015 18:09:24 +0200
>
> > This is RFC because i personally don't know if this is the best fix.
> > The patch restores previous behavior, while still keeping the bug fix.
> >
> > It is not obvious what is the correct source address for an IGMP
> > message when an interface has multiple addresses. IGMP messages are
> > sent either spontaneously, or as a result of a query. It could be
> > argued that when replying to a query, an address take from the same
> > subnet as the querier should be used. Doing this adds complexity for a
> > corner case which does not seem to effect people. In the spontaneous
> > case, there is no such hint, so an address has to be picked some other
> > way. Taking the highest scope address seems reasonable, and works for
> > me.
>
> Yes, unfortunately almost no guidance is given in this area in the
> various RFCs.
>
> Except that one recommended way to avoid forged IGMPs is to reject
> any IGMP that has a source address no on that interface's subnet.
>
> And if people actually do that, then the link scope address is the
> best address to use and that is what we do now if I understand things
> correctly.
Hi David
We currently take the first address from the interface which is scope
link or higher.
Historically, the global scope address would of been used, but my
previous fix, which stopped it taking a global scope address from a
different interface altogether under some conditions, changed this
behaviour.
The first address from the interface, then broke one of my use
case. The querier is only in one of the subnets on this interface, and
using an address from the global scope address range. It then drops
the membership reports when they are sent from the first address on
the interface. This is why i want to restore the previous behaviour,
take the global scope address from this interface.
The patch works for me and is restoring previous behaviour, but is
that sufficient to make it correct?
Andrew
next prev parent reply other threads:[~2015-08-18 13:43 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-17 16:09 [net-next RFC] net: ipv4: Send IGMP messages from highest scoped address Andrew Lunn
2015-08-18 4:31 ` David Miller
2015-08-18 13:36 ` Andrew Lunn [this message]
2015-08-18 19:01 ` David Miller
2015-08-18 20:31 ` Andrew Lunn
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=20150818133641.GB4381@lunn.ch \
--to=andrew@lunn$(echo .)ch \
--cc=davem@davemloft$(echo .)net \
--cc=netdev@vger$(echo .)kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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