mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-06 02:50:43 +00:00
[SCSI] fcoe: Only rmmod fcoe.ko if there are no active connections
Currently we're gracefully tearing down each active connection when fcoe.ko is removed. We shouldn't allow the user to destroy connections by removing the module. We should force the user to destroy each connection and then the module can be removed. This patch makes it so a refrerence count on the module is taken each time a fcoe_interface is created. The reference count is dropped when the fcoe_interface is destroyed. This makes it so that module_exit() doesn't get called unless all fcoe_interfaces have been destroyed. This patch leaves the removal of interfaces in the module_exit routine so that if the user does a 'rmmod -f' we'll clean everything up before removing the module. The module_put line was put before the out_putdev goto line because we should only be decrementing the reference count if a fcoe_interface is actually destroyed. If we can't find the netdev or the fcoe_interface then it's assumed that something else has destroyed the fcoe_interface and it would have decremented the reference count at that time. Signed-off-by: Robert Love <robert.w.love@intel.com> Signed-off-by: James Bottomley <James.Bottomley@suse.de>
This commit is contained in:
parent
03bf7a5af5
commit
e9ee2cf438
1 changed files with 16 additions and 2 deletions
|
@ -2009,6 +2009,8 @@ static int fcoe_destroy(const char *buffer, struct kernel_param *kp)
|
|||
fcoe_interface_cleanup(fcoe);
|
||||
rtnl_unlock();
|
||||
fcoe_if_destroy(fcoe->ctlr.lp);
|
||||
module_put(THIS_MODULE);
|
||||
|
||||
out_putdev:
|
||||
dev_put(netdev);
|
||||
out_nodev:
|
||||
|
@ -2059,6 +2061,11 @@ static int fcoe_create(const char *buffer, struct kernel_param *kp)
|
|||
}
|
||||
#endif
|
||||
|
||||
if (!try_module_get(THIS_MODULE)) {
|
||||
rc = -EINVAL;
|
||||
goto out_nomod;
|
||||
}
|
||||
|
||||
rtnl_lock();
|
||||
netdev = fcoe_if_to_netdev(buffer);
|
||||
if (!netdev) {
|
||||
|
@ -2099,17 +2106,24 @@ static int fcoe_create(const char *buffer, struct kernel_param *kp)
|
|||
if (!fcoe_link_ok(lport))
|
||||
fcoe_ctlr_link_up(&fcoe->ctlr);
|
||||
|
||||
rc = 0;
|
||||
out_free:
|
||||
/*
|
||||
* Release from init in fcoe_interface_create(), on success lport
|
||||
* should be holding a reference taken in fcoe_if_create().
|
||||
*/
|
||||
fcoe_interface_put(fcoe);
|
||||
dev_put(netdev);
|
||||
rtnl_unlock();
|
||||
mutex_unlock(&fcoe_config_mutex);
|
||||
|
||||
return 0;
|
||||
out_free:
|
||||
fcoe_interface_put(fcoe);
|
||||
out_putdev:
|
||||
dev_put(netdev);
|
||||
out_nodev:
|
||||
rtnl_unlock();
|
||||
module_put(THIS_MODULE);
|
||||
out_nomod:
|
||||
mutex_unlock(&fcoe_config_mutex);
|
||||
return rc;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue