From owner-cvs-src@FreeBSD.ORG Fri Nov 14 13:27:17 2003 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 C04C116A4CE; Fri, 14 Nov 2003 13:27:17 -0800 (PST) Received: from xorpc.icir.org (xorpc.icir.org [192.150.187.68]) by mx1.FreeBSD.org (Postfix) with ESMTP id 894B843FB1; Fri, 14 Nov 2003 13:27:16 -0800 (PST) (envelope-from rizzo@xorpc.icir.org) Received: from xorpc.icir.org (localhost [127.0.0.1]) by xorpc.icir.org (8.12.9p1/8.12.3) with ESMTP id hAELRFFw018187; Fri, 14 Nov 2003 13:27:15 -0800 (PST) (envelope-from rizzo@xorpc.icir.org) Received: (from rizzo@localhost) by xorpc.icir.org (8.12.9p1/8.12.3/Submit) id hAELRFIx018186; Fri, 14 Nov 2003 13:27:15 -0800 (PST) (envelope-from rizzo) Date: Fri, 14 Nov 2003 13:27:14 -0800 From: Luigi Rizzo To: Sam Leffler Message-ID: <20031114132714.A88606@xorpc.icir.org> References: <200311141900.hAEJ0XBn058450@repoman.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: <200311141900.hAEJ0XBn058450@repoman.freebsd.org>; from sam@FreeBSD.org on Fri, Nov 14, 2003 at 11:00:33AM -0800 cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/an if_an.c if_anreg.h src/sys/dev/bfe if_bfe.c src/sys/dev/my if_my.c src/sys/dev/owi if_owi.c if_wivar.h src/sys/dev/re if_re.c src/sys/dev/wl if_wl.c src/sys/pci if_dc.c if_dcreg.h if_pcn.c if_pcnreg.h if_rl.c ... 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: Fri, 14 Nov 2003 21:27:18 -0000 On Fri, Nov 14, 2003 at 11:00:33AM -0800, Sam Leffler wrote: > sam 2003/11/14 11:00:33 PST ... > Log: > Drop the driver lock around calls to if_input to avoid a LOR when > the packets are immediately returned for sending (e.g. when bridging > or packet forwarding). There are more efficient ways to do this > but for now use the least intrusive approach. the number of places that this commit had to touch makes me wonder whether it wouldn't be better to include somehow these calls into the if_input routine somehow... cheers luigi