From owner-freebsd-current@FreeBSD.ORG Sun Sep 28 22:04:46 2003 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8739616A4B3 for ; Sun, 28 Sep 2003 22:04:46 -0700 (PDT) Received: from odin.ac.hmc.edu (Odin.AC.HMC.Edu [134.173.32.75]) by mx1.FreeBSD.org (Postfix) with ESMTP id A52684403F for ; Sun, 28 Sep 2003 22:04:45 -0700 (PDT) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (IDENT:brdavis@localhost.localdomain [127.0.0.1]) by odin.ac.hmc.edu (8.12.9/8.12.3) with ESMTP id h8T54hgg013549; Sun, 28 Sep 2003 22:04:43 -0700 Received: (from brdavis@localhost) by odin.ac.hmc.edu (8.12.9/8.12.3/Submit) id h8T54hrE013548; Sun, 28 Sep 2003 22:04:43 -0700 Date: Sun, 28 Sep 2003 22:04:42 -0700 From: Brooks Davis To: Poul-Henning Kamp Message-ID: <20030929050442.GA20995@Odin.AC.HMC.Edu> References: <92028.1064699839@critter.freebsd.dk> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="YZ5djTAD1cGYuMQK" Content-Disposition: inline In-Reply-To: <92028.1064699839@critter.freebsd.dk> User-Agent: Mutt/1.5.4i X-Virus-Scanned: by amavisd-milter (http://amavis.org/) on odin.ac.hmc.edu cc: current@freebsd.org Subject: Re: TEST PLEASE: if_tun patch X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 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: Mon, 29 Sep 2003 05:04:46 -0000 --YZ5djTAD1cGYuMQK Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Sep 27, 2003 at 11:57:19PM +0200, Poul-Henning Kamp wrote: >=20 > Please test this patch: >=20 > http://phk.freebsd.dk/patch/if_tun.patch >=20 > There is a slight change in semantics in that the interface will disappear > entirely when the /dev/tun%d device is closed. >=20 > If no objections this will be committed in some days. > | Properly dismantle and remove the interface and destroy the dev_t=20 > | at last close of the device. I'm not convinced this is the right direction to move in. The problem is that users are beginning to expect that pseudo-interfaces be created with network interface cloning, but tun, tap, and vmnet aren't. I'm concerned that this destroy on last close semantic will make it harder to implement that. I guess if we moved to a model where we allowed both methods to work, we could somehow add an extra reference when we cloned interfaces via "ifconfig create", but I don't know the devfs side well enough to know if that's possible. This doesn't constitute an objection since I've had over a year to fix these drivers, but I'd appreciate it if you would give it some thought. -- Brooks P.S. Since you're moving it anyway, we make the assumption in the network code that the ifnet struct is the first member of the softc so you might want to fix tun while you're there. See the comment around if_var.h:111. --=20 Any statement of the form "X is the one, true Y" is FALSE. PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4 --YZ5djTAD1cGYuMQK Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (GNU/Linux) iD8DBQE/d71pXY6L6fI4GtQRAvmcAJsF0CzrHrC1Z/fYAHX5ew/K2BiPwwCgr4xL 10UJaIFY1UFVmz50klFZnCA= =PnEq -----END PGP SIGNATURE----- --YZ5djTAD1cGYuMQK--