From owner-freebsd-current@FreeBSD.ORG  Tue Oct 11 23:58:59 2005
Return-Path: <owner-freebsd-current@FreeBSD.ORG>
X-Original-To: current@freebsd.org
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 3CCE316A41F;
	Tue, 11 Oct 2005 23:58:59 +0000 (GMT)
	(envelope-from thompsa@freebsd.org)
Received: from heff.fud.org.nz (60-234-149-201.bitstream.orcon.net.nz
	[60.234.149.201])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 16A8143D66;
	Tue, 11 Oct 2005 23:58:52 +0000 (GMT)
	(envelope-from thompsa@freebsd.org)
Received: by heff.fud.org.nz (Postfix, from userid 1001)
	id 3C14F1CCE5; Wed, 12 Oct 2005 12:58:49 +1300 (NZDT)
Date: Wed, 12 Oct 2005 12:58:49 +1300
From: Andrew Thompson <thompsa@freebsd.org>
To: Brooks Davis <brooks@one-eyed-alien.net>
Message-ID: <20051011235849.GB6179@heff.fud.org.nz>
Mail-Followup-To: Andrew Thompson <thompsa@freebsd.org>,
	Brooks Davis <brooks@one-eyed-alien.net>,
	Yar Tikhiy <yar@comp.chem.msu.su>,
	Pawel Jakub Dawidek <pjd@freebsd.org>,
	FreeBSD Current <current@freebsd.org>
References: <20051005203639.GA20552@garage.freebsd.pl>
	<20051005205515.GA30350@odin.ac.hmc.edu>
	<20051005210950.GB75848@heff.fud.org.nz>
	<20051009232849.GA27349@comp.chem.msu.su>
	<20051010022208.GA97249@heff.fud.org.nz>
	<20051010202900.GA24213@odin.ac.hmc.edu>
	<20051011210602.GA5714@heff.fud.org.nz>
	<20051011220749.GD13461@odin.ac.hmc.edu>
	<20051011234014.GA6179@heff.fud.org.nz>
	<20051011234551.GF13461@odin.ac.hmc.edu>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20051011234551.GF13461@odin.ac.hmc.edu>
User-Agent: Mutt/1.4.2.1i
Cc: Yar Tikhiy <yar@comp.chem.msu.su>, Pawel Jakub Dawidek <pjd@freebsd.org>,
	FreeBSD Current <current@freebsd.org>
Subject: Re: panic: ifc_free_unit: bit is already cleared
X-BeenThere: freebsd-current@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussions about the use of FreeBSD-current
	<freebsd-current.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>, 
	<mailto:freebsd-current-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-current>
List-Post: <mailto:freebsd-current@freebsd.org>
List-Help: <mailto:freebsd-current-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
	<mailto:freebsd-current-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 11 Oct 2005 23:58:59 -0000

On Tue, Oct 11, 2005 at 04:45:52PM -0700, Brooks Davis wrote:
> On Wed, Oct 12, 2005 at 12:40:14PM +1300, Andrew Thompson wrote:
> > On Tue, Oct 11, 2005 at 03:07:49PM -0700, Brooks Davis wrote:
> > > On Wed, Oct 12, 2005 at 10:06:02AM +1300, Andrew Thompson wrote:
> > > > On Mon, Oct 10, 2005 at 01:29:00PM -0700, Brooks Davis wrote:
> > > > > On Mon, Oct 10, 2005 at 03:22:08PM +1300, Andrew Thompson wrote:
> > > > > > I have been testing this patch and I think it fixes all the problems
> > > > > > discussed.
> > > > > > 
> > > > > 
> > > > > I don't see any reason why you can't just replace the specific destroy
> > > > > calls with calls to ifc_simple_destroy().  That would avoid expanding
> > > > > the API.
> > > > 
> > > > I have updated the patch and yes, its a nicer way to do it. Please
> > > > review.
> > > > 
> > > > Ive run through interations of create/kldunload with bridge, disc,
> > > > faith, gif, gre and ppp with extra printf's and its freeing correctly.
> > > 
> > > This looks good to me, thanks for working on this and doing the
> > > <ifn>_destory removals.  Let's see about getting this committed.
> > > 
> > 
> > There was one problem where pflog0 would loop on EINVAL since it was a
> > precloned device, livelocking the system.
> > 
> > This addition fixes it, it was either this or a dying flag.
> 
> Good catch.  I think this is an OK fix.  Did lo0 have the same issue?

No, as its not an unloadable module :) Just pflog and pfsync.


Andrew