From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756385AbZDLOqn (ORCPT ); Sun, 12 Apr 2009 10:46:43 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754344AbZDLOqb (ORCPT ); Sun, 12 Apr 2009 10:46:31 -0400 Received: from hera.kernel.org ([140.211.167.34]:34585 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753126AbZDLOqa (ORCPT ); Sun, 12 Apr 2009 10:46:30 -0400 Date: Sun, 12 Apr 2009 14:45:46 GMT From: Frederic Weisbecker To: linux-tip-commits@vger.kernel.org Cc: linux-kernel@vger.kernel.org, hpa@zytor.com, mingo@redhat.com, peterz@infradead.org, ltp-list@lists.sourceforge.net, fweisbec@gmail.com, gregkh@suse.de, tglx@linutronix.de, mingo@elte.hu Reply-To: mingo@redhat.com, hpa@zytor.com, linux-kernel@vger.kernel.org, fweisbec@gmail.com, peterz@infradead.org, gregkh@suse.de, ltp-list@lists.sourceforge.net, tglx@linutronix.de, mingo@elte.hu In-Reply-To: <1239412638-6739-2-git-send-email-fweisbec@gmail.com> References: <1239412638-6739-2-git-send-email-fweisbec@gmail.com> Subject: [tip:core/urgent] lockdep: continue lock debugging despite some taints Message-ID: Git-Commit-ID: 574bbe782057fdf0490dc7dec906a2dc26363e20 X-Mailer: tip-git-log-daemon MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.0 (hera.kernel.org [127.0.0.1]); Sun, 12 Apr 2009 14:45:48 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: 574bbe782057fdf0490dc7dec906a2dc26363e20 Gitweb: http://git.kernel.org/tip/574bbe782057fdf0490dc7dec906a2dc26363e20 Author: Frederic Weisbecker AuthorDate: Sat, 11 Apr 2009 03:17:18 +0200 Committer: Ingo Molnar CommitDate: Sun, 12 Apr 2009 16:10:52 +0200 lockdep: continue lock debugging despite some taints Impact: broaden lockdep checks Lockdep is disabled after any kernel taints. This might be convenient to ignore bad locking issues which sources come from outside the kernel tree. Nevertheless, it might be a frustrating experience for the staging developers or those who experience a warning but are focused on another things that require lockdep. The v2 of this patch simply don't disable anymore lockdep in case of TAINT_CRAP and TAINT_WARN events. Signed-off-by: Frederic Weisbecker Cc: LTP Cc: Peter Zijlstra Cc: Greg KH LKML-Reference: <1239412638-6739-2-git-send-email-fweisbec@gmail.com> Signed-off-by: Ingo Molnar --- kernel/panic.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/kernel/panic.c b/kernel/panic.c index 940ca14..934fb37 100644 --- a/kernel/panic.c +++ b/kernel/panic.c @@ -217,8 +217,10 @@ void add_taint(unsigned flag) * Can't trust the integrity of the kernel anymore. * We don't call directly debug_locks_off() because the issue * is not necessarily serious enough to set oops_in_progress to 1 + * Also we want to keep up lockdep for staging development and + * post-warning case. */ - if (__debug_locks_off()) + if (flag != TAINT_CRAP && flag != TAINT_WARN && __debug_locks_off()) printk(KERN_WARNING "Disabling lockdep due to kernel taint\n"); set_bit(flag, &tainted_mask);