On Tue, Nov 08, 2022 at 08:04:16PM +0300, Vitaliy Makkoveev wrote:
> The `if_cloners’ list is immutable. You don't need kernel lock
> around if_clone_list() call.
> 
> >     case SIOCIFGCLONERS:
> > +           KERNEL_LOCK();
> >             error = if_clone_list((struct if_clonereq *)data);
> > +           KERNEL_UNLOCK();
> >             return (error);
> 
> 
> With this fix diff looks good for me.

This is going the be first unlocking diff, I just want to clearly
separate setup churn (like this) from actual lock semantic changes.

Reply via email to