Commit 23d828fc authored by Bob Peterson's avatar Bob Peterson Committed by Andreas Gruenbacher

gfs2: rename variable error to ret in gfs2_evict_inode

Function gfs2_evict_inode is too big and unreadable. This patch is just
a baby step toward improving that. This first step just renames variable
error to ret. This will help make future patches more readable.
Signed-off-by: default avatarBob Peterson <rpeterso@redhat.com>
Signed-off-by: default avatarAndreas Gruenbacher <agruenba@redhat.com>
parent e8a8023e
...@@ -1338,7 +1338,7 @@ static void gfs2_evict_inode(struct inode *inode) ...@@ -1338,7 +1338,7 @@ static void gfs2_evict_inode(struct inode *inode)
struct gfs2_inode *ip = GFS2_I(inode); struct gfs2_inode *ip = GFS2_I(inode);
struct gfs2_holder gh; struct gfs2_holder gh;
struct address_space *metamapping; struct address_space *metamapping;
int error; int ret;
if (test_bit(GIF_FREE_VFS_INODE, &ip->i_flags)) { if (test_bit(GIF_FREE_VFS_INODE, &ip->i_flags)) {
clear_inode(inode); clear_inode(inode);
...@@ -1362,8 +1362,8 @@ static void gfs2_evict_inode(struct inode *inode) ...@@ -1362,8 +1362,8 @@ static void gfs2_evict_inode(struct inode *inode)
goto out; goto out;
/* Must not read inode block until block type has been verified */ /* Must not read inode block until block type has been verified */
error = gfs2_glock_nq_init(ip->i_gl, LM_ST_EXCLUSIVE, GL_SKIP, &gh); ret = gfs2_glock_nq_init(ip->i_gl, LM_ST_EXCLUSIVE, GL_SKIP, &gh);
if (unlikely(error)) { if (unlikely(ret)) {
glock_clear_object(ip->i_iopen_gh.gh_gl, ip); glock_clear_object(ip->i_iopen_gh.gh_gl, ip);
ip->i_iopen_gh.gh_flags |= GL_NOCACHE; ip->i_iopen_gh.gh_flags |= GL_NOCACHE;
gfs2_glock_dq_uninit(&ip->i_iopen_gh); gfs2_glock_dq_uninit(&ip->i_iopen_gh);
...@@ -1372,13 +1372,13 @@ static void gfs2_evict_inode(struct inode *inode) ...@@ -1372,13 +1372,13 @@ static void gfs2_evict_inode(struct inode *inode)
if (gfs2_inode_already_deleted(ip->i_gl, ip->i_no_formal_ino)) if (gfs2_inode_already_deleted(ip->i_gl, ip->i_no_formal_ino))
goto out_truncate; goto out_truncate;
error = gfs2_check_blk_type(sdp, ip->i_no_addr, GFS2_BLKST_UNLINKED); ret = gfs2_check_blk_type(sdp, ip->i_no_addr, GFS2_BLKST_UNLINKED);
if (error) if (ret)
goto out_truncate; goto out_truncate;
if (test_bit(GIF_INVALID, &ip->i_flags)) { if (test_bit(GIF_INVALID, &ip->i_flags)) {
error = gfs2_inode_refresh(ip); ret = gfs2_inode_refresh(ip);
if (error) if (ret)
goto out_truncate; goto out_truncate;
} }
...@@ -1399,20 +1399,20 @@ static void gfs2_evict_inode(struct inode *inode) ...@@ -1399,20 +1399,20 @@ static void gfs2_evict_inode(struct inode *inode)
if (S_ISDIR(inode->i_mode) && if (S_ISDIR(inode->i_mode) &&
(ip->i_diskflags & GFS2_DIF_EXHASH)) { (ip->i_diskflags & GFS2_DIF_EXHASH)) {
error = gfs2_dir_exhash_dealloc(ip); ret = gfs2_dir_exhash_dealloc(ip);
if (error) if (ret)
goto out_unlock; goto out_unlock;
} }
if (ip->i_eattr) { if (ip->i_eattr) {
error = gfs2_ea_dealloc(ip); ret = gfs2_ea_dealloc(ip);
if (error) if (ret)
goto out_unlock; goto out_unlock;
} }
if (!gfs2_is_stuffed(ip)) { if (!gfs2_is_stuffed(ip)) {
error = gfs2_file_dealloc(ip); ret = gfs2_file_dealloc(ip);
if (error) if (ret)
goto out_unlock; goto out_unlock;
} }
...@@ -1421,7 +1421,7 @@ static void gfs2_evict_inode(struct inode *inode) ...@@ -1421,7 +1421,7 @@ static void gfs2_evict_inode(struct inode *inode)
location and try to set gl_object again. We clear gl_object here so location and try to set gl_object again. We clear gl_object here so
that subsequent inode creates don't see an old gl_object. */ that subsequent inode creates don't see an old gl_object. */
glock_clear_object(ip->i_gl, ip); glock_clear_object(ip->i_gl, ip);
error = gfs2_dinode_dealloc(ip); ret = gfs2_dinode_dealloc(ip);
gfs2_inode_remember_delete(ip->i_gl, ip->i_no_formal_ino); gfs2_inode_remember_delete(ip->i_gl, ip->i_no_formal_ino);
goto out_unlock; goto out_unlock;
...@@ -1436,8 +1436,8 @@ static void gfs2_evict_inode(struct inode *inode) ...@@ -1436,8 +1436,8 @@ static void gfs2_evict_inode(struct inode *inode)
write_inode_now(inode, 1); write_inode_now(inode, 1);
gfs2_ail_flush(ip->i_gl, 0); gfs2_ail_flush(ip->i_gl, 0);
error = gfs2_trans_begin(sdp, 0, sdp->sd_jdesc->jd_blocks); ret = gfs2_trans_begin(sdp, 0, sdp->sd_jdesc->jd_blocks);
if (error) if (ret)
goto out_unlock; goto out_unlock;
/* Needs to be done before glock release & also in a transaction */ /* Needs to be done before glock release & also in a transaction */
truncate_inode_pages(&inode->i_data, 0); truncate_inode_pages(&inode->i_data, 0);
...@@ -1452,8 +1452,8 @@ static void gfs2_evict_inode(struct inode *inode) ...@@ -1452,8 +1452,8 @@ static void gfs2_evict_inode(struct inode *inode)
glock_clear_object(ip->i_gl, ip); glock_clear_object(ip->i_gl, ip);
gfs2_glock_dq_uninit(&gh); gfs2_glock_dq_uninit(&gh);
} }
if (error && error != GLR_TRYFAILED && error != -EROFS) if (ret && ret != GLR_TRYFAILED && ret != -EROFS)
fs_warn(sdp, "gfs2_evict_inode: %d\n", error); fs_warn(sdp, "gfs2_evict_inode: %d\n", ret);
out: out:
truncate_inode_pages_final(&inode->i_data); truncate_inode_pages_final(&inode->i_data);
if (ip->i_qadata) if (ip->i_qadata)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment