Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754044AbYLNOeu (ORCPT ); Sun, 14 Dec 2008 09:34:50 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752514AbYLNOek (ORCPT ); Sun, 14 Dec 2008 09:34:40 -0500 Received: from rv-out-0506.google.com ([209.85.198.235]:51010 "EHLO rv-out-0506.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752237AbYLNOej (ORCPT ); Sun, 14 Dec 2008 09:34:39 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=HH3JtPuM2dHlEOl5+jkHtTJeF2WdCxZQyvVo8RmtWnaGmm+X2/mZ4JPXJWCIbMwfTE DhFuHxZEw0mrq1U+bH4fDQ2aOb88W6CzL2yLrhhlE5vvTRbpzBkiikb03oCVehPa2Fzq 90WpgsFQFmLU+ZONXnOa/BLIGqkEnp4kzCrB4= Message-ID: Date: Sun, 14 Dec 2008 17:34:38 +0300 From: "Alexey Zaytsev" To: "Stephen Rothwell" Subject: Re: linux-next: Tree for December 11 Cc: linux-next@vger.kernel.org, LKML , "Ingo Molnar" , "Nick Piggin" In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20081211200459.14d18d7a.sfr@canb.auug.org.au> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4646 Lines: 113 On Thu, Dec 11, 2008 at 15:40, Alexey Zaytsev wrote: > On Thu, Dec 11, 2008 at 12:04, Stephen Rothwell wrote: >> Hi all, >> >> Changes since 20081210: >> >> New tree: >> nommu >> >> Undropped tree: >> sound >> >> Dropped trees (temporarily): >> v4l-dvb (build problem) >> mtd (difficult conflicts) >> drm (build problem) >> semaphore-removal (due to unfixed conflicts against Linus' tree) >> cpu_alloc (build problem) >> perfmon3 (concerns from the x86 team) >> audit (difficult conflicts) >> nommu (build problem) >> staging (build failure) >> >> The driver-core tree gained a build failure that needed a commit reverted. >> >> The ftrace tree gained a conflict against Linus' tree. >> >> The pci tree gained a conflict against the driver-core tree. >> >> The mtd tree gained 3 conflicts against the arm tree which I could not >> easily resolve, so it was dropped. >> >> The ttydev tree gained a conflict against the async_tx tree requiring a >> commit from the async_tx tree to be reverted. >> >> The nommu tree gained conflicts against the slab and kmemcheck trees and >> also a build failure so it was dropped. >> >> ---------------------------------------------------------------------------- > > Hi. > > I'm seeing this warning early in boot logs. It does not appear on 2.6.28-rc7. > Not sure how long it's been around. Haven't built -next for some time. > > [ 0.004000] Intel machine check reporting enabled on CPU#0. > [ 0.004000] using mwait in idle threads. > [ 0.004000] Checking 'hlt' instruction... <4>------------[ cut here > ]------------ > [ 0.004167] WARNING: at kernel/sched.c:4364 sub_preempt_count+0xae/0xc0() > [ 0.004266] Hardware name: HP Compaq nx7300 (GB848ES#ACB) > [ 0.004361] Modules linked in: > [ 0.004497] Pid: 0, comm: swapper Not tainted 2.6.28-rc8-next-20081211 #117 > [ 0.004595] Call Trace: > [ 0.004689] [] warn_slowpath+0x86/0xa0 > [ 0.004789] [] ? check_usage_forwards+0x10/0xb0 > [ 0.004886] [] ? save_trace+0x3a/0xa0 > [ 0.004981] [] ? mark_lock+0x37d/0xe00 > [ 0.005076] [] ? __lock_acquire+0x249/0x610 > [ 0.005175] [] ? _spin_unlock_irq+0x22/0x50 > [ 0.005272] [] ? trace_hardirqs_on_caller+0x70/0x1a0 > [ 0.005369] [] ? _spin_unlock_irq+0x2d/0x50 > [ 0.005465] [] sub_preempt_count+0xae/0xc0 > [ 0.005564] [] _local_bh_enable+0x52/0xc0 > [ 0.005661] [] __do_softirq+0x11f/0x170 > [ 0.005756] [] ? __do_softirq+0x0/0x170 > [ 0.005851] [] ? irq_exit+0x89/0xa0 > [ 0.005993] [] ? do_IRQ+0xad/0x120 > [ 0.006088] [] ? common_interrupt+0x2c/0x34 > [ 0.006184] [] ? mmput+0x2b/0xc0 > [ 0.006281] [] ? check_bugs+0xb8/0xe0 > [ 0.006379] [] ? start_kernel+0x26a/0x310 > [ 0.006475] [] ? unknown_bootoption+0x0/0x210 > [ 0.006572] [] ? __init_begin+0x77/0xb0 > [ 0.006674] ---[ end trace 4eaa2a86a8e2da22 ]--- > [ 0.016004] OK. > [ 0.016560] ACPI: Core revision 20081031 > [ 0.044495] ..TIMER: vector=0x30 apic1=0 pin1=2 apic2=-1 pin2=-1 > > Config and full dmesg attached. > The warning can also be reproduced in qemu, so it was easy to bisect. commit 7317d7b87edb41a9135e30be1ec3f7ef817c53dd Author: Nick Piggin Date: Tue Sep 30 20:50:27 2008 +1000 sched: improve preempt debugging This patch helped me out with a problem I recently had.... Basically, when the kernel lock is held, then preempt_count underflow does not get detected until it is released which may be a long time (and arbitrarily, eg at different points it may be rescheduled). If the bkl is released at schedule, the resulting output is actually fairly cryptic... With any other lock that elevates preempt_count, it is illegal to schedule under it (which would get found pretty quickly). bkl allows scheduling with preempt_count elevated, which makes underflows hard to debug. Signed-off-by: Ingo Molnar I understand that not this particular commit is buggy, but at least I've got someone to add to the CC. ;) Also the author's e-mail looks suspicious. -- 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/