[GFS2] Align all labels against LH side
This makes everything consistent.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
diff --git a/fs/gfs2/acl.c b/fs/gfs2/acl.c
index 60c98c0..d846b5a 100644
--- a/fs/gfs2/acl.c
+++ b/fs/gfs2/acl.c
@@ -65,9 +65,8 @@
error = 0;
}
- out:
+out:
posix_acl_release(acl);
-
return error;
}
@@ -132,18 +131,16 @@
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 @@
} else
munge_mode(ip, mode);
- out:
+out:
posix_acl_release(acl);
kfree(er.er_data);
return error;
@@ -303,11 +300,10 @@
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 b75a2f9..bb59783 100644
--- a/fs/gfs2/bmap.c
+++ b/fs/gfs2/bmap.c
@@ -170,12 +170,10 @@
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 @@
break;
}
- out:
+out:
brelse(bh);
-
return error;
}
@@ -788,15 +785,12 @@
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 @@
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 @@
brelse(dibh);
- out:
+out:
gfs2_trans_end(sdp);
-
return error;
}
@@ -1035,7 +1024,7 @@
gfs2_quota_unhold(ip);
- out:
+out:
gfs2_alloc_put(ip);
return error;
}
@@ -1070,11 +1059,9 @@
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 7b8a38e..59d0cbc 100644
--- a/fs/gfs2/dir.c
+++ b/fs/gfs2/dir.c
@@ -1134,9 +1134,8 @@
return error;
- fail:
+fail:
kfree(buf);
-
return error;
}
@@ -1890,23 +1889,18 @@
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 9081822..5a56e65 100644
--- a/fs/gfs2/eattr.c
+++ b/fs/gfs2/eattr.c
@@ -107,7 +107,7 @@
return error;
- fail:
+fail:
gfs2_consist_inode(ip);
return -EIO;
}
@@ -152,9 +152,8 @@
if (error)
break;
}
- out:
+out:
brelse(bh);
-
return error;
}
@@ -319,9 +318,8 @@
gfs2_trans_end(sdp);
- out_gunlock:
+out_gunlock:
gfs2_glock_dq_uninit(&rg_gh);
-
return error;
}
@@ -348,12 +346,10 @@
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 @@
brelse(bh[x]);
}
- out:
+out:
kfree(bh);
-
return error;
}
@@ -747,18 +742,14 @@
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 @@
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 @@
if (private)
ea_set_remove_stuffed(ip, (struct gfs2_ea_location *)private);
- out:
+out:
brelse(indbh);
-
return error;
}
@@ -1285,15 +1274,13 @@
brelse(bh[x]);
}
- out:
+out:
kfree(bh);
-
return error;
- fail:
+fail:
gfs2_trans_end(sdp);
kfree(bh);
-
return error;
}
@@ -1442,15 +1429,12 @@
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 @@
gfs2_trans_end(sdp);
- out_gunlock:
+out_gunlock:
gfs2_glock_dq_uninit(&al->al_rgd_gh);
-
return error;
}
@@ -1534,15 +1517,12 @@
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 fac271f..1c916fe 100644
--- a/fs/gfs2/glock.c
+++ b/fs/gfs2/glock.c
@@ -2179,9 +2179,8 @@
error = 0;
- out:
+out:
spin_unlock(&gl->gl_spin);
-
return error;
}
diff --git a/fs/gfs2/lm.c b/fs/gfs2/lm.c
index fb918c7..e60f95c 100644
--- a/fs/gfs2/lm.c
+++ b/fs/gfs2/lm.c
@@ -76,7 +76,7 @@
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 2d81d90..1f15b6e 100644
--- a/fs/gfs2/locking/dlm/lock.c
+++ b/fs/gfs2/locking/dlm/lock.c
@@ -437,7 +437,7 @@
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 f279385..82ac00a 100644
--- a/fs/gfs2/locking/dlm/mount.c
+++ b/fs/gfs2/locking/dlm/mount.c
@@ -164,13 +164,13 @@
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 @@
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 0b4be10..a782246 100644
--- a/fs/gfs2/locking/dlm/thread.c
+++ b/fs/gfs2/locking/dlm/thread.c
@@ -206,7 +206,7 @@
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 03850b6..3f6da00 100644
--- a/fs/gfs2/meta_io.c
+++ b/fs/gfs2/meta_io.c
@@ -756,7 +756,7 @@
break;
}
- out:
+out:
brelse(first_bh);
}
diff --git a/fs/gfs2/mount.c b/fs/gfs2/mount.c
index b660278..257c4a1 100644
--- a/fs/gfs2/mount.c
+++ b/fs/gfs2/mount.c
@@ -203,11 +203,11 @@
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 a1ba1ec..fa6ceff 100644
--- a/fs/gfs2/ops_dentry.c
+++ b/fs/gfs2/ops_dentry.c
@@ -81,31 +81,27 @@
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 07a0c86..53ce781 100644
--- a/fs/gfs2/ops_file.c
+++ b/fs/gfs2/ops_file.c
@@ -345,9 +345,8 @@
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 46f910e..d39314d 100644
--- a/fs/gfs2/ops_fstype.c
+++ b/fs/gfs2/ops_fstype.c
@@ -173,9 +173,8 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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 1786a48..bd9b995 100644
--- a/fs/gfs2/ops_inode.c
+++ b/fs/gfs2/ops_inode.c
@@ -228,32 +228,25 @@
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 @@
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 @@
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 @@
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 451f48d..32ec857 100644
--- a/fs/gfs2/ops_vm.c
+++ b/fs/gfs2/ops_vm.c
@@ -115,18 +115,14 @@
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 be87983..572b92e 100644
--- a/fs/gfs2/quota.c
+++ b/fs/gfs2/quota.c
@@ -105,7 +105,7 @@
return 0;
- fail:
+fail:
kfree(qd);
return error;
}
@@ -199,7 +199,7 @@
goto fail;
- found:
+found:
for (b = 0; b < 8; b++)
if (!(byte & (1 << b)))
break;
@@ -214,7 +214,7 @@
return 0;
- fail:
+fail:
qd->qd_slot_count--;
spin_unlock(&sdp->sd_quota_spin);
return -ENOSPC;
@@ -283,10 +283,9 @@
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 @@
return 0;
- fail_slot:
+fail_slot:
slot_put(*qdp);
-
- fail:
+fail:
qd_put(*qdp);
return error;
}
@@ -482,10 +480,9 @@
qd++;
}
- out:
+out:
if (error)
gfs2_quota_unhold(ip);
-
return error;
}
@@ -728,26 +725,21 @@
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 @@
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 @@
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 @@
qd_unlock(qda[x]);
}
- out:
+out:
gfs2_quota_unhold(ip);
}
@@ -1098,9 +1088,8 @@
gfs2_glock_dq_uninit(&q_gh);
- out:
+out:
qd_put(qd);
-
return error;
}
#endif /* 0 */
@@ -1202,7 +1191,7 @@
return 0;
- fail:
+fail:
gfs2_quota_cleanup(sdp);
return error;
}
diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c
index e93066f..520266c 100644
--- a/fs/gfs2/super.c
+++ b/fs/gfs2/super.c
@@ -535,7 +535,7 @@
error = gfs2_quota_init(sdp);
if (error)
- goto fail_unlinked;
+ goto fail;
set_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags);
@@ -543,9 +543,7 @@
return 0;
- fail_unlinked:
-
- fail:
+fail:
t_gh.gh_flags |= GL_NOCACHE;
gfs2_glock_dq_uninit(&t_gh);
@@ -625,12 +623,10 @@
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 @@
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 @@
gfs2_glock_dq_uninit(&ri_gh);
- out:
+out:
kfree(gha);
-
return error;
}
@@ -924,7 +916,7 @@
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 @@
kfree(lfcc);
}
gfs2_glock_dq_uninit(&ji_gh);
-
return error;
}
diff --git a/fs/gfs2/sys.c b/fs/gfs2/sys.c
index 3ffa885..7919973 100644
--- a/fs/gfs2/sys.c
+++ b/fs/gfs2/sys.c
@@ -543,15 +543,15 @@
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;
}