[Cluster-devel] [GFS2 PATCH] GFS2: Eliminate always-0 return from gfs2_set_mode

[Bob Peterson <rpeterso@redhat.com>]

Hi,

This patch eliminates a return code from function gfs2_set_mode that
was always 0. We could also just eliminate the function altogether
because after the recent acl reforms, it's only called in one place.

Regards,

Bob Peterson
Red Hat File Systems

Signed-off-by: Bob Peterson <rpeterso@redhat.com> 
---
diff --git a/fs/gfs2/acl.c b/fs/gfs2/acl.c
index 394dc55..568f650 100644
--- a/fs/gfs2/acl.c
+++ b/fs/gfs2/acl.c
@@ -64,16 +64,12 @@ struct posix_acl *gfs2_get_acl(struct inode *inode, int type)
 	return acl;
 }
 
-static int gfs2_set_mode(struct inode *inode, umode_t mode)
+static void gfs2_set_mode(struct inode *inode, umode_t mode)
 {
-	int error = 0;
-
 	if (mode != inode->i_mode) {
 		inode->i_mode = mode;
 		mark_inode_dirty(inode);
 	}
-
-	return error;
 }
 
 int gfs2_set_acl(struct inode *inode, struct posix_acl *acl, int type)
@@ -98,9 +94,7 @@ int gfs2_set_acl(struct inode *inode, struct posix_acl *acl, int type)
 		if (error == 0)
 			acl = NULL;
 
-		error = gfs2_set_mode(inode, mode);
-		if (error)
-			return error;
+		gfs2_set_mode(inode, mode);
 	}
 
 	if (acl) {