From: "Bjørn Mork" <bjorn@mork.no>
To: nicolas.dichtel@6wind.com
Cc: David Miller <davem@davemloft.net>,
netdev@vger.kernel.org, yoshfuji@linux-ipv6.org
Subject: Re: [PATCH net-next] ip6_tunnel: ensure to always have a link local address
Date: Wed, 21 Aug 2013 11:02:52 +0200 [thread overview]
Message-ID: <87haej76wz.fsf@nemi.mork.no> (raw)
In-Reply-To: <52146EE9.1060101@6wind.com> (Nicolas Dichtel's message of "Wed, 21 Aug 2013 09:40:25 +0200")
Nicolas Dichtel <nicolas.dichtel@6wind.com> writes:
> Le 21/08/2013 08:48, David Miller a écrit :
>
>> Applied, but this brings up an issue I keep noticing.
>>
>> We talk about eth_random_addr() and "uniqueness" together all the
>> time, but the former never implies the latter.
>>
>> And we're going to run into situations where any conflicts generated
>> by this random address generater will cause reall failures.
>>
>> Therefore we'll have to create a system to prevent them. Probably
>> using some simple table that keeps track of the addresses we've
>> generated.
>>
> Ok, I will look at this.
Are eth_random_addr() collisions really any different than interfaces
having the same address for other reasons? Reality is that we never can
trust a mac address to be truly unique, regardless of source. And most
of the time it doesn't matter. There is no problem with two interfaces
having the same link local address as long as they are on different
links, for example.
What's important is that we deal with collisions gracefully in cases
where they do matter, allowing an administrator to fix the issue. I
believe we already do that on DAD failure with any mac address derived
link local address, disabling IPv6 and logging the reason.
Bjørn
next prev parent reply other threads:[~2013-08-21 9:03 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-20 10:16 [PATCH net-next] ip6_tunnel: ensure to always have a link local address Nicolas Dichtel
2013-08-21 6:48 ` David Miller
2013-08-21 7:40 ` Nicolas Dichtel
2013-08-21 9:02 ` Bjørn Mork [this message]
2013-08-21 10:25 ` Nicolas Dichtel
2013-08-21 11:37 ` Bjørn Mork
2013-08-21 12:11 ` Nicolas Dichtel
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=87haej76wz.fsf@nemi.mork.no \
--to=bjorn@mork.no \
--cc=davem@davemloft.net \
--cc=netdev@vger.kernel.org \
--cc=nicolas.dichtel@6wind.com \
--cc=yoshfuji@linux-ipv6.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).