batman-adv: Avoid deadlock between rtnl_lock and s_active
authorSven Eckelmann <sven@narfation.org>
Tue, 3 May 2011 11:10:06 +0000 (13:10 +0200)
committerSven Eckelmann <sven@narfation.org>
Sun, 8 May 2011 14:10:42 +0000 (16:10 +0200)
The hard_if_event is called by the notifier with rtnl_lock and tries to
remove sysfs entries when a NETDEV_UNREGISTER event is received. This
will automatically take the s_active lock.

The s_active lock is also used when a new interface is added to a meshif
through sysfs. In that situation we cannot wait for the rntl_lock before
creating the actual batman-adv interface to prevent a deadlock. It is
still possible to try to get the rtnl_lock and immediately abort the
current operation when the trylock call failed.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
net/batman-adv/bat_sysfs.c

index 85ba20d98a66fb76a2f9258cbe1e46bfdb2054c9..497a0700cc3cb03881edd881c7850973fc45f2b8 100644 (file)
@@ -488,24 +488,24 @@ static ssize_t store_mesh_iface(struct kobject *kobj, struct attribute *attr,
            (strncmp(hard_iface->soft_iface->name, buff, IFNAMSIZ) == 0))
                goto out;
 
+       if (!rtnl_trylock()) {
+               ret = -ERESTARTSYS;
+               goto out;
+       }
+
        if (status_tmp == IF_NOT_IN_USE) {
-               rtnl_lock();
                hardif_disable_interface(hard_iface);
-               rtnl_unlock();
-               goto out;
+               goto unlock;
        }
 
        /* if the interface already is in use */
-       if (hard_iface->if_status != IF_NOT_IN_USE) {
-               rtnl_lock();
+       if (hard_iface->if_status != IF_NOT_IN_USE)
                hardif_disable_interface(hard_iface);
-               rtnl_unlock();
-       }
 
-       rtnl_lock();
        ret = hardif_enable_interface(hard_iface, buff);
-       rtnl_unlock();
 
+unlock:
+       rtnl_unlock();
 out:
        hardif_free_ref(hard_iface);
        return ret;