diff options
| author | NeilBrown <neilb@suse.com> | 2015-08-14 03:22:00 +0300 |
|---|---|---|
| committer | NeilBrown <neilb@suse.com> | 2015-08-31 20:43:17 +0300 |
| commit | 18b9f67962eb890da0c053e04c3cf0e91871d4fa (patch) | |
| tree | 567f33d5e225d3bc7f42a2854f75e3db405b6a84 /drivers/md | |
| parent | 6022e75bf0686799a6ecca3c33a669e6c70e9d26 (diff) | |
| download | linux-18b9f67962eb890da0c053e04c3cf0e91871d4fa.tar.xz | |
md-cluster: remove inappropriate try_module_get from join()
md_setup_cluster already calls try_module_get(), so this
try_module_get isn't needed.
Also, there is no matching module_put (except in error patch),
so this leaves an unbalanced module count.
Signed-off-by: NeilBrown <neilb@suse.com>
Diffstat (limited to 'drivers/md')
| -rw-r--r-- | drivers/md/md-cluster.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/md/md-cluster.c b/drivers/md/md-cluster.c index c35a03a7eb11..11e3bc9d2a4b 100644 --- a/drivers/md/md-cluster.c +++ b/drivers/md/md-cluster.c @@ -687,9 +687,6 @@ static int join(struct mddev *mddev, int nodes) int ret, ops_rv; char str[64]; - if (!try_module_get(THIS_MODULE)) - return -ENOENT; - cinfo = kzalloc(sizeof(struct md_cluster_info), GFP_KERNEL); if (!cinfo) return -ENOMEM; @@ -771,7 +768,6 @@ err: dlm_release_lockspace(cinfo->lockspace, 2); mddev->cluster_info = NULL; kfree(cinfo); - module_put(THIS_MODULE); return ret; } |