From owner-freebsd-bugs@FreeBSD.ORG Mon Mar 8 16:50:14 2004 Return-Path: Delivered-To: freebsd-bugs@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id E55D316A4CE for ; Mon, 8 Mar 2004 16:50:14 -0800 (PST) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id E0E9C43D1F for ; Mon, 8 Mar 2004 16:50:14 -0800 (PST) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) i290oEbv015004 for ; Mon, 8 Mar 2004 16:50:14 -0800 (PST) (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.12.10/8.12.10/Submit) id i290oEuG015002; Mon, 8 Mar 2004 16:50:14 -0800 (PST) (envelope-from gnats) Date: Mon, 8 Mar 2004 16:50:14 -0800 (PST) Message-Id: <200403090050.i290oEuG015002@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org From: Stefan `Sec` Zehl Subject: Re: kern/63772: tap device / exclusive open problem X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list Reply-To: Stefan `Sec` Zehl List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 09 Mar 2004 00:50:15 -0000 The following reply was made to PR kern/63772; it has been noted by GNATS. From: Stefan `Sec` Zehl To: Brooks Davis Cc: FreeBSD-gnats-submit@freebsd.org, freebsd-bugs@freebsd.org Subject: Re: kern/63772: tap device / exclusive open problem Date: Tue, 9 Mar 2004 01:46:45 +0100 On Fri, Mar 05, 2004 at 10:55 -0800, Brooks Davis wrote: [...] > > KASSERT(!(tp->tap_flags & TAP_OPEN), > > ("%s flags is out of sync", tp->tap_if.if_xname)); > > > > + if (tp->tap_flags & TAP_OPEN) > > + return (EBUSY); > > + [...] > There's obviously something complicated going on since with INVARIANTS > enabled this would either return EBUSY or panic. This should definitely > not be committed as is. I must admit, I don't understand the problem. this is an really simple if. Nothing is modified, and tp is not NULL, or the KASSERT above would be broken already. How do you get it to panic? It works fine for me. CU, Sec -- In 1968 it took the computing-Power of 2 C-64 to fly a rocket to the moon. Now, 1997 it takes the Power of a Pentium 133 to run Microsoft Windows 95. Something must have gone wrong.