Re: [PATCH] debug workqueue locking sanity

From: Ingo Molnar
Date: Mon Nov 06 2006 - 02:24:35 EST



* Peter Zijlstra <a.p.zijlstra@xxxxxxxxx> wrote:

> + if (unlikely(in_atomic()
> +#ifdef CONFIG_LOCKDEP
> + || current->lockdep_depth > 0
> +#endif
> + )) {

i agree with this patch, but shouldnt this #ifdef be hidden via some
sort of lockdep_depth() inline in lockdep.h that just returns 0 if
!LOCKDEP?

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/