On Tue, 2018-06-19 at 22:04 -0500, Denis Kenzior wrote:
> 
> Right, but you’re much more aware of all the locking issues than I am.

:-)

> > Clearly this comes from cfg80211 without any locking other than rtnl, so
> > you don't have preemption disabled. That's the minimum needed to get rid
> > of the warning you found.
> 
> In my defense, I did ask you whether there are any potential locking
> issues in the RFC and you didn’t think there were any.

Yep, I missed that too. More precisely, ISTR actually thinking about it
and deciding it was fine, so ... my bad for sure.

> I posted a fix for this.  Could you please review?

I think it's fine. I'll make a pass later today/this week and send
patches upstream.

johannes

Reply via email to