On Sat, 13 May 2017 00:15:03 +0200 (CEST)
Thomas Gleixner <[email protected]> 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
