From owner-freebsd-net@FreeBSD.ORG Tue May 18 01:48:31 2004 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id A0A2A16A4CE; Tue, 18 May 2004 01:48:31 -0700 (PDT) Received: from xorpc.icir.org (xorpc.icir.org [192.150.187.68]) by mx1.FreeBSD.org (Postfix) with ESMTP id 5F89F43D54; Tue, 18 May 2004 01:48:29 -0700 (PDT) (envelope-from rizzo@icir.org) Received: from xorpc.icir.org (localhost [127.0.0.1]) by xorpc.icir.org (8.12.9p1/8.12.8) with ESMTP id i4I8mTgd002554; Tue, 18 May 2004 01:48:29 -0700 (PDT) (envelope-from rizzo@xorpc.icir.org) Received: (from rizzo@localhost) by xorpc.icir.org (8.12.9p1/8.12.3/Submit) id i4I8mSkM002553; Tue, 18 May 2004 01:48:28 -0700 (PDT) (envelope-from rizzo) Date: Tue, 18 May 2004 01:48:28 -0700 From: Luigi Rizzo To: Harti Brandt Message-ID: <20040518014828.B2380@xorpc.icir.org> References: <20040425094940.A50968@xorpc.icir.org> <200405162013.33894.dfr@nlsystems.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: ; from harti@freebsd.org on Tue, May 18, 2004 at 10:24:26AM +0200 cc: current@freebsd.org cc: net@freebsd.org Subject: Re: new arp code snapshot for review... X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 May 2004 08:48:31 -0000 I will try to remove as many assumptions as possible. thanks for the feedback. cheers luigi On Tue, May 18, 2004 at 10:24:26AM +0200, Harti Brandt wrote: > On Sun, 16 May 2004, Doug Rabson wrote: > > DR>On Sunday 25 April 2004 17:49, Luigi Rizzo wrote: > DR>> Here is a snapshot of the new arp code that i have been working on > DR>> lately, based a on Andre's ideas. (I say 'ARP' for brevity, what i > DR>> mean is the layer3-to-layer2 address translation code -- arp, aarp, > DR>> nd6 all fit in the category). > DR> > DR>Sorry for the delay but I've only just had reason to look at the arp > DR>code since I've recently been working on an implementation of rfc2734 > DR>IP over firewire. In your patch, you assume that the size of the > DR>link-level address is always six bytes. This assumption is not valid - > DR>from the looks of the existing arp code, people went to great lengths > DR>to avoid making this assumption throughout the networking code. > DR> > DR>For IP over firewire, the link-level address is sixteen bytes. Other > DR>link types have various sizes. You must use ifp->if_addrlen in the > DR>generic code to cope with this correctly. > > Sorry to step in that late, but I'm currently moving jobs. > > I had the same comment. The ARP table could be used for the various IP > over ATM address mappings. In that case it needs to support 20 byte NSAPs > with and optional subaddress (also 20 byte) and variable size E.164 > addresses. > > harti > _______________________________________________ > freebsd-current@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-current > To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"