From owner-freebsd-current@FreeBSD.ORG Tue May 22 09:47:26 2007 Return-Path: X-Original-To: freebsd-current@freebsd.org Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 2B57216A469; Tue, 22 May 2007 09:47:26 +0000 (UTC) (envelope-from fli@FreeBSD.org) Received: from mx1.h3q.net (mx1.h3q.net [212.37.5.30]) by mx1.freebsd.org (Postfix) with ESMTP id D623E13C457; Tue, 22 May 2007 09:47:25 +0000 (UTC) (envelope-from fli@FreeBSD.org) Received: from [192.168.1.100] (81-232-22-115-no50.tbcn.telia.com [81.232.22.115]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) (Authenticated sender: fli@shapeshifter.se) by mx1.h3q.net (Postfix) with ESMTP id A693B78C20; Tue, 22 May 2007 11:47:23 +0200 (CEST) Message-ID: <4652BC26.3070403@FreeBSD.org> Date: Tue, 22 May 2007 11:47:18 +0200 From: Fredrik Lindberg User-Agent: Thunderbird 2.0.0.0 (X11/20070420) MIME-Version: 1.0 To: Andrew Thompson References: <4645E8A2.1040408@FreeBSD.org> <20070512202004.GA71624@heff.fud.org.nz> <46462CB2.9050008@FreeBSD.org> <20070521212309.GA4320@heff.fud.org.nz> In-Reply-To: <20070521212309.GA4320@heff.fud.org.nz> Content-Type: multipart/mixed; boundary="------------090602070001030503020604" Cc: yar@freebsd.org, freebsd-current@freebsd.org Subject: Re: Network interface modules keeps re-loading X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 May 2007 09:47:26 -0000 This is a multi-part message in MIME format. --------------090602070001030503020604 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Andrew Thompson wrote: > > Here is a patch to do this, i'll commit it unless there are any > objections. > > Seems fine, about the same thing I had in mind. Also, network.subr needs to be modified to use this, I believe it's only necessary to change ifexists(). That would leave module loading intact in thinks like rc.d/netif start foo0 (if that's what people want?) Fredrik --------------090602070001030503020604 Content-Type: text/plain; name="network.subr-20070522.patch" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="network.subr-20070522.patch" Index: network.subr =================================================================== RCS file: /home/ncvs/src/etc/network.subr,v retrieving revision 1.179 diff -u -r1.179 network.subr --- network.subr 29 Mar 2007 21:42:19 -0000 1.179 +++ network.subr 22 May 2007 09:44:18 -0000 @@ -265,7 +265,7 @@ # Returns 0 if the interface exists and 1 otherwise. ifexists() { - ifconfig $1 > /dev/null 2>&1 + ifconfig -n $1 > /dev/null 2>&1 } # ipv4_up if --------------090602070001030503020604--