From: Al Viro <viro@ZenIV.linux.org.uk>
To: Christoph Lameter <clameter@sgi.com>
Cc: Steven Rostedt <rostedt@goodmis.org>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
LKML <linux-kernel@vger.kernel.org>,
"Rafael J. Wysocki" <rjw@sisk.pl>
Subject: Re: SLUB sysfs support
Date: Fri, 28 Dec 2007 00:45:04 +0000 [thread overview]
Message-ID: <20071228004504.GK27894@ZenIV.linux.org.uk> (raw)
In-Reply-To: <Pine.LNX.4.64.0712271601200.1294@schroedinger.engr.sgi.com>
On Thu, Dec 27, 2007 at 04:02:56PM -0800, Christoph Lameter wrote:
> > Why would you want these symlinks to stick around for longer than that?
>
> /sys/kernel/slab/cache_name is a real directory but then there are the
> aliases in /sys/kernel/slab/alias_name pointing to that directory that
> also have to be removed.
>
> So I need to scan for symlinks in /sys/kernel/slab/* pointing to
> /sys/kernel/slab/cache_name.
Oh, lovely. So we can have module A do kmem_cache_create(), calling
cache "foo". Then module B does (without any knowlegde about A)
completely unrelated kmem_cache_create(), calling the sucker "bar".
mm/slub decides that they are mergable. Then we get rmmod A... and
no way to find out if that's foo or bar going away - kmem_cache_destroy()
doesn't have enough information to figure that out. So we have to keep
both slab/foo and slab/bar around until both are gone or until somebody
kind enough creates a cache called foo. Better yet, on some systems you
get things like slab/nfsd4_delegations sticking around long after nfsd is
gone just because slub.c decides that it's sharable, but in the next
kernel version the thing it's shared with gets different object size and
behaviour suddenly changes - now it's gone as soon as nfsd is gone.
Brilliant interface, that...
next prev parent reply other threads:[~2007-12-28 0:45 UTC|newest]
Thread overview: 93+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-12-07 17:50 Major regression on hackbench with SLUB Steven Rostedt
2007-12-07 17:55 ` Ingo Molnar
2007-12-07 17:59 ` Linus Torvalds
2007-12-07 18:36 ` Linus Torvalds
2007-12-07 18:58 ` Steven Rostedt
2007-12-08 22:16 ` Steven Rostedt
2007-12-10 7:38 ` Björn Steinbrink
2007-12-10 14:00 ` Steven Rostedt
2007-12-09 0:28 ` Steven Rostedt
2007-12-13 22:11 ` Christoph Lameter
2007-12-11 14:33 ` Major regression on hackbench with SLUB (more numbers) Ingo Molnar
2007-12-13 22:22 ` Christoph Lameter
2007-12-14 4:24 ` Christoph Lameter
2007-12-14 6:15 ` Steven Rostedt
2007-12-21 12:09 ` Ingo Molnar
2007-12-21 12:26 ` Ingo Molnar
2007-12-21 16:26 ` Christoph Lameter
2007-12-21 16:33 ` Ingo Molnar
2007-12-21 21:56 ` Christoph Lameter
2007-12-21 16:18 ` Christoph Lameter
2007-12-21 16:44 ` Linus Torvalds
2007-12-21 22:11 ` Christoph Lameter
2007-12-21 22:16 ` Peter Zijlstra
2007-12-21 22:17 ` Peter Zijlstra
2007-12-21 22:27 ` Christoph Lameter
2007-12-21 22:54 ` Ingo Molnar
2007-12-21 23:20 ` David Miller
2007-12-22 9:45 ` Ingo Molnar
2007-12-24 19:24 ` Christoph Lameter
2007-12-21 23:27 ` Andrew Morton
2007-12-21 23:51 ` Christoph Lameter
2007-12-22 0:28 ` Andi Kleen
2007-12-22 0:33 ` Chuck Ebbert
2007-12-22 2:19 ` Matt Mackall
2007-12-22 2:44 ` Andi Kleen
2007-12-22 10:03 ` Ingo Molnar
2007-12-22 12:37 ` Andi Kleen
2007-12-22 12:51 ` Pekka Enberg
2007-12-22 12:54 ` Andi Kleen
2007-12-22 13:27 ` Pekka Enberg
2007-12-22 13:01 ` Alexey Dobriyan
2007-12-22 18:01 ` Linus Torvalds
2007-12-22 19:09 ` Ingo Molnar
2007-12-22 19:29 ` Ingo Molnar
2007-12-22 22:52 ` Jason L Tibbitts III
2007-12-24 3:59 ` Alessandro Suardi
2007-12-22 19:25 ` Theodore Tso
2007-12-22 21:00 ` Linus Torvalds
2007-12-22 21:50 ` Al Viro
2007-12-22 23:29 ` Al Viro
2007-12-22 22:10 ` Willy Tarreau
2007-12-23 0:59 ` Steven Rostedt
2007-12-23 5:12 ` Willy Tarreau
2007-12-23 14:15 ` Andi Kleen
2007-12-24 3:45 ` Theodore Tso
2007-12-24 19:21 ` Christoph Lameter
2007-12-24 23:37 ` Theodore Tso
2007-12-25 3:34 ` Andi Kleen
2008-01-01 12:47 ` Theodore Tso
2008-01-01 15:19 ` Andi Kleen
2008-01-01 16:23 ` [patch] slub: provide /proc/slabinfo Ingo Molnar
2008-01-05 0:27 ` Arjan van de Ven
2008-01-05 0:49 ` Linus Torvalds
2007-12-26 21:31 ` Major regression on hackbench with SLUB (more numbers) Christoph Lameter
2007-12-26 22:16 ` Al Viro
2007-12-27 5:51 ` Christoph Lameter
2007-12-27 20:28 ` SLUB sysfs support Christoph Lameter
2007-12-27 22:59 ` Al Viro
2007-12-27 23:22 ` Christoph Lameter
2007-12-27 23:53 ` Christoph Lameter
2007-12-27 23:58 ` Al Viro
2007-12-28 0:02 ` Christoph Lameter
2007-12-28 0:45 ` Al Viro [this message]
2007-12-28 2:19 ` Christoph Lameter
2007-12-28 3:26 ` Al Viro
2008-01-02 20:25 ` Christoph Lameter
2007-12-27 23:54 ` Al Viro
2007-12-28 9:00 ` Major regression on hackbench with SLUB (more numbers) Ingo Molnar
2007-12-28 14:52 ` Steven Rostedt
2007-12-22 19:46 ` slabtop replacement was " Andi Kleen
2007-12-22 23:28 ` Karol Swietlicki
2007-12-29 18:08 ` Karol Swietlicki
2007-12-21 22:49 ` Linus Torvalds
2007-12-21 17:44 ` Pekka Enberg
2007-12-21 22:22 ` Christoph Lameter
2007-12-21 22:37 ` Christoph Lameter
2007-12-21 22:51 ` Linus Torvalds
2007-12-21 23:17 ` Ingo Molnar
2007-12-21 23:40 ` Pekka Enberg
2007-12-21 23:54 ` Christoph Lameter
2007-12-22 0:02 ` Chuck Ebbert
2007-12-21 22:52 ` Steven Rostedt
2007-12-21 22:56 ` Ingo Molnar
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=20071228004504.GK27894@ZenIV.linux.org.uk \
--to=viro@zeniv.linux.org.uk \
--cc=a.p.zijlstra@chello.nl \
--cc=clameter@sgi.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rjw@sisk.pl \
--cc=rostedt@goodmis.org \
/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