From owner-freebsd-net@FreeBSD.ORG Tue May 18 01:25:29 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 99B0716A4CE; Tue, 18 May 2004 01:25:29 -0700 (PDT) Received: from n33.kp.t-systems-sfr.com (n33.kp.t-systems-sfr.com [129.247.16.33]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9021D43D5A; Tue, 18 May 2004 01:25:27 -0700 (PDT) (envelope-from harti@freebsd.org) Received: from n81.sp.op.dlr.de (n81g.sp.op.dlr.de [129.247.163.1]) i4I8OUj47378; Tue, 18 May 2004 10:24:31 +0200 Received: from zeus.nt.op.dlr.de (zeus.nt.op.dlr.de [129.247.173.3]) by n81.sp.op.dlr.de (AIX5.1/8.11.6p2/8.11.0) with ESMTP id i4I8OUK73220; Tue, 18 May 2004 10:24:30 +0200 Received: from localhost (brandt@localhost) by zeus.nt.op.dlr.de (8.11.7+Sun/8.9.1) with ESMTP id i4I8OSG08259; Tue, 18 May 2004 10:24:28 +0200 (MET DST) X-Authentication-Warning: zeus.nt.op.dlr.de: brandt owned process doing -bs Date: Tue, 18 May 2004 10:24:26 +0200 (MET DST) From: Harti Brandt X-X-Sender: brandt@zeus To: Luigi Rizzo In-Reply-To: <200405162013.33894.dfr@nlsystems.com> Message-ID: References: <20040425094940.A50968@xorpc.icir.org> <200405162013.33894.dfr@nlsystems.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: Doug Rabson 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 Reply-To: Harti Brandt 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:25:29 -0000 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