From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bob Peterson Date: Wed, 19 Mar 2014 11:31:26 -0400 (EDT) Subject: [Cluster-devel] [GFS2 PATCH] GFS2: Eliminate always-0 return from gfs2_set_mode In-Reply-To: <1051482030.2085840.1395243027100.JavaMail.zimbra@redhat.com> Message-ID: <322086118.2087373.1395243086363.JavaMail.zimbra@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit 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 --- 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) {