From owner-svn-src-all@FreeBSD.ORG  Thu Dec 25 22:32:32 2008
Return-Path: <owner-svn-src-all@FreeBSD.ORG>
Delivered-To: svn-src-all@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id D5ED7106564A;
	Thu, 25 Dec 2008 22:32:32 +0000 (UTC)
	(envelope-from qingli@FreeBSD.org)
Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:4f8:fff6::2c])
	by mx1.freebsd.org (Postfix) with ESMTP id C291D8FC12;
	Thu, 25 Dec 2008 22:32:32 +0000 (UTC)
	(envelope-from qingli@FreeBSD.org)
Received: from svn.freebsd.org (localhost [127.0.0.1])
	by svn.freebsd.org (8.14.3/8.14.3) with ESMTP id mBPMWWKd024607;
	Thu, 25 Dec 2008 22:32:32 GMT (envelope-from qingli@svn.freebsd.org)
Received: (from qingli@localhost)
	by svn.freebsd.org (8.14.3/8.14.3/Submit) id mBPMWWtH024606;
	Thu, 25 Dec 2008 22:32:32 GMT (envelope-from qingli@svn.freebsd.org)
Message-Id: <200812252232.mBPMWWtH024606@svn.freebsd.org>
From: Qing Li <qingli@FreeBSD.org>
Date: Thu, 25 Dec 2008 22:32:32 +0000 (UTC)
To: src-committers@freebsd.org, svn-src-all@freebsd.org,
	svn-src-head@freebsd.org
X-SVN-Group: head
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Cc: 
Subject: svn commit: r186497 - head/sys/net
X-BeenThere: svn-src-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: "SVN commit messages for the entire src tree \(except for &quot;
	user&quot; and &quot; projects&quot; \)" <svn-src-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-all>,
	<mailto:svn-src-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-all>
List-Post: <mailto:svn-src-all@freebsd.org>
List-Help: <mailto:svn-src-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-all>,
	<mailto:svn-src-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 25 Dec 2008 22:32:32 -0000

Author: qingli
Date: Thu Dec 25 22:32:32 2008
New Revision: 186497
URL: http://svn.freebsd.org/changeset/base/186497

Log:
  The "tun?" dev need not be opened at all. One is allowed to perform
  the following operations, e.g.:
  1) ifconfig tun0 create
  2) ifconfig tun0 10.1.1.1 10.1.1.2
  3) route add -net 192.103.54.0/24 -iface tun0
  4) ifconfig tun0 destroy
  If cv wait on the TUN_CLOSED flag, then the last operation (4) will
  block forever.
  
  Revert the previous changes and fix the mtx_unlock() leak.

Modified:
  head/sys/net/if_tun.c

Modified: head/sys/net/if_tun.c
==============================================================================
--- head/sys/net/if_tun.c	Thu Dec 25 16:59:35 2008	(r186496)
+++ head/sys/net/if_tun.c	Thu Dec 25 22:32:32 2008	(r186497)
@@ -79,7 +79,6 @@ struct tun_softc {
 #define	TUN_RWAIT	0x0040
 #define	TUN_ASYNC	0x0080
 #define	TUN_IFHEAD	0x0100
-#define	TUN_CLOSED	0x0200
 
 #define TUN_READY       (TUN_OPEN | TUN_INITED)
 
@@ -257,11 +256,11 @@ tun_destroy(struct tun_softc *tp)
 
 	/* Unlocked read. */
 	mtx_lock(&tp->tun_mtx);
-	if ((tp->tun_flags & (TUN_OPEN|TUN_CLOSED)) != TUN_CLOSED)
+	if ((tp->tun_flags & TUN_OPEN) != 0)
 		cv_wait_unlock(&tp->tun_cv, &tp->tun_mtx);
 	else
 		mtx_unlock(&tp->tun_mtx);
-	    
+
 	CURVNET_SET(TUN2IFP(tp)->if_vnet);
 	dev = tp->tun_dev;
 	bpfdetach(TUN2IFP(tp));
@@ -500,7 +499,6 @@ tunclose(struct cdev *dev, int foo, int 
 	KNOTE_UNLOCKED(&tp->tun_rsel.si_note, 0);
 	TUNDEBUG (ifp, "closed\n");
 
-	tp->tun_flags |= TUN_CLOSED;
 	cv_broadcast(&tp->tun_cv);
 	mtx_unlock(&tp->tun_mtx);
 	return (0);