[SCSI] fcoe: Remove mutex_trylock/restart_syscall checks
authorRobert Love <robert.w.love@intel.com>
Fri, 1 Apr 2011 23:05:59 +0000 (16:05 -0700)
committerJames Bottomley <James.Bottomley@suse.de>
Sun, 1 May 2011 15:19:03 +0000 (10:19 -0500)
These checks were initially added to avoid a lockdep
false positive when dealing with the s_active, rtnl
and fcoe_config_mutex mutexes. Recently the create,
destroy, enable and disable sysfs entries were moved
from fcoe.ko to libfcoe.ko. With this change the mutex
usage was shuffled around and the lockdep false
positive stopped happening. We can now remove these
checks.

Signed-off-by: Robert Love <robert.w.love@intel.com>
Tested-by: Ross Brattain <ross.b.brattain@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/fcoe/fcoe.c

index bde6ee5333eba62f7ac5a87c566c58c1dc31075b..9e7206ebadf05e54b425a692b5330e6a8eeb51f9 100644 (file)
@@ -1795,11 +1795,7 @@ static int fcoe_disable(struct net_device *netdev)
        }
 #endif
 
-       if (!rtnl_trylock()) {
-               mutex_unlock(&fcoe_config_mutex);
-               return -ERESTARTSYS;
-       }
-
+       rtnl_lock();
        fcoe = fcoe_hostlist_lookup_port(netdev);
        rtnl_unlock();
 
@@ -1839,11 +1835,7 @@ static int fcoe_enable(struct net_device *netdev)
                goto out_nodev;
        }
 #endif
-       if (!rtnl_trylock()) {
-               mutex_unlock(&fcoe_config_mutex);
-               return -ERESTARTSYS;
-       }
-
+       rtnl_lock();
        fcoe = fcoe_hostlist_lookup_port(netdev);
        rtnl_unlock();
 
@@ -1882,11 +1874,7 @@ static int fcoe_destroy(struct net_device *netdev)
                goto out_nodev;
        }
 #endif
-       if (!rtnl_trylock()) {
-               mutex_unlock(&fcoe_config_mutex);
-               return -ERESTARTSYS;
-       }
-
+       rtnl_lock();
        fcoe = fcoe_hostlist_lookup_port(netdev);
        if (!fcoe) {
                rtnl_unlock();
@@ -1948,11 +1936,7 @@ static int fcoe_create(struct net_device *netdev, enum fip_state fip_mode)
        struct fc_lport *lport;
 
        mutex_lock(&fcoe_config_mutex);
-
-       if (!rtnl_trylock()) {
-               mutex_unlock(&fcoe_config_mutex);
-               return -ERESTARTSYS;
-       }
+       rtnl_lock();
 
 #ifdef CONFIG_FCOE_MODULE
        /*