This makes everything consistent.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
error = 0;
}
- out:
+out:
posix_acl_release(acl);
-
return error;
}
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;
}
} else
munge_mode(ip, mode);
- out:
+out:
posix_acl_release(acl);
kfree(er.er_data);
return error;
error = gfs2_ea_acl_chmod(ip, &el, attr, data);
}
- out:
+out:
posix_acl_release(acl);
brelse(el.el_bh);
kfree(data);
-
return error;
}
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;
}
break;
}
- out:
+out:
brelse(bh);
-
return error;
}
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;
}
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;
}
brelse(dibh);
- out:
+out:
gfs2_trans_end(sdp);
-
return error;
}
gfs2_quota_unhold(ip);
- out:
+out:
gfs2_alloc_put(ip);
return error;
}
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;
}
return error;
- fail:
+fail:
kfree(buf);
-
return error;
}
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;
}
return error;
- fail:
+fail:
gfs2_consist_inode(ip);
return -EIO;
}
if (error)
break;
}
- out:
+out:
brelse(bh);
-
return error;
}
gfs2_trans_end(sdp);
- out_gunlock:
+out_gunlock:
gfs2_glock_dq_uninit(&rg_gh);
-
return error;
}
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;
}
brelse(bh[x]);
}
- out:
+out:
kfree(bh);
-
return error;
}
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;
}
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;
}
if (private)
ea_set_remove_stuffed(ip, (struct gfs2_ea_location *)private);
- out:
+out:
brelse(indbh);
-
return error;
}
brelse(bh[x]);
}
- out:
+out:
kfree(bh);
-
return error;
- fail:
+fail:
gfs2_trans_end(sdp);
kfree(bh);
-
return error;
}
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;
}
gfs2_trans_end(sdp);
- out_gunlock:
+out_gunlock:
gfs2_glock_dq_uninit(&al->al_rgd_gh);
-
return error;
}
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;
}
error = 0;
- out:
+out:
spin_unlock(&gl->gl_spin);
-
return error;
}
sdp->sd_args.ar_localcaching = 1;
}
- out:
+out:
return error;
}
gdlm_delete_lp(lpn);
lpn = NULL;
}
- out:
+out:
lp->hold_null = lpn;
return error;
}
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;
}
rv = gdlm_release_all_locks(ls);
if (rv)
log_info("gdlm_unmount: %d stray locks freed", rv);
- out:
+out:
kfree(ls);
}
if (lp->lksb.sb_flags & DLM_SBF_DEMOTED)
set_bit(LFL_NOCACHE, &lp->flags);
- out:
+out:
/*
* This is an internal lock_dlm lock
*/
break;
}
- out:
+out:
brelse(first_bh);
}
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;
}
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;
}
file->f_pos = offset;
- out:
+out:
kfree(fdb);
-
return error;
}
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;
}
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;
}
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;
}
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:
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;
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;
}
return error;
}
+
static int set_bdev_super(struct super_block *s, void *data)
{
s->s_bdev = data;
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;
}
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;
}
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;
}
else
error = gfs2_setattr_simple(ip, attr);
- out:
+out:
gfs2_glock_dq_uninit(&i_gh);
-
if (!error)
mark_inode_dirty(inode);
-
return error;
}
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;
}
return 0;
- fail:
+fail:
kfree(qd);
return error;
}
goto fail;
- found:
+found:
for (b = 0; b < 8; b++)
if (!(byte & (1 << b)))
break;
return 0;
- fail:
+fail:
qd->qd_slot_count--;
spin_unlock(&sdp->sd_quota_spin);
return -ENOSPC;
return 0;
- fail_brelse:
+fail_brelse:
brelse(bh);
-
- fail:
+fail:
qd->qd_bh_count--;
mutex_unlock(&sdp->sd_quota_mutex);
return error;
return 0;
- fail_slot:
+fail_slot:
slot_put(*qdp);
-
- fail:
+fail:
qd_put(*qdp);
return error;
}
qd++;
}
- out:
+out:
if (error)
gfs2_quota_unhold(ip);
-
return error;
}
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;
}
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;
return 0;
- fail_gunlock:
+fail_gunlock:
gfs2_glock_dq_uninit(&i_gh);
-
- fail:
+fail:
gfs2_glock_dq_uninit(q_gh);
-
return error;
}
qd_unlock(qda[x]);
}
- out:
+out:
gfs2_quota_unhold(ip);
}
gfs2_glock_dq_uninit(&q_gh);
- out:
+out:
qd_put(qd);
-
return error;
}
#endif /* 0 */
return 0;
- fail:
+fail:
gfs2_quota_cleanup(sdp);
return error;
}
error = gfs2_quota_init(sdp);
if (error)
- goto fail_unlinked;
+ goto fail;
set_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags);
return 0;
- fail_unlinked:
-
- fail:
+fail:
t_gh.gh_flags |= GL_NOCACHE;
gfs2_glock_dq_uninit(&t_gh);
brelse(l_bh);
}
- out_m_bh:
+out_m_bh:
brelse(m_bh);
-
- out:
+out:
gfs2_glock_dq_uninit(&gh);
-
return 0;
}
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;
}
gfs2_glock_dq_uninit(&ri_gh);
- out:
+out:
kfree(gha);
-
return error;
}
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);
kfree(lfcc);
}
gfs2_glock_dq_uninit(&ji_gh);
-
return error;
}
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;
}