GFS2: Eliminate always-0 return from gfs2_set_mode

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 <[email protected]>

---

fs/gfs2/acl.c +2 -8

index 394dc55..568f650 100644
@@ -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) {