From: Andreas Herrmann <andreas.herrmann@calxeda.com>
To: Christoph Lameter <cl@linux.com>
Cc: Pekka Enberg <penberg@kernel.org>, Matt Mackall <mpm@selenic.com>,
"linux-mm@kvack.org" <linux-mm@kvack.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] mm/slub: Switch slub_debug kernel option to early_param to avoid boot panic
Date: Thu, 7 Nov 2013 09:41:29 +0100 [thread overview]
Message-ID: <20131107084129.GP5661@alberich> (raw)
In-Reply-To: <20131107082732.GN5661@alberich>
On Thu, Nov 07, 2013 at 09:27:32AM +0100, Andreas Herrmann wrote:
> On Wed, Nov 06, 2013 at 04:38:10PM -0500, Christoph Lameter wrote:
> > On Wed, 6 Nov 2013, Andreas Herrmann wrote:
> >
> > > Would be nice, if your patch is pushed upstream asap.
> >
> > Ok so this is a
> >
> > Tested-by: Andreas Herrmann <andreas.herrmann@calxeda.com>
> >
> > I think?
>
> Yes.
And for sake of completeness. Here is some debug output with a kernel
that had your "slub: Handle NULL parameter in kmem_cache_flags" patch
applied. And of course there were a couple of unnamed slabs:
...
.bss : 0xc089fd80 - 0xc094cc4c ( 692 kB)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name (c06fc90c): kmem_cache_node
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name (c06fc91c): kmem_cache
a?? slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
SLUB: HWalign=64, Order=0-3, MinObjects=0, CPUs=4, Nodes=1
...
The third one is wheree the panic happened w/o the fix.
Andreas
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
WARNING: multiple messages have this Message-ID (diff)
From: Andreas Herrmann <andreas.herrmann@calxeda.com>
To: Christoph Lameter <cl@linux.com>
Cc: Pekka Enberg <penberg@kernel.org>, Matt Mackall <mpm@selenic.com>,
"linux-mm@kvack.org" <linux-mm@kvack.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] mm/slub: Switch slub_debug kernel option to early_param to avoid boot panic
Date: Thu, 7 Nov 2013 09:41:29 +0100 [thread overview]
Message-ID: <20131107084129.GP5661@alberich> (raw)
In-Reply-To: <20131107082732.GN5661@alberich>
On Thu, Nov 07, 2013 at 09:27:32AM +0100, Andreas Herrmann wrote:
> On Wed, Nov 06, 2013 at 04:38:10PM -0500, Christoph Lameter wrote:
> > On Wed, 6 Nov 2013, Andreas Herrmann wrote:
> >
> > > Would be nice, if your patch is pushed upstream asap.
> >
> > Ok so this is a
> >
> > Tested-by: Andreas Herrmann <andreas.herrmann@calxeda.com>
> >
> > I think?
>
> Yes.
And for sake of completeness. Here is some debug output with a kernel
that had your "slub: Handle NULL parameter in kmem_cache_flags" patch
applied. And of course there were a couple of unnamed slabs:
...
.bss : 0xc089fd80 - 0xc094cc4c ( 692 kB)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name (c06fc90c): kmem_cache_node
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name (c06fc91c): kmem_cache
→ slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
slub_debug_slabs (c2956a08): skbuff_fclone_cache, name ( (null)): (null)
SLUB: HWalign=64, Order=0-3, MinObjects=0, CPUs=4, Nodes=1
...
The third one is wheree the panic happened w/o the fix.
Andreas
next prev parent reply other threads:[~2013-11-07 8:41 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-06 18:45 [PATCH] mm/slub: Switch slub_debug kernel option to early_param to avoid boot panic Andreas Herrmann
2013-11-06 18:45 ` Andreas Herrmann
2013-11-06 19:16 ` Christoph Lameter
2013-11-06 19:16 ` Christoph Lameter
2013-11-06 19:21 ` Andreas Herrmann
2013-11-06 19:21 ` Andreas Herrmann
2013-11-06 19:54 ` Andreas Herrmann
2013-11-06 19:54 ` Andreas Herrmann
2013-11-06 20:34 ` Andreas Herrmann
2013-11-06 20:34 ` Andreas Herrmann
2013-11-06 21:16 ` Andreas Herrmann
2013-11-06 21:16 ` Andreas Herrmann
2013-11-06 21:38 ` Christoph Lameter
2013-11-06 21:38 ` Christoph Lameter
2013-11-07 8:27 ` Andreas Herrmann
2013-11-07 8:27 ` Andreas Herrmann
2013-11-07 8:41 ` Andreas Herrmann [this message]
2013-11-07 8:41 ` Andreas Herrmann
2013-11-07 16:09 ` Christoph Lameter
2013-11-07 16:09 ` Christoph Lameter
[not found] ` <alpine.DEB.2.02.1311071008010.22533@gentwo.org>
2013-11-07 16:29 ` Christoph Lameter
2013-11-07 16:29 ` Christoph Lameter
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=20131107084129.GP5661@alberich \
--to=andreas.herrmann@calxeda.com \
--cc=cl@linux.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mpm@selenic.com \
--cc=penberg@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.