Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933863Ab1CXT1P (ORCPT ); Thu, 24 Mar 2011 15:27:15 -0400 Received: from mail-yi0-f46.google.com ([209.85.218.46]:32797 "EHLO mail-yi0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933687Ab1CXT1K convert rfc822-to-8bit (ORCPT ); Thu, 24 Mar 2011 15:27:10 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; b=TslVoINfUzWjNHMkoFTWqEr6gFWBHsClxltuJx2d9hPQxoZoeuX3/zCBzdnxKceBPg 2LnElYPwTZHe5LfEfXQ9UNs1Ey2OLDkClEqL225I3PwR5L8jvKn49YtYeo0udyD86JmI RR2SrRoTP3boO9TkYW/IbgJprWhTHGBhUgv+k= MIME-Version: 1.0 In-Reply-To: <20110324192247.GA5477@elte.hu> References: <20110324142146.GA11682@elte.hu> <20110324172653.GA28507@elte.hu> <20110324185258.GA28370@elte.hu> <20110324192247.GA5477@elte.hu> Date: Thu, 24 Mar 2011 21:27:09 +0200 X-Google-Sender-Auth: _iAPM6QIYoP1czodZWIA8Jq3l9c Message-ID: Subject: Re: [boot crash #2] Re: [GIT PULL] SLAB changes for v2.6.39-rc1 From: Pekka Enberg To: Ingo Molnar Cc: Thomas Gleixner , Christoph Lameter , 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 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1949 Lines: 50 On Thu, Mar 24, 2011 at 9:22 PM, Ingo Molnar wrote: > > * Thomas Gleixner wrote: > >> On Thu, 24 Mar 2011, Ingo Molnar wrote: >> > RIP: 0010:[] ?[] get_next_timer_interrupt+0x119/0x260 >> >> That's a typical timer crash, but you were unable to debug it with >> debugobjects because commit d3f661d6 broke those. >> >> Christoph, debugobjects do not need to run with interupts >> disabled. And just because they were in that section to keep all the >> debug stuff together does not make an excuse for not looking at the >> code and just slopping it into some totally unrelated ifdef along with >> a completely bogus comment. >> >> Signed-off-by: Thomas Gleixner >> --- >> ?mm/slub.c | ? ?4 ++-- >> ?1 file changed, 2 insertions(+), 2 deletions(-) >> >> Index: linux-2.6/mm/slub.c >> =================================================================== >> --- linux-2.6.orig/mm/slub.c >> +++ linux-2.6/mm/slub.c >> @@ -849,11 +849,11 @@ static inline void slab_free_hook(struct >> ? ? ? ? ? ? ? local_irq_save(flags); >> ? ? ? ? ? ? ? kmemcheck_slab_free(s, x, s->objsize); >> ? ? ? ? ? ? ? debug_check_no_locks_freed(x, s->objsize); >> - ? ? ? ? ? ? if (!(s->flags & SLAB_DEBUG_OBJECTS)) >> - ? ? ? ? ? ? ? ? ? ? debug_check_no_obj_freed(x, s->objsize); >> ? ? ? ? ? ? ? local_irq_restore(flags); >> ? ? ? } >> ?#endif >> + ? ? if (!(s->flags & SLAB_DEBUG_OBJECTS)) >> + ? ? ? ? ? ? debug_check_no_obj_freed(x, s->objsize); > > Thanks, this did the trick! > > Tested-by: Ingo Molnar > > With this fix i got the warning below - pinpointing a net/bluetooth/hci_core.c > timer bug. Applied, thanks! -- 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/