On Fri, 2007-03-30 at 12:11 +0200, Michael Buesch wrote:
> On Thursday 29 March 2007 01:00, Johannes Berg wrote:
> > - if(handler != NULL) {
> > + if (handler) {
> > + if (!dev->wireless_handlers->no_locking)
> > + rtnl_lock();
>
> Is sparse OK with this conditional locking?
I also thought it wasn't but it didn't complain. The only reason why I
didn't just duplicate the code was the deep indentation here...
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
