Date: Mon, 29 Mar 2004 15:31:17 -0800
From: Andrew Morton <akpm@osdl.org>
To: Greg KH <greg@kroah.com>
Cc: stern@rowland.harvard.edu, david-b@pacbell.net, viro@math.psu.edu,
       maneesh@in.ibm.com, linux-usb-devel@lists.sourceforge.net,
       linux-kernel@vger.kernel.org
Subject: Re: Unregistering interfaces
Message-Id: <20040329153117.558c3263.akpm@osdl.org>
In-Reply-To: <20040329231604.GA29494@kroah.com>
References: <20040328063711.GA6387@kroah.com>
	<Pine.LNX.4.44L0.0403281057100.17150-100000@netrider.rowland.org>
	<20040328123857.55f04527.akpm@osdl.org>
	<20040329210219.GA16735@kroah.com>
	<20040329132551.23e12144.akpm@osdl.org>
	<20040329231604.GA29494@kroah.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1214
Lines: 30

Greg KH <greg@kroah.com> wrote:
>
> > The module should remain in memory, "unhashed", until the final kobject
> > reference falls to zero.  Destruction of that kobject causes the refcount
> > on the module to fall to zero which causes the entire module to be
> > released.
> > 
> > (hmm, the existence of a kobject doesn't appear to contribute to its
> > module's refcount.  Why not?)
> 
> It does, if a file for that kobject is opened.  In this case, there was
> no file opened, so the module refcount isn't incremented.

hm, surprised.  Shouldn't the existence of a kobject contribute to its
module's refcount?

> > Maybe a shrink_dcache_parent(dentry) on entry to simple_rmdir() would
> > suffice?
> 
> Will that get rid of the references properly nwhen we remove the
> kobject?

That's one the dcache guys could address better, but I was mainly proposing
it as a way of removing any negative dentries.  But it appears that we have
problems beyond negative dentries?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/