On Fri, Aug 09, 2013 at 04:04:14PM -0700, Andi Kleen wrote: > From: Andi Kleen <a...@linux.intel.com> > > Add a might_fault_debug_only() that only does something in the PROVE_LOCKING > case, but does not cond_resched for PREEMPT_VOLUNTARY. This is for > cases when the cond_resched is done elsewhere > > Signed-off-by: Andi Kleen <a...@linux.intel.com> > --- > include/linux/sched.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/linux/sched.h b/include/linux/sched.h > index 773f21d..bb7a08a 100644 > --- a/include/linux/sched.h > +++ b/include/linux/sched.h > @@ -2473,11 +2473,13 @@ static inline void cond_resched_rcu(void) > > #ifdef CONFIG_PROVE_LOCKING > void might_fault(void); > +#define might_fault_debug_only() might_fault() > #else > static inline void might_fault(void) > { > might_sleep(); > } > +#define might_fault_debug_only() do {} while(0)
Hi Andy, this is against which kernel version? In 3.11-rc3 I see: #if defined(CONFIG_PROVE_LOCKING) || defined(CONFIG_DEBUG_ATOMIC_SLEEP) void might_fault(void); #else static inline void might_fault(void) { } #endif So it's not clear to me how it's different from your might_fault_debug_only .. > #endif > > /* > -- > 1.8.3.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/