From: Greg KH <greg@kroah.com>
To: Miklos Szeredi <miklos@szeredi.hu>
Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org
Subject: Re: subsystem_unregister() breakage in -mm
Date: Tue, 25 Sep 2007 15:53:30 -0700 [thread overview]
Message-ID: <20070925225330.GA26235@kroah.com> (raw)
In-Reply-To: <E1IaIs6-00051t-00@dorka.pomaz.szeredi.hu>
On Wed, Sep 26, 2007 at 12:27:14AM +0200, Miklos Szeredi wrote:
> I get "BUG: failure at mm/slab.c:591/page_get_cache()!" in latest -mm
> when removing the fuse module.
>
> This patch titled "kobject: remove the static array for the name"
> looks like it's responsible. Reverting it cures the problem.
>
> The root of the problem seems to be, that decl_subsys() initializes
> k_name with a string constant. Then subsystem_unregister() will put
> the kobject, which will free the k_name.
>
> So, what's the right way to deal with this?
How is the static kobject created by decl_subsys() getting it's release
function called when it is never really "released" as it is a static
kobject?
The "correct" way to fix this is to dynamically create the subsystem
kobject, and is what I am working toward accomplishing. But I didn't
think it was necessary just yet.
Let me go dig some more...
thanks,
greg k-h
next prev parent reply other threads:[~2007-09-25 22:53 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-25 22:27 subsystem_unregister() breakage in -mm Miklos Szeredi
2007-09-25 22:53 ` Greg KH [this message]
2007-09-25 23:02 ` Miklos Szeredi
2007-09-26 0:36 ` Greg KH
2007-09-26 2:41 ` Greg KH
2007-09-26 7:21 ` Cornelia Huck
2007-09-26 15:14 ` Greg KH
2007-09-26 8:17 ` Miklos Szeredi
2007-09-26 15:15 ` Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070925225330.GA26235@kroah.com \
--to=greg@kroah.com \
--cc=akpm@linux-foundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=miklos@szeredi.hu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox