Florian Westphal <f...@strlen.de> wrote: > + mutex_lock(&ifalias_mutex); > + > + old = rcu_dereference_protected(dev->ifalias, > + mutex_is_locked(&ifalias_mutex)); > + if (len) { > + memcpy(new_alias->ifalias, alias, len); > + new_alias->ifalias[len] = 0; > + }
This can be done outside of the lock, so this can be > + > + rcu_assign_pointer(dev->ifalias, new_alias); rcu_swap_protected(). I'll send a v3.