ceph: delete an unnecessary return statement in update_dentry_lease()

elfring · idryomov · commit b529d1b382f7 · 2017-09-06T19:56:51.000+02:00
The script "checkpatch.pl" pointed information out like the following.

WARNING: void function return statements are not generally useful

Thus remove such a statement in the affected function.

Signed-off-by: Markus Elfring &lt;elfring@users.sourceforge.net&gt;
Reviewed-by: Yan, Zheng &lt;zyan@redhat.com&gt;
Signed-off-by: Ilya Dryomov &lt;idryomov@gmail.com&gt;
diff --git a/fs/ceph/inode.c b/fs/ceph/inode.c
@@ -1067,7 +1067,6 @@ static void update_dentry_lease(struct dentry *dentry,
 	spin_unlock(&dentry->d_lock);
 	if (old_lease_session)
 		ceph_put_mds_session(old_lease_session);
-	return;
 }
 
 /*

Original file line number	Diff line number	Diff line change
`@@ -1067,7 +1067,6 @@ static void update_dentry_lease(struct dentry *dentry,`
`1067`	`1067`	`spin_unlock(&dentry->d_lock);`
`1068`	`1068`	`if (old_lease_session)`
`1069`	`1069`	`ceph_put_mds_session(old_lease_session);`
`1070`		`- return;`
`1071`	`1070`	`}`
`1072`	`1071`
`1073`	`1072`	`/*`