[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090802123958.cbd497a0.akpm@linux-foundation.org>
Date: Sun, 2 Aug 2009 12:39:58 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: Ingo Molnar <mingo@...e.hu>
Cc: paulmck@...ux.vnet.ibm.com, mingo@...hat.com, hpa@...or.com,
linux-kernel@...r.kernel.org, a.p.zijlstra@...llo.nl,
torvalds@...ux-foundation.org, tglx@...utronix.de,
linux-tip-commits@...r.kernel.org
Subject: Re: [tip:core/debug] debug lockups: Improve lockup detection
On Sun, 2 Aug 2009 21:26:57 +0200 Ingo Molnar <mingo@...e.hu> wrote:
> > I think this just broke all non-x86 non-sparc SMP architectures.
>
> Yeah - it 'broke' them in the sense of them not having a working
> trigger_all_cpu_backtrace() implementation to begin with.
c'mon. It broke them in the sense that sysrq-l went from "works" to
"doesn't work".
It would take months for the relevant arch maintainers to even find out
about this, after which they're left with dud kernels out in the field.
It's better to break the build or to emit warnings than to silently and
secretly break their stuff.
--- a/include/linux/nmi.h~a
+++ a/include/linux/nmi.h
@@ -29,6 +29,9 @@ static inline void acpi_nmi_enable(void)
#endif
#ifndef trigger_all_cpu_backtrace
+#ifdef CONFIG_SMP
+#warning This architecture is missing a trigger_all_cpu_backtrace() implementation
+#endif
#define trigger_all_cpu_backtrace() do { } while (0)
#endif
_
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists