Date: Fri, 29 Oct 2010 13:42:33 -0400 From: George Neville-Neil <gnn@neville-neil.com> To: Garrett Cooper <gcooper@freebsd.org> Cc: arch@freebsd.org Subject: Re: RFC: ARP Packet Queues Message-ID: <B96F636A-C2A3-43F9-8F4C-93889FB49C53@neville-neil.com> In-Reply-To: <AANLkTinO-vSUYLvg9R=OiZ-aYgZkFUNcAzfYf7vTBjNE@mail.gmail.com> References: <D92A9067-7305-42B6-8D15-D0A49A0542A4@neville-neil.com> <AANLkTinO-vSUYLvg9R=OiZ-aYgZkFUNcAzfYf7vTBjNE@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Oct 29, 2010, at 13:19 , Garrett Cooper wrote: > 1. This needs a tab after the #define for the macro: > > +#define V_arp_maxhold VNET(arp_maxhold) > OK. > 2. Is the naming convention for these sysctls always > net.link.ether imply arp, or is it another naming convention? I'm not quite sure what you're asking? ARP maps from inet to ether so net.link.ether.inet is where these are. Are you thinking about Neighbor Discovery and other such ARP like protocols for other, higher layer, protocols? > 3. Is there a reason why packets_dropped is a signed quantity, > i.e. int, not size_t, etc? I should fix that. > The rest looks ok, but I could be missing some context, or a > subtlety of some kind. I'll give this a shot on my new router box this > weekend because it looks interesting. Thanks. Be sure so increase maxtries as well as maxhold otherwise you'll be limited in how many packets you can hold. Best, George
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?B96F636A-C2A3-43F9-8F4C-93889FB49C53>