From owner-cvs-src@FreeBSD.ORG Sat May 8 10:15:01 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C24A716A4CE; Sat, 8 May 2004 10:15:01 -0700 (PDT) Received: from xorpc.icir.org (xorpc.icir.org [192.150.187.68]) by mx1.FreeBSD.org (Postfix) with ESMTP id 8339D43D39; Sat, 8 May 2004 10:15:01 -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 i48HExgd099372; Sat, 8 May 2004 10:14:59 -0700 (PDT) (envelope-from rizzo@xorpc.icir.org) Received: (from rizzo@localhost) by xorpc.icir.org (8.12.9p1/8.12.3/Submit) id i48HExG6099371; Sat, 8 May 2004 10:14:59 -0700 (PDT) (envelope-from rizzo) Date: Sat, 8 May 2004 10:14:59 -0700 From: Luigi Rizzo To: Darren Reed Message-ID: <20040508101459.A98855@xorpc.icir.org> References: <200405061846.i46Ik3Jc060969@repoman.freebsd.org> <20040506185854.GB1777@madman.celabo.org> <20040507072031.GA48708@hub.freebsd.org> <200405070755.36055.sam@errno.com> <20040508152531.GA96827@hub.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <20040508152531.GA96827@hub.freebsd.org>; from darrenr@hub.freebsd.org on Sat, May 08, 2004 at 08:25:31AM -0700 cc: src-committers@FreeBSD.org cc: Andre Oppermann cc: "Jacques A. Vidrine" cc: cvs-src@FreeBSD.org cc: cvs-all@FreeBSD.org cc: Sam Leffler Subject: Re: cvs commit: src/sys/netinet ip_fastfwd.c ip_input.c ip_var.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 08 May 2004 17:15:01 -0000 On the principle, I tend to agree with Darren here... it is not nice to replicate functionality in multiple places by using specialized code instead of relying on (and possibly optimizing) the generic one. It makes a lot harder to clean up the replication later, and i believe Andre knows that quite well given the cleanup work he has done in the past in the network stack. I don't think it is worth making a bit fuss about this particular change, but certainly, as a general principle, we should try as much as possible to use the generic mechanisms when available -- especialliy given that performance killers are elsewhere (locking etc.). cheers luigi On Sat, May 08, 2004 at 08:25:31AM -0700, Darren Reed wrote: > On Fri, May 07, 2004 at 07:55:36AM -0700, Sam Leffler wrote: > > > > Employing a packet filter is not equivalent as it requires every packet to be > > processed while this (effectively 7-line change) adds no new overhead to the > > normal processing path for packets. It would be nice if packet filtering > > were cheap enough that we could use it in this way but I don't think that's > > the case just yet. > > Using that argument, is that clearance to put all of the normalization > from pf into the various parts of the networking code (not every type of > normalisation needs to be done on every packet but it is all useful), with > sysctls to turn it on or off, and maybe we'll add the ability to log packets > at various points because we don't want the overhead of BPF (it has to > process every packet too) and that's just for starters. I'm sure I can > think of some more, in time. How about you? > > If there were a core@ for freebsd that was active, this is the kind of > thing I'd be writing to them about, asking for it to be backed out. > > Darren