linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: jamal <hadi@cyberus.ca>
To: David Daney <ddaney@avtrex.com>
Cc: Janos Farkas <chexum+dev@gmail.com>,
	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: Fri, 07 Apr 2006 09:30:38 -0400	[thread overview]
Message-ID: <1144416638.5082.33.camel@jzny2> (raw)
In-Reply-To: <44353F36.9070404@avtrex.com>

On Thu, 2006-06-04 at 09:17 -0700, David Daney wrote:
> Janos Farkas wrote:

> > 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?
> > 
> 

Excellent point! It was bothering me as well but i couldnt express my
view eloquently as you did.

> RFC 3927 specifies the Ethernet arp broadcast behavior for only 
> 169.254.0.0/16.

Thats besides the point. You could, for example, use 1.1.1.1/24 in your
network instead of the 10.x or 192.x; and i have seen people use 10.x
in what appears to be public networks. We dont have speacial checks for 
RFC 1918 IP addresses for example.

169.254.0.0/16 is by definition link local. I think point made by Janos
is we should look at the attributes rather than value.

Have your user space set it to be link local and then fix the kernel if
it doesnt do the right thing.

>   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.
> 

Again, that is beside the point. 

cheers,
jamal


  parent reply	other threads:[~2006-04-07 13:30 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
2006-04-06 16:53     ` Janos Farkas
2006-04-07 13:30     ` jamal [this message]
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=1144416638.5082.33.camel@jzny2 \
    --to=hadi@cyberus.ca \
    --cc=chexum+dev@gmail.com \
    --cc=ddaney@avtrex.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).