All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Pekka Enberg <penberg@kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>,
	Christoph Lameter <cl@linux.com>,
	torvalds@linux-foundation.org, akpm@linux-foundation.org,
	tj@kernel.org, npiggin@kernel.dk, rientjes@google.com,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org
Subject: [PATCH] slub: Disable the lockless allocator
Date: Sat, 26 Mar 2011 12:47:36 +0100	[thread overview]
Message-ID: <20110326114736.GA8251@elte.hu> (raw)
In-Reply-To: <20110326112725.GA28612@elte.hu>


The commit below solves this crash for me. Could we please apply this simple 
patch, until the real bug has been found, to keep upstream debuggable? The 
eventual fix can then re-enable the lockless allocator.

Thanks,

	Ingo

--------------->
>From bb764182707216faeb815c3bbdf6a9e9992a459a Mon Sep 17 00:00:00 2001
From: Ingo Molnar <mingo@elte.hu>
Date: Sat, 26 Mar 2011 12:40:30 +0100
Subject: [PATCH] slub: Disable the lockless allocator

This boot crash:

Inode-cache hash table entries: 65536 (order: 7, 524288 bytes)
Memory: 1011352k/1048512k available (11595k kernel code, 452k absent, 36708k reserved, 6343k data, 1024k init)
BUG: unable to handle kernel paging request at ffff87ffc1fdd020
IP: [<ffffffff812b50c2>] this_cpu_cmpxchg16b_emu+0x2/0x1c
PGD 0
Oops: 0000 [#1]
last sysfs file:
CPU 0
Pid: 0, comm: swapper Not tainted 2.6.38-08569-g16c29da #110593 System manufacturer System Product Name/A8N-E
RIP: 0010:[<ffffffff812b50c2>]  [<ffffffff812b50c2>] this_cpu_cmpxchg16b_emu+0x2/0x1c
RSP: 0000:ffffffff82003e78  EFLAGS: 00010086
RAX: ffff88003f8020c0 RBX: ffff88003f802180 RCX: 0000000000000002
RDX: 0000000000000001 RSI: ffff88003ffc2020 RDI: ffffffff8219be43
RBP: ffffffff82003ed8 R08: 0000000000000000 R09: ffffea0000de40d0
R10: ffffffff821666b0 R11: ffffffff821666b0 R12: ffff88003f8020c0
R13: ffff88003f802000 R14: ffffffffffffffff R15: 0000000000000000
FS:  0000000000000000(0000) GS:ffffffff8201b000(0000) knlGS:0000000000000000
CS:  0010 DS: 0000 ES: 0000 CR0: 000000008005003b
CR2: ffff87ffc1fdd020 CR3: 000000000200b000 CR4: 00000000000006b0
DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
Process swapper (pid: 0, threadinfo ffffffff82002000, task ffffffff82015300)
Stack:
 0000000000000086 ffffffff810dcd03 ffffffffffffffff 0000000000093e30
 ffffffff82003ef8 0000000000000000 00000000000000c0 0000000000000000
 ffff88003f800000 ffff88003f8000c0 ffffffffffffffff 0000000000093e30
Call Trace:
 [<ffffffff810dcd03>] ? kmem_cache_alloc+0x53/0x130
 [<ffffffff8219be43>] kmem_cache_init+0xeb/0x2b0
 [<ffffffff821869eb>] start_kernel+0x1cb/0x322
 [<ffffffff82186338>] x86_64_start_reservations+0xfe/0x102
 [<ffffffff82186140>] ? early_idt_handlers+0x140/0x140
 [<ffffffff82186409>] x86_64_start_kernel+0xcd/0xdc
Code: 47 08 48 89 47 10 48 89 47 18 48 89 47 20 48 89 47 28 48 89 47 30 48 89 47 38 48 8d 7f 40 75 d9 90 c3 90 90 90 90 90 90 90 9c fa
 48 3b 06 75 14 65 48 3b 56 08 75 0d 65 48 89 1e 65 48 89 4e
RIP  [<ffffffff812b50c2>] this_cpu_cmpxchg16b_emu+0x2/0x1c
 RSP <ffffffff82003e78>
CR2: ffff87ffc1fdd020
---[ end trace a7919e7f17c0a725 ]---

Occurs due to:

  8a5ec0ba42c4: Lockless (and preemptless) fastpaths for slub

Disable this code for now, it's not yet fully cooked.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
 mm/slub.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/mm/slub.c b/mm/slub.c
index f881874..37ccf77 100644
--- a/mm/slub.c
+++ b/mm/slub.c
@@ -30,6 +30,7 @@
 
 #include <trace/events/kmem.h>
 
+#undef CONFIG_CMPXCHG_LOCAL
 /*
  * Lock order:
  *   1. slab_lock(page)

WARNING: multiple messages have this Message-ID (diff)
From: Ingo Molnar <mingo@elte.hu>
To: Pekka Enberg <penberg@kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>,
	Christoph Lameter <cl@linux.com>,
	torvalds@linux-foundation.org, akpm@linux-foundation.org,
	tj@kernel.org, npiggin@kernel.dk, rientjes@google.com,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org
Subject: [PATCH] slub: Disable the lockless allocator
Date: Sat, 26 Mar 2011 12:47:36 +0100	[thread overview]
Message-ID: <20110326114736.GA8251@elte.hu> (raw)
In-Reply-To: <20110326112725.GA28612@elte.hu>


The commit below solves this crash for me. Could we please apply this simple 
patch, until the real bug has been found, to keep upstream debuggable? The 
eventual fix can then re-enable the lockless allocator.

Thanks,

	Ingo

--------------->

  reply	other threads:[~2011-03-26 11:47 UTC|newest]

Thread overview: 134+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-22 14:40 [GIT PULL] SLAB changes for v2.6.39-rc1 Pekka Enberg
2011-03-22 14:40 ` Pekka Enberg
2011-03-24 14:21 ` Ingo Molnar
2011-03-24 14:41   ` Christoph Lameter
2011-03-24 16:50     ` Pekka Enberg
2011-03-24 17:26       ` Ingo Molnar
2011-03-24 17:26         ` Ingo Molnar
2011-03-24 17:43         ` Christoph Lameter
2011-03-24 17:43           ` Christoph Lameter
2011-03-24 17:56           ` Pekka Enberg
2011-03-24 17:56             ` Pekka Enberg
2011-03-24 18:01             ` Christoph Lameter
2011-03-24 18:01               ` Christoph Lameter
2011-03-24 18:06               ` Pekka Enberg
2011-03-24 18:06                 ` Pekka Enberg
2011-03-24 18:15                 ` Christoph Lameter
2011-03-24 18:15                   ` Christoph Lameter
2011-03-24 18:20                   ` Eric Dumazet
2011-03-24 18:20                     ` Eric Dumazet
2011-03-24 18:47                     ` Christoph Lameter
2011-03-24 18:47                       ` Christoph Lameter
2011-03-24 18:51                       ` Eric Dumazet
2011-03-24 18:51                         ` Eric Dumazet
2011-03-24 19:02                         ` Christoph Lameter
2011-03-24 19:04                           ` Pekka Enberg
2011-03-24 19:04                             ` Pekka Enberg
2011-03-24 18:55                       ` Pekka Enberg
2011-03-24 18:55                         ` Pekka Enberg
2011-03-24 18:59                         ` Ingo Molnar
2011-03-24 18:59                           ` Ingo Molnar
2011-03-24 19:03                           ` Pekka Enberg
2011-03-24 19:03                             ` Pekka Enberg
2011-03-24 19:05                             ` Christoph Lameter
2011-03-24 19:05                               ` Christoph Lameter
2011-03-24 19:14                               ` Pekka Enberg
2011-03-24 19:14                                 ` Pekka Enberg
2011-03-24 19:36                                 ` Ingo Molnar
2011-03-24 19:36                                   ` Ingo Molnar
2011-03-24 19:41                                   ` Pekka Enberg
2011-03-24 19:41                                     ` Pekka Enberg
2011-03-24 19:51                                     ` Christoph Lameter
2011-03-24 19:51                                       ` Christoph Lameter
2011-03-24 20:01                                       ` Ingo Molnar
2011-03-24 20:01                                         ` Ingo Molnar
2011-03-24 20:08                                       ` Eric Dumazet
2011-03-24 20:08                                         ` Eric Dumazet
2011-03-24 20:43                                         ` Christoph Lameter
2011-03-24 20:43                                           ` Christoph Lameter
2011-03-25 15:13                                           ` Tejun Heo
2011-03-25 15:13                                             ` Tejun Heo
2011-03-25 15:45                                             ` Christoph Lameter
2011-03-25 15:45                                               ` Christoph Lameter
2011-03-27  9:59                                       ` Avi Kivity
2011-03-27  9:59                                         ` Avi Kivity
2011-03-24 19:02                         ` Christoph Lameter
2011-03-24 19:02                           ` Christoph Lameter
2011-03-27  9:57                       ` Avi Kivity
2011-03-27  9:57                         ` Avi Kivity
2011-03-24 18:13         ` Pekka Enberg
2011-03-24 18:13           ` Pekka Enberg
2011-03-24 19:27           ` Ingo Molnar
2011-03-24 19:27             ` Ingo Molnar
2011-03-24 19:29             ` Pekka Enberg
2011-03-24 19:29               ` Pekka Enberg
2011-03-24 18:52         ` [boot crash #2] " Ingo Molnar
2011-03-24 18:52           ` Ingo Molnar
2011-03-24 19:10           ` Thomas Gleixner
2011-03-24 19:10             ` Thomas Gleixner
2011-03-24 19:22             ` Ingo Molnar
2011-03-24 19:22               ` Ingo Molnar
2011-03-24 19:27               ` Pekka Enberg
2011-03-24 19:27                 ` Pekka Enberg
2011-03-25 20:00                 ` Ingo Molnar
2011-03-25 20:00                   ` Ingo Molnar
2011-03-26 11:27                 ` Ingo Molnar
2011-03-26 11:47                   ` Ingo Molnar [this message]
2011-03-26 11:47                     ` [PATCH] slub: Disable the lockless allocator Ingo Molnar
2011-03-26 17:45                     ` Eric Dumazet
2011-03-26 17:45                       ` Eric Dumazet
2011-03-26 19:18                       ` Christoph Lameter
2011-03-26 19:18                         ` Christoph Lameter
2011-03-26 19:30                         ` Christoph Lameter
2011-03-26 19:30                           ` Christoph Lameter
2011-03-26 19:49                           ` Christoph Lameter
2011-03-26 19:49                             ` Christoph Lameter
2011-03-26 19:57                             ` Ingo Molnar
2011-03-26 19:57                               ` Ingo Molnar
2011-03-26 21:36                               ` Christoph Lameter
2011-03-26 21:36                                 ` Christoph Lameter
2011-03-26 20:00                             ` Ingo Molnar
2011-03-26 20:00                               ` Ingo Molnar
2011-03-26 22:18                             ` Linus Torvalds
2011-03-27  1:29                               ` Christoph Lameter
2011-03-27  1:57                                 ` Christoph Lameter
2011-03-27  7:52                                   ` Eric Dumazet
2011-03-27  7:52                                     ` Eric Dumazet
2011-03-28 10:32                                     ` [PATCH] percpu: avoid extra NOP in percpu_cmpxchg16b_double Eric Dumazet
2011-03-28 10:32                                       ` Eric Dumazet
2011-03-28 13:46                                       ` Christoph Lameter
2011-03-28 13:46                                         ` Christoph Lameter
2011-03-28 16:06                                         ` Tejun Heo
2011-03-28 16:06                                           ` Tejun Heo
2011-03-28  6:12                                   ` [PATCH] slub: Disable the lockless allocator Pekka Enberg
2011-03-28  6:12                                     ` Pekka Enberg
2011-03-28  6:19                                     ` Ingo Molnar
2011-03-28  6:19                                       ` Ingo Molnar
2011-03-28  6:25                                       ` Pekka Enberg
2011-03-28  6:25                                         ` Pekka Enberg
2011-03-28  6:36                                         ` Ingo Molnar
2011-03-28  6:36                                           ` Ingo Molnar
2011-03-28  7:26                                           ` Eric Dumazet
2011-03-28  7:26                                             ` Eric Dumazet
2011-03-28 13:44                                           ` Christoph Lameter
2011-03-28 13:44                                             ` Christoph Lameter
2011-03-28  7:35                                     ` Tejun Heo
2011-03-28  7:35                                       ` Tejun Heo
2011-03-26 19:30                         ` Linus Torvalds
2011-03-26 19:30                           ` Linus Torvalds
2011-03-26 17:58                   ` [boot crash #2] Re: [GIT PULL] SLAB changes for v2.6.39-rc1 Linus Torvalds
2011-03-26 17:58                     ` Linus Torvalds
2011-03-24 19:16           ` Call Trace:Re: " Thomas Gleixner
2011-03-24 19:16             ` Thomas Gleixner
2011-03-24 19:23             ` Ingo Molnar
2011-03-24 19:23               ` Ingo Molnar
2011-03-24 20:04               ` Gustavo F. Padovan
2011-03-24 20:04                 ` Gustavo F. Padovan
2011-03-24 16:03   ` Ingo Molnar
2011-03-24 16:03     ` Ingo Molnar
2011-03-24 16:14     ` Ingo Molnar
2011-03-24 16:14       ` Ingo Molnar
2011-03-24 17:07       ` Pekka Enberg
2011-03-24 17:07         ` Pekka Enberg
2011-03-24 17:21         ` Ingo Molnar
2011-03-24 17:21           ` 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=20110326114736.GA8251@elte.hu \
    --to=mingo@elte.hu \
    --cc=akpm@linux-foundation.org \
    --cc=cl@linux.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=npiggin@kernel.dk \
    --cc=penberg@kernel.org \
    --cc=rientjes@google.com \
    --cc=tglx@linutronix.de \
    --cc=tj@kernel.org \
    --cc=torvalds@linux-foundation.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.