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.

Reply via email to