Re: [patch 8/8] lock validator: add s390 to supported options

From: Ingo Molnar
Date: Fri Jun 23 2006 - 09:57:55 EST



* Heiko Carstens <heiko.carstens@xxxxxxxxxx> wrote:

> On Mon, Jun 19, 2006 at 03:05:47PM -0700, Andrew Morton wrote:
> > Heiko Carstens <heiko.carstens@xxxxxxxxxx> wrote:
> > >
> > > config DEBUG_SPINLOCK_ALLOC
> > > bool "Spinlock debugging: detect incorrect freeing of live spinlocks"
> > > - depends on DEBUG_SPINLOCK && X86
> > > + depends on DEBUG_SPINLOCK && (X86 || S390)
> >
> > Can we please stomp this out before it starts to look like
> > CONFIG_FRAME_POINTER?
> >
> > We should define CONFIG_ARCH_SUPPORTS_LOCKDEP down in
> > arch/[i386|x86_64|s390]/Kconfig and use that in lib/Kconfig.debug.
>
> How about this:
>
> From: Heiko Carstens <heiko.carstens@xxxxxxxxxx>
>
> Add LOCKDEP_SUPPORT config option per architecture.

fine with me. Perhaps we can thus replace IRQ_TRACE_SUPPORT with
LOCKDEP_SUPPORT as well, and would codify LOCKDEP_SUPPORT to mean
irqflags-trace support and stacktrace support?

Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/