From: David Daney <ddaney@avtrex.com>
To: Janos Farkas <chexum+dev@gmail.com>
Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
pgf@foxharp.boston.ma.us, freek@macfreek.nl
Subject: Re: Broadcast ARP packets on link local addresses (Version2).
Date: Thu, 06 Apr 2006 09:17:58 -0700 [thread overview]
Message-ID: <44353F36.9070404@avtrex.com> (raw)
In-Reply-To: <priv$efbe06144502$2d51735f79@200604.gmail.com>
Janos Farkas wrote:
> On 2006-04-05 at 14:22:08, David Daney wrote:
>
>>The changes in this version are that it tests the source IP address
>>instead of the destination. The test now matches the test described
>>in the RFC. Also a small cleanup as suggested by Herbert Xu.
>>
>>Some comments on the first version of the patch suggested that I do
>>'X' instead. Where 'X' was behavior different than that REQUIRED by
>>the RFC (the RFC's always seem to capitalize the word 'required').
>>
>>The reason that I implemented the behavior required by the RFC is so
>>that a device running the kernel can pass compliance tests that
>>mandate RFC compliance.
>
>
> Sorry for chiming in this late in the discussion, but... Shouldn't it
> be more correct to not depend on the ip address of the used network,
> but to use the "scope" parameter of the given address?
>
RFC 3927 specifies the Ethernet arp broadcast behavior for only
169.254.0.0/16. Presumably you could set the scope parameter to local
for addresses outside of that range or even for protocols other than
Ethernet. Since broadcasting ARP packets usually adversely effects
usable network bandwidth, we should probably only do it where it is
absolutely required. The overhead of testing the value required by the
RFC is quite low (3 machine instructions on i686 is the size of the
entire patch), so using some proxy like the scope parameter would not
even be a performance win.
David Daney
next prev parent reply other threads:[~2006-04-06 16:17 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-05 21:22 [PATCH] net: Broadcast ARP packets on link local addresses (Version2) David Daney
2006-04-06 10:24 ` Janos Farkas
2006-04-06 16:17 ` David Daney [this message]
2006-04-06 16:53 ` Janos Farkas
2006-04-07 13:30 ` jamal
2006-04-07 16:18 ` David Daney
2006-04-07 20:38 ` Mark Butler
2006-04-22 17:14 ` [PATCH] net: " Anand Kumria
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=44353F36.9070404@avtrex.com \
--to=ddaney@avtrex.com \
--cc=chexum+dev@gmail.com \
--cc=freek@macfreek.nl \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pgf@foxharp.boston.ma.us \
/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).