Re: [tip:sched/urgent] sched: Teach might_sleep() aboutpreemptible RCU

From: Frederic Weisbecker
Date: Wed Dec 16 2009 - 13:57:56 EST


On Wed, Dec 16, 2009 at 07:30:20AM -0800, Paul E. McKenney wrote:
> On Wed, Dec 16, 2009 at 03:25:36PM +0100, Ingo Molnar wrote:
> >
> > * tip-bot for Frederic Weisbecker <fweisbec@xxxxxxxxx> wrote:
> >
> > > Commit-ID: ba1b1cbcc9b458c4b9ebb28c9cf6a2ccf64ba1cd
> > > Gitweb: http://git.kernel.org/tip/ba1b1cbcc9b458c4b9ebb28c9cf6a2ccf64ba1cd
> > > Author: Frederic Weisbecker <fweisbec@xxxxxxxxx>
> > > AuthorDate: Mon, 14 Dec 2009 23:44:32 +0100
> > > Committer: Ingo Molnar <mingo@xxxxxxx>
> > > CommitDate: Wed, 16 Dec 2009 08:43:16 +0100
> > >
> > > sched: Teach might_sleep() about preemptible RCU
> >
> > Causes a build failure, so skipped this patch for now:
> >
> > kernel/sched.c:9661: error: implicit declaration of function 'rcu_preempt_depth'
>
> I guess my testing missed something. :-/ Could you please send the
> .config file that caused this?
>
> Thanx, Paul


Looks like the problem is here:

rcupdate.h:

#if defined(CONFIG_TREE_RCU) || defined(CONFIG_TREE_PREEMPT_RCU)
#include <linux/rcutree.h>
#elif defined(CONFIG_TINY_RCU)
#include <linux/rcutiny.h>

I guess it barfs in rcutiny configs.

I should define these helpers in rcupdate.

--
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/