Commit 043beebb authored by Joel Becker's avatar Joel Becker

ocfs2: ocfs2_truncate_rec() doesn't need struct inode.

It's not using it anymore.  Remove it from the parameter list.
Signed-off-by: default avatarJoel Becker <joel.becker@oracle.com>
parent d401dc12
...@@ -5223,16 +5223,16 @@ out: ...@@ -5223,16 +5223,16 @@ out:
return ret; return ret;
} }
static int ocfs2_truncate_rec(struct inode *inode, handle_t *handle, static int ocfs2_truncate_rec(handle_t *handle,
struct ocfs2_extent_tree *et,
struct ocfs2_path *path, int index, struct ocfs2_path *path, int index,
struct ocfs2_cached_dealloc_ctxt *dealloc, struct ocfs2_cached_dealloc_ctxt *dealloc,
u32 cpos, u32 len, u32 cpos, u32 len)
struct ocfs2_extent_tree *et)
{ {
int ret; int ret;
u32 left_cpos, rec_range, trunc_range; u32 left_cpos, rec_range, trunc_range;
int wants_rotate = 0, is_rightmost_tree_rec = 0; int wants_rotate = 0, is_rightmost_tree_rec = 0;
struct super_block *sb = inode->i_sb; struct super_block *sb = ocfs2_metadata_cache_get_super(et->et_ci);
struct ocfs2_path *left_path = NULL; struct ocfs2_path *left_path = NULL;
struct ocfs2_extent_list *el = path_leaf_el(path); struct ocfs2_extent_list *el = path_leaf_el(path);
struct ocfs2_extent_rec *rec; struct ocfs2_extent_rec *rec;
...@@ -5271,14 +5271,13 @@ static int ocfs2_truncate_rec(struct inode *inode, handle_t *handle, ...@@ -5271,14 +5271,13 @@ static int ocfs2_truncate_rec(struct inode *inode, handle_t *handle,
* by this leaf and the one to it's left. * by this leaf and the one to it's left.
* *
* There are two cases we can skip: * There are two cases we can skip:
* 1) Path is the leftmost one in our inode tree. * 1) Path is the leftmost one in our btree.
* 2) The leaf is rightmost and will be empty after * 2) The leaf is rightmost and will be empty after
* we remove the extent record - the rotate code * we remove the extent record - the rotate code
* knows how to update the newly formed edge. * knows how to update the newly formed edge.
*/ */
ret = ocfs2_find_cpos_for_left_leaf(inode->i_sb, path, ret = ocfs2_find_cpos_for_left_leaf(sb, path, &left_cpos);
&left_cpos);
if (ret) { if (ret) {
mlog_errno(ret); mlog_errno(ret);
goto out; goto out;
...@@ -5353,8 +5352,9 @@ static int ocfs2_truncate_rec(struct inode *inode, handle_t *handle, ...@@ -5353,8 +5352,9 @@ static int ocfs2_truncate_rec(struct inode *inode, handle_t *handle,
ocfs2_adjust_rightmost_records(handle, et, path, rec); ocfs2_adjust_rightmost_records(handle, et, path, rec);
} else { } else {
/* Caller should have trapped this. */ /* Caller should have trapped this. */
mlog(ML_ERROR, "Inode %llu: Invalid record truncate: (%u, %u) " mlog(ML_ERROR, "Owner %llu: Invalid record truncate: (%u, %u) "
"(%u, %u)\n", (unsigned long long)OCFS2_I(inode)->ip_blkno, "(%u, %u)\n",
(unsigned long long)ocfs2_metadata_cache_owner(et->et_ci),
le32_to_cpu(rec->e_cpos), le32_to_cpu(rec->e_cpos),
le16_to_cpu(rec->e_leaf_clusters), cpos, len); le16_to_cpu(rec->e_leaf_clusters), cpos, len);
BUG(); BUG();
...@@ -5447,8 +5447,8 @@ int ocfs2_remove_extent(struct inode *inode, ...@@ -5447,8 +5447,8 @@ int ocfs2_remove_extent(struct inode *inode,
le32_to_cpu(rec->e_cpos), ocfs2_rec_clusters(el, rec)); le32_to_cpu(rec->e_cpos), ocfs2_rec_clusters(el, rec));
if (le32_to_cpu(rec->e_cpos) == cpos || rec_range == trunc_range) { if (le32_to_cpu(rec->e_cpos) == cpos || rec_range == trunc_range) {
ret = ocfs2_truncate_rec(inode, handle, path, index, dealloc, ret = ocfs2_truncate_rec(handle, et, path, index, dealloc,
cpos, len, et); cpos, len);
if (ret) { if (ret) {
mlog_errno(ret); mlog_errno(ret);
goto out; goto out;
...@@ -5502,8 +5502,8 @@ int ocfs2_remove_extent(struct inode *inode, ...@@ -5502,8 +5502,8 @@ int ocfs2_remove_extent(struct inode *inode,
goto out; goto out;
} }
ret = ocfs2_truncate_rec(inode, handle, path, index, dealloc, ret = ocfs2_truncate_rec(handle, et, path, index, dealloc,
cpos, len, et); cpos, len);
if (ret) { if (ret) {
mlog_errno(ret); mlog_errno(ret);
goto out; goto out;
......
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