[GFS2] Align all labels against LH side

This makes everything consistent.

Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>

19 files changed, 129 insertions, 245 deletions

diff --git a/fs/gfs2/acl.c b/fs/gfs2/acl.c
index 60c98c0314a1..d846b5ad1d87 100644
--- a/fs/gfs2/acl.c
+++ b/fs/gfs2/acl.c
@@ -65,9 +65,8 @@ int gfs2_acl_validate_set(struct gfs2_inode *ip, int access,
 			error = 0;
 	}
 
- out:
+out:
 	posix_acl_release(acl);
-
 	return error;
 }
 
@@ -132,18 +131,16 @@ static int acl_get(struct gfs2_inode *ip, int access, struct posix_acl **acl,
 			error = PTR_ERR(*acl);
 	}
 
- out_kfree:
+out_kfree:
 	if (error || !data)
 		kfree(er.er_data);
 	else {
 		*data = er.er_data;
 		*len = er.er_data_len;
 	}
-
- out:
+out:
 	if (error || el == &el_this)
 		brelse(el->el_bh);
-
 	return error;
 }
 
@@ -270,7 +267,7 @@ int gfs2_acl_create(struct gfs2_inode *dip, struct gfs2_inode *ip)
 	} else
 		munge_mode(ip, mode);
 
- out:
+out:
 	posix_acl_release(acl);
 	kfree(er.er_data);
 	return error;
@@ -303,11 +300,10 @@ int gfs2_acl_chmod(struct gfs2_inode *ip, struct iattr *attr)
 		error = gfs2_ea_acl_chmod(ip, &el, attr, data);
 	}
 
- out:
+out:
 	posix_acl_release(acl);
 	brelse(el.el_bh);
 	kfree(data);
-
 	return error;
 }
 
diff --git a/fs/gfs2/bmap.c b/fs/gfs2/bmap.c
index b75a2f93dada..bb597839c0c9 100644
--- a/fs/gfs2/bmap.c
+++ b/fs/gfs2/bmap.c
@@ -170,12 +170,10 @@ int gfs2_unstuff_dinode(struct gfs2_inode *ip, struct page *page)
 
 	gfs2_dinode_out(&ip->i_di, dibh->b_data);
 
- out_brelse:
+out_brelse:
 	brelse(dibh);
-
- out:
+out:
 	up_write(&ip->i_rw_mutex);
-
 	return error;
 }
 
@@ -642,9 +640,8 @@ static int recursive_scan(struct gfs2_inode *ip, struct buffer_head *dibh,
 				break;
 		}
 
- out:
+out:
 	brelse(bh);
-
 	return error;
 }
 
@@ -788,15 +785,12 @@ static int do_strip(struct gfs2_inode *ip, struct buffer_head *dibh,
 
 	gfs2_trans_end(sdp);
 
- out_rg_gunlock:
+out_rg_gunlock:
 	gfs2_glock_dq_m(rlist.rl_rgrps, rlist.rl_ghs);
-
- out_rlist:
+out_rlist:
 	gfs2_rlist_free(&rlist);
-
- out:
+out:
 	gfs2_glock_dq_uninit(&ip->i_alloc.al_ri_gh);
-
 	return error;
 }
 
@@ -868,18 +862,14 @@ static int do_grow(struct gfs2_inode *ip, uint64_t size)
 	gfs2_dinode_out(&ip->i_di, dibh->b_data);
 	brelse(dibh);
 
- out_end_trans:
+out_end_trans:
 	gfs2_trans_end(sdp);
-
- out_ipres:
+out_ipres:
 	gfs2_inplace_release(ip);
-
- out_gunlock_q:
+out_gunlock_q:
 	gfs2_quota_unlock(ip);
-
- out:
+out:
 	gfs2_alloc_put(ip);
-
 	return error;
 }
 
@@ -998,9 +988,8 @@ static int trunc_start(struct gfs2_inode *ip, uint64_t size)
 
 	brelse(dibh);
 
- out:
+out:
 	gfs2_trans_end(sdp);
-
 	return error;
 }
 
@@ -1035,7 +1024,7 @@ static int trunc_dealloc(struct gfs2_inode *ip, uint64_t size)
 
 	gfs2_quota_unhold(ip);
 
- out:
+out:
 	gfs2_alloc_put(ip);
 	return error;
 }
@@ -1070,11 +1059,9 @@ static int trunc_end(struct gfs2_inode *ip)
 	gfs2_dinode_out(&ip->i_di, dibh->b_data);
 	brelse(dibh);
 
- out:
+out:
 	up_write(&ip->i_rw_mutex);
-
 	gfs2_trans_end(sdp);
-
 	return error;
 }
 
diff --git a/fs/gfs2/dir.c b/fs/gfs2/dir.c
index 7b8a38eaa41a..59d0cbcaecff 100644
--- a/fs/gfs2/dir.c
+++ b/fs/gfs2/dir.c
@@ -1134,9 +1134,8 @@ static int dir_double_exhash(struct gfs2_inode *dip)
 
 	return error;
 
- fail:
+fail:
 	kfree(buf);
-
 	return error;
 }
 
@@ -1890,23 +1889,18 @@ static int leaf_dealloc(struct gfs2_inode *dip, uint32_t index, uint32_t len,
 	gfs2_dinode_out(&dip->i_di, dibh->b_data);
 	brelse(dibh);
 
- out_end_trans:
+out_end_trans:
 	gfs2_trans_end(sdp);
-
- out_rg_gunlock:
+out_rg_gunlock:
 	gfs2_glock_dq_m(rlist.rl_rgrps, rlist.rl_ghs);
-
- out_rlist:
+out_rlist:
 	gfs2_rlist_free(&rlist);
 	gfs2_glock_dq_uninit(&dip->i_alloc.al_ri_gh);
-
- out_qs:
+out_qs:
 	gfs2_quota_unhold(dip);
-
- out:
+out:
 	gfs2_alloc_put(dip);
 	kfree(ht);
-
 	return error;
 }
 
diff --git a/fs/gfs2/eattr.c b/fs/gfs2/eattr.c
index 9081822ce80c..5a56e6568622 100644
--- a/fs/gfs2/eattr.c
+++ b/fs/gfs2/eattr.c
@@ -107,7 +107,7 @@ static int ea_foreach_i(struct gfs2_inode *ip, struct buffer_head *bh,
 
 	return error;
 
- fail:
+fail:
 	gfs2_consist_inode(ip);
 	return -EIO;
 }
@@ -152,9 +152,8 @@ static int ea_foreach(struct gfs2_inode *ip, ea_call_t ea_call, void *data)
 		if (error)
 			break;
 	}
- out:
+out:
 	brelse(bh);
-
 	return error;
 }
 
@@ -319,9 +318,8 @@ static int ea_dealloc_unstuffed(struct gfs2_inode *ip, struct buffer_head *bh,
 
 	gfs2_trans_end(sdp);
 
- out_gunlock:
+out_gunlock:
 	gfs2_glock_dq_uninit(&rg_gh);
-
 	return error;
 }
 
@@ -348,12 +346,10 @@ static int ea_remove_unstuffed(struct gfs2_inode *ip, struct buffer_head *bh,
 
 	gfs2_glock_dq_uninit(&al->al_ri_gh);
 
- out_quota:
+out_quota:
 	gfs2_quota_unhold(ip);
-
- out_alloc:
+out_alloc:
 	gfs2_alloc_put(ip);
-
 	return error;
 }
 
@@ -506,9 +502,8 @@ static int ea_get_unstuffed(struct gfs2_inode *ip, struct gfs2_ea_header *ea,
 		brelse(bh[x]);
 	}
 
- out:
+out:
 	kfree(bh);
-
 	return error;
 }
 
@@ -747,18 +742,14 @@ static int ea_alloc_skeleton(struct gfs2_inode *ip, struct gfs2_ea_request *er,
 		brelse(dibh);
 	}
 
- out_end_trans:
+out_end_trans:
 	gfs2_trans_end(GFS2_SB(&ip->i_inode));
-
- out_ipres:
+out_ipres:
 	gfs2_inplace_release(ip);
-
- out_gunlock_q:
+out_gunlock_q:
 	gfs2_quota_unlock(ip);
-
- out:
+out:
 	gfs2_alloc_put(ip);
-
 	return error;
 }
 
@@ -884,9 +875,8 @@ static int ea_set_simple_noalloc(struct gfs2_inode *ip, struct buffer_head *bh,
 	gfs2_trans_add_bh(ip->i_gl, dibh, 1);
 	gfs2_dinode_out(&ip->i_di, dibh->b_data);
 	brelse(dibh);
- out:
+out:
 	gfs2_trans_end(GFS2_SB(&ip->i_inode));
-
 	return error;
 }
 
@@ -1025,9 +1015,8 @@ static int ea_set_block(struct gfs2_inode *ip, struct gfs2_ea_request *er,
 	if (private)
 		ea_set_remove_stuffed(ip, (struct gfs2_ea_location *)private);
 
- out:
+out:
 	brelse(indbh);
-
 	return error;
 }
 
@@ -1285,15 +1274,13 @@ static int ea_acl_chmod_unstuffed(struct gfs2_inode *ip,
 		brelse(bh[x]);
 	}
 
- out:
+out:
 	kfree(bh);
-
 	return error;
 
- fail:
+fail:
 	gfs2_trans_end(sdp);
 	kfree(bh);
-
 	return error;
 }
 
@@ -1442,15 +1429,12 @@ static int ea_dealloc_indirect(struct gfs2_inode *ip)
 
 	gfs2_trans_end(sdp);
 
- out_gunlock:
+out_gunlock:
 	gfs2_glock_dq_m(rlist.rl_rgrps, rlist.rl_ghs);
-
- out_rlist_free:
+out_rlist_free:
 	gfs2_rlist_free(&rlist);
-
- out:
+out:
 	brelse(indbh);
-
 	return error;
 }
 
@@ -1494,9 +1478,8 @@ static int ea_dealloc_block(struct gfs2_inode *ip)
 
 	gfs2_trans_end(sdp);
 
- out_gunlock:
+out_gunlock:
 	gfs2_glock_dq_uninit(&al->al_rgd_gh);
-
 	return error;
 }
 
@@ -1534,15 +1517,12 @@ int gfs2_ea_dealloc(struct gfs2_inode *ip)
 
 	error = ea_dealloc_block(ip);
 
- out_rindex:
+out_rindex:
 	gfs2_glock_dq_uninit(&al->al_ri_gh);
-
- out_quota:
+out_quota:
 	gfs2_quota_unhold(ip);
-
- out_alloc:
+out_alloc:
 	gfs2_alloc_put(ip);
-
 	return error;
 }
 
diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c
index fac271f390bf..1c916fedc04b 100644
--- a/fs/gfs2/glock.c
+++ b/fs/gfs2/glock.c
@@ -2179,9 +2179,8 @@ static int dump_glock(struct gfs2_glock *gl)
 
 	error = 0;
 
- out:
+out:
 	spin_unlock(&gl->gl_spin);
-
 	return error;
 }
 
diff --git a/fs/gfs2/lm.c b/fs/gfs2/lm.c
index fb918c7de655..e60f95cae6c5 100644
--- a/fs/gfs2/lm.c
+++ b/fs/gfs2/lm.c
@@ -76,7 +76,7 @@ int gfs2_lm_mount(struct gfs2_sbd *sdp, int silent)
 		sdp->sd_args.ar_localcaching = 1;
 	}
 
- out:
+out:
 	return error;
 }
 
diff --git a/fs/gfs2/locking/dlm/lock.c b/fs/gfs2/locking/dlm/lock.c
index 2d81d90db097..1f15b6e8b2cf 100644
--- a/fs/gfs2/locking/dlm/lock.c
+++ b/fs/gfs2/locking/dlm/lock.c
@@ -437,7 +437,7 @@ static int hold_null_lock(struct gdlm_lock *lp)
 		gdlm_delete_lp(lpn);
 		lpn = NULL;
 	}
- out:
+out:
 	lp->hold_null = lpn;
 	return error;
 }
diff --git a/fs/gfs2/locking/dlm/mount.c b/fs/gfs2/locking/dlm/mount.c
index f279385774b7..82ac00af84a3 100644
--- a/fs/gfs2/locking/dlm/mount.c
+++ b/fs/gfs2/locking/dlm/mount.c
@@ -164,13 +164,13 @@ static int gdlm_mount(char *table_name, char *host_data,
 	lockstruct->ls_lvb_size = GDLM_LVB_SIZE;
 	return 0;
 
- out_kobj:
+out_kobj:
 	gdlm_kobject_release(ls);
- out_thread:
+out_thread:
 	gdlm_release_threads(ls);
- out_free:
+out_free:
 	kfree(ls);
- out:
+out:
 	return error;
 }
 
@@ -194,7 +194,7 @@ static void gdlm_unmount(lm_lockspace_t *lockspace)
 	rv = gdlm_release_all_locks(ls);
 	if (rv)
 		log_info("gdlm_unmount: %d stray locks freed", rv);
- out:
+out:
 	kfree(ls);
 }
 
diff --git a/fs/gfs2/locking/dlm/thread.c b/fs/gfs2/locking/dlm/thread.c
index 0b4be102e170..a782246d666b 100644
--- a/fs/gfs2/locking/dlm/thread.c
+++ b/fs/gfs2/locking/dlm/thread.c
@@ -206,7 +206,7 @@ static void process_complete(struct gdlm_lock *lp)
 	if (lp->lksb.sb_flags & DLM_SBF_DEMOTED)
 		set_bit(LFL_NOCACHE, &lp->flags);
 
- out:
+out:
 	/*
 	 * This is an internal lock_dlm lock
 	 */
diff --git a/fs/gfs2/meta_io.c b/fs/gfs2/meta_io.c
index 03850b64c072..3f6da00e2f59 100644
--- a/fs/gfs2/meta_io.c
+++ b/fs/gfs2/meta_io.c
@@ -756,7 +756,7 @@ void gfs2_meta_ra(struct gfs2_glock *gl, uint64_t dblock, uint32_t extlen)
 			break;
 	}
 
- out:
+out:
 	brelse(first_bh);
 }
 
diff --git a/fs/gfs2/mount.c b/fs/gfs2/mount.c
index b66027827aaa..257c4a179dc6 100644
--- a/fs/gfs2/mount.c
+++ b/fs/gfs2/mount.c
@@ -203,11 +203,11 @@ int gfs2_mount_args(struct gfs2_sbd *sdp, char *data_arg, int remount)
 
 	return error;
 
- need_value:
+need_value:
 	fs_info(sdp, "need value for option %s\n", o);
 	return -EINVAL;
 
- cant_remount:
+cant_remount:
 	fs_info(sdp, "can't remount with option %s\n", o);
 	return -EINVAL;
 }
diff --git a/fs/gfs2/ops_dentry.c b/fs/gfs2/ops_dentry.c
index a1ba1ec8eef4..fa6ceffc7d82 100644
--- a/fs/gfs2/ops_dentry.c
+++ b/fs/gfs2/ops_dentry.c
@@ -81,31 +81,27 @@ static int gfs2_drevalidate(struct dentry *dentry, struct nameidata *nd)
 		goto fail_gunlock;
 	}
 
- valid_gunlock:
+valid_gunlock:
 	gfs2_glock_dq_uninit(&d_gh);
-
- valid:
+valid:
 	dput(parent);
 	return 1;
 
- invalid_gunlock:
+invalid_gunlock:
 	gfs2_glock_dq_uninit(&d_gh);
-
- invalid:
+invalid:
 	if (inode && S_ISDIR(inode->i_mode)) {
 		if (have_submounts(dentry))
 			goto valid;
 		shrink_dcache_parent(dentry);
 	}
 	d_drop(dentry);
-
 	dput(parent);
 	return 0;
 
- fail_gunlock:
+fail_gunlock:
 	gfs2_glock_dq_uninit(&d_gh);
-
- fail:
+fail:
 	dput(parent);
 	return 0;
 }
diff --git a/fs/gfs2/ops_file.c b/fs/gfs2/ops_file.c
index 07a0c861ac41..53ce7816dc18 100644
--- a/fs/gfs2/ops_file.c
+++ b/fs/gfs2/ops_file.c
@@ -345,9 +345,8 @@ static int readdir_bad(struct file *file, void *dirent, filldir_t filldir)
 
 	file->f_pos = offset;
 
- out:
+out:
 	kfree(fdb);
-
 	return error;
 }
 
diff --git a/fs/gfs2/ops_fstype.c b/fs/gfs2/ops_fstype.c
index 46f910e29bf0..d39314d5dd0a 100644
--- a/fs/gfs2/ops_fstype.c
+++ b/fs/gfs2/ops_fstype.c
@@ -173,9 +173,8 @@ static int init_names(struct gfs2_sbd *sdp, int silent)
 	snprintf(sdp->sd_proto_name, GFS2_FSNAME_LEN, "%s", proto);
 	snprintf(sdp->sd_table_name, GFS2_FSNAME_LEN, "%s", table);
 
- out:
+out:
 	kfree(sb);
-
 	return error;
 }
 
@@ -246,22 +245,17 @@ static int init_locking(struct gfs2_sbd *sdp, struct gfs2_holder *mount_gh,
 
 fail_trans:
 	gfs2_glock_put(sdp->sd_trans_gl);
-
 fail_rename:
 	gfs2_glock_put(sdp->sd_rename_gl);
-
 fail_live:
 	gfs2_glock_dq_uninit(&sdp->sd_live_gh);
-
 fail_mount:
 	gfs2_glock_dq_uninit(mount_gh);
-
 fail:
 	while (sdp->sd_glockd_num--)
 		kthread_stop(sdp->sd_glockd_process[sdp->sd_glockd_num]);
 
 	kthread_stop(sdp->sd_scand_process);
-
 	return error;
 }
 
@@ -451,25 +445,20 @@ static int init_journal(struct gfs2_sbd *sdp, int undo)
 
 	return 0;
 
- fail_recoverd:
+fail_recoverd:
 	kthread_stop(sdp->sd_recoverd_process);
-
- fail_jinode_gh:
+fail_jinode_gh:
 	if (!sdp->sd_args.ar_spectator)
 		gfs2_glock_dq_uninit(&sdp->sd_jinode_gh);
-
- fail_journal_gh:
+fail_journal_gh:
 	if (!sdp->sd_args.ar_spectator)
 		gfs2_glock_dq_uninit(&sdp->sd_journal_gh);
-
- fail_jindex:
+fail_jindex:
 	gfs2_jindex_free(sdp);
 	if (jindex)
 		gfs2_glock_dq_uninit(&ji_gh);
-
- fail:
+fail:
 	iput(sdp->sd_jindex);
-
 	return error;
 }
 
@@ -534,14 +523,11 @@ static int init_inodes(struct gfs2_sbd *sdp, int undo)
 
 fail_qinode:
 	iput(sdp->sd_quota_inode);
-
 fail_rindex:
 	gfs2_clear_rgrpd(sdp);
 	iput(sdp->sd_rindex);
-
 fail_statfs:
 	iput(sdp->sd_statfs_inode);
-
 fail_inum:
 	iput(sdp->sd_inum_inode);
 fail_journal:
@@ -628,27 +614,19 @@ static int init_per_node(struct gfs2_sbd *sdp, int undo)
 
 	return 0;
 
- fail_qc_gh:
+fail_qc_gh:
 	gfs2_glock_dq_uninit(&sdp->sd_qc_gh);
-
- fail_ut_gh:
-
+fail_ut_gh:
 	gfs2_glock_dq_uninit(&sdp->sd_sc_gh);
-
- fail_ir_gh:
+fail_ir_gh:
 	gfs2_glock_dq_uninit(&sdp->sd_ir_gh);
-
- fail_qc_i:
+fail_qc_i:
 	iput(sdp->sd_qc_inode);
-
- fail_ut_i:
-
+fail_ut_i:
 	iput(sdp->sd_sc_inode);
-
- fail_ir_i:
+fail_ir_i:
 	iput(sdp->sd_ir_inode);
-
- fail:
+fail:
 	if (pn)
 		iput(pn);
 	return error;
@@ -781,34 +759,26 @@ static int fill_super(struct super_block *sb, void *data, int silent)
 
 	return 0;
 
- fail_threads:
+fail_threads:
 	init_threads(sdp, UNDO);
-
- fail_per_node:
+fail_per_node:
 	init_per_node(sdp, UNDO);
-
- fail_inodes:
+fail_inodes:
 	init_inodes(sdp, UNDO);
-
- fail_sb:
+fail_sb:
 	init_sb(sdp, 0, UNDO);
-
- fail_locking:
+fail_locking:
 	init_locking(sdp, &mount_gh, UNDO);
-
- fail_lm:
+fail_lm:
 	gfs2_gl_hash_clear(sdp, WAIT);
 	gfs2_lm_unmount(sdp);
 	while (invalidate_inodes(sb))
 		yield();
-
- fail_sys:
+fail_sys:
 	gfs2_sys_fs_del(sdp);
-
- fail:
+fail:
 	vfree(sdp);
 	sb->s_fs_info = NULL;
-
 	return error;
 }
 
@@ -852,6 +822,7 @@ static int fill_super_meta(struct super_block *sb, struct super_block *new,
 
 	return error;
 }
+
 static int set_bdev_super(struct super_block *s, void *data)
 {
 	s->s_bdev = data;
diff --git a/fs/gfs2/ops_inode.c b/fs/gfs2/ops_inode.c
index 1786a485acc5..bd9b9957f707 100644
--- a/fs/gfs2/ops_inode.c
+++ b/fs/gfs2/ops_inode.c
@@ -228,32 +228,25 @@ static int gfs2_link(struct dentry *old_dentry, struct inode *dir,
 
 out_end_trans:
 	gfs2_trans_end(sdp);
-
 out_ipres:
 	if (alloc_required)
 		gfs2_inplace_release(dip);
-
 out_gunlock_q:
 	if (alloc_required)
 		gfs2_quota_unlock(dip);
-
 out_alloc:
 	if (alloc_required)
 		gfs2_alloc_put(dip);
-
 out_gunlock:
 	gfs2_glock_dq_m(2, ghs);
-
 out:
 	gfs2_holder_uninit(ghs);
 	gfs2_holder_uninit(ghs + 1);
-
 	if (!error) {
 		atomic_inc(&inode->i_count);
 		d_instantiate(dentry, inode);
 		mark_inode_dirty(inode);
 	}
-
 	return error;
 }
 
@@ -491,13 +484,11 @@ static int gfs2_rmdir(struct inode *dir, struct dentry *dentry)
 
 	gfs2_trans_end(sdp);
 
- out_gunlock:
+out_gunlock:
 	gfs2_glock_dq_m(2, ghs);
-
- out:
+out:
 	gfs2_holder_uninit(ghs);
 	gfs2_holder_uninit(ghs + 1);
-
 	return error;
 }
 
@@ -966,15 +957,12 @@ static int setattr_chown(struct inode *inode, struct iattr *attr)
 		gfs2_quota_change(ip, ip->i_di.di_blocks, nuid, ngid);
 	}
 
- out_end_trans:
+out_end_trans:
 	gfs2_trans_end(sdp);
-
- out_gunlock_q:
+out_gunlock_q:
 	gfs2_quota_unlock(ip);
-
- out_alloc:
+out_alloc:
 	gfs2_alloc_put(ip);
-
 	return error;
 }
 
@@ -1017,12 +1005,10 @@ static int gfs2_setattr(struct dentry *dentry, struct iattr *attr)
 	else
 		error = gfs2_setattr_simple(ip, attr);
 
- out:
+out:
 	gfs2_glock_dq_uninit(&i_gh);
-
 	if (!error)
 		mark_inode_dirty(inode);
-
 	return error;
 }
 
diff --git a/fs/gfs2/ops_vm.c b/fs/gfs2/ops_vm.c
index 451f48d62e58..32ec8574ac8e 100644
--- a/fs/gfs2/ops_vm.c
+++ b/fs/gfs2/ops_vm.c
@@ -115,18 +115,14 @@ static int alloc_page_backing(struct gfs2_inode *ip, struct page *page)
 
 	gfs2_assert_warn(sdp, al->al_alloced);
 
- out_trans:
+out_trans:
 	gfs2_trans_end(sdp);
-
- out_ipres:
+out_ipres:
 	gfs2_inplace_release(ip);
-
- out_gunlock_q:
+out_gunlock_q:
 	gfs2_quota_unlock(ip);
-
- out:
+out:
 	gfs2_alloc_put(ip);
-
 	return error;
 }
 
diff --git a/fs/gfs2/quota.c b/fs/gfs2/quota.c
index be87983a20a9..572b92eac367 100644
--- a/fs/gfs2/quota.c
+++ b/fs/gfs2/quota.c
@@ -105,7 +105,7 @@ static int qd_alloc(struct gfs2_sbd *sdp, int user, uint32_t id,
 
 	return 0;
 
- fail:
+fail:
 	kfree(qd);
 	return error;
 }
@@ -199,7 +199,7 @@ static int slot_get(struct gfs2_quota_data *qd)
 
 	goto fail;
 
- found:
+found:
 	for (b = 0; b < 8; b++)
 		if (!(byte & (1 << b)))
 			break;
@@ -214,7 +214,7 @@ static int slot_get(struct gfs2_quota_data *qd)
 
 	return 0;
 
- fail:
+fail:
 	qd->qd_slot_count--;
 	spin_unlock(&sdp->sd_quota_spin);
 	return -ENOSPC;
@@ -283,10 +283,9 @@ static int bh_get(struct gfs2_quota_data *qd)
 
 	return 0;
 
- fail_brelse:
+fail_brelse:
 	brelse(bh);
-
- fail:
+fail:
 	qd->qd_bh_count--;
 	mutex_unlock(&sdp->sd_quota_mutex);
 	return error;
@@ -425,10 +424,9 @@ static int qdsb_get(struct gfs2_sbd *sdp, int user, uint32_t id, int create,
 
 	return 0;
 
- fail_slot:
+fail_slot:
 	slot_put(*qdp);
-
- fail:
+fail:
 	qd_put(*qdp);
 	return error;
 }
@@ -482,10 +480,9 @@ int gfs2_quota_hold(struct gfs2_inode *ip, uint32_t uid, uint32_t gid)
 		qd++;
 	}
 
- out:
+out:
 	if (error)
 		gfs2_quota_unhold(ip);
-
 	return error;
 }
 
@@ -728,26 +725,21 @@ static int do_sync(unsigned int num_qd, struct gfs2_quota_data **qda)
 
 	error = 0;
 
- out_end_trans:
+out_end_trans:
 	gfs2_trans_end(sdp);
-
- out_ipres:
+out_ipres:
 	if (nalloc)
 		gfs2_inplace_release(ip);
-
- out_alloc:
+out_alloc:
 	if (nalloc)
 		gfs2_alloc_put(ip);
-
- out_gunlock:
+out_gunlock:
 	gfs2_glock_dq_uninit(&i_gh);
-
- out:
+out:
 	while (qx--)
 		gfs2_glock_dq_uninit(&ghs[qx]);
 	kfree(ghs);
 	gfs2_log_flush(ip->i_gl->gl_sbd, ip->i_gl);
-
 	return error;
 }
 
@@ -764,7 +756,7 @@ static int do_glock(struct gfs2_quota_data *qd, int force_refresh,
 	struct gfs2_quota_lvb *qlvb;
 
 	file_ra_state_init(&ra_state, sdp->sd_quota_inode->i_mapping);
- restart:
+restart:
 	error = gfs2_glock_nq_init(qd->qd_gl, LM_ST_SHARED, 0, q_gh);
 	if (error)
 		return error;
@@ -812,12 +804,10 @@ static int do_glock(struct gfs2_quota_data *qd, int force_refresh,
 
 	return 0;
 
- fail_gunlock:
+fail_gunlock:
 	gfs2_glock_dq_uninit(&i_gh);
-
- fail:
+fail:
 	gfs2_glock_dq_uninit(q_gh);
-
 	return error;
 }
 
@@ -919,7 +909,7 @@ void gfs2_quota_unlock(struct gfs2_inode *ip)
 			qd_unlock(qda[x]);
 	}
 
- out:
+out:
 	gfs2_quota_unhold(ip);
 }
 
@@ -1098,9 +1088,8 @@ int gfs2_quota_read(struct gfs2_sbd *sdp, int user, uint32_t id,
 
 	gfs2_glock_dq_uninit(&q_gh);
 
- out:
+out:
 	qd_put(qd);
-
 	return error;
 }
 #endif  /*  0  */
@@ -1202,7 +1191,7 @@ int gfs2_quota_init(struct gfs2_sbd *sdp)
 
 	return 0;
 
- fail:
+fail:
 	gfs2_quota_cleanup(sdp);
 	return error;
 }
diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c
index e93066f930c1..520266c2044b 100644
--- a/fs/gfs2/super.c
+++ b/fs/gfs2/super.c
@@ -535,7 +535,7 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp)
 
 	error = gfs2_quota_init(sdp);
 	if (error)
-		goto fail_unlinked;
+		goto fail;
 
 	set_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags);
 
@@ -543,9 +543,7 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp)
 
 	return 0;
 
- fail_unlinked:
-
- fail:
+fail:
 	t_gh.gh_flags |= GL_NOCACHE;
 	gfs2_glock_dq_uninit(&t_gh);
 
@@ -625,12 +623,10 @@ int gfs2_statfs_init(struct gfs2_sbd *sdp)
 		brelse(l_bh);
 	}
 
- out_m_bh:
+out_m_bh:
 	brelse(m_bh);
-
- out:
+out:
 	gfs2_glock_dq_uninit(&gh);
-
 	return 0;
 }
 
@@ -715,15 +711,12 @@ int gfs2_statfs_sync(struct gfs2_sbd *sdp)
 
 	gfs2_trans_end(sdp);
 
- out_bh2:
+out_bh2:
 	brelse(l_bh);
-
- out_bh:
+out_bh:
 	brelse(m_bh);
-
- out:
+out:
 	gfs2_glock_dq_uninit(&gh);
-
 	return error;
 }
 
@@ -853,9 +846,8 @@ int gfs2_statfs_slow(struct gfs2_sbd *sdp, struct gfs2_statfs_change *sc)
 
 	gfs2_glock_dq_uninit(&ri_gh);
 
- out:
+out:
 	kfree(gha);
-
 	return error;
 }
 
@@ -924,7 +916,7 @@ static int gfs2_lock_fs_check_clean(struct gfs2_sbd *sdp,
 	if (error)
 		gfs2_glock_dq_uninit(t_gh);
 
- out:
+out:
 	while (!list_empty(&list)) {
 		lfcc = list_entry(list.next, struct lfcc, list);
 		list_del(&lfcc->list);
@@ -932,7 +924,6 @@ static int gfs2_lock_fs_check_clean(struct gfs2_sbd *sdp,
 		kfree(lfcc);
 	}
 	gfs2_glock_dq_uninit(&ji_gh);
-
 	return error;
 }
 
diff --git a/fs/gfs2/sys.c b/fs/gfs2/sys.c
index 3ffa88506c44..79199738c389 100644
--- a/fs/gfs2/sys.c
+++ b/fs/gfs2/sys.c
@@ -543,15 +543,15 @@ int gfs2_sys_fs_add(struct gfs2_sbd *sdp)
 
 	return 0;
 
- fail_args:
+fail_args:
 	sysfs_remove_group(&sdp->sd_kobj, &args_group);
- fail_counters:
+fail_counters:
 	sysfs_remove_group(&sdp->sd_kobj, &counters_group);
- fail_lockstruct:
+fail_lockstruct:
 	sysfs_remove_group(&sdp->sd_kobj, &lockstruct_group);
- fail_reg:
+fail_reg:
 	kobject_unregister(&sdp->sd_kobj);
- fail:
+fail:
 	return error;
 }