Re: [RFC][PATCH 2/5] cpu-hotplug: Allow get_online_cpus() to nest

From: Steven Rostedt
Date: Fri May 12 2017 - 20:23:24 EST


On Sat, 13 May 2017 00:15:03 +0200 (CEST)
Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:

> On Fri, 12 May 2017, Steven Rostedt wrote:
> > void get_online_cpus(void)
> > {
> > +#ifdef CONFIG_LOCKDEP
> > + if (current->goc_depth++)
> > + return;
>
> This must be unconditional and not depend on lockdep. The percpu rwsem is
> going to deadlock silently otherwise when a writer is waiting ....
>

After I sent the updated changes to Paul, I realized this. But didn't
update it as I turned my attention to seeing if we can still get it
done without it.

But I'm not convinced that we can yet. I'll reply to Peter's email
again later.

-- Steve