ocfs2: Pull ocfs2_xattr_value_buf up from __ocfs2_remove_xattr_range().
authorJoel Becker <joel.becker@oracle.com>
Tue, 9 Dec 2008 22:30:41 +0000 (14:30 -0800)
committerMark Fasheh <mfasheh@suse.com>
Mon, 5 Jan 2009 16:40:32 +0000 (08:40 -0800)
Place an ocfs2_xattr_value_buf in __ocfs2_xattr_shrink_size() and pass
it down to __ocfs2_remove_xattr_range().

Signed-off-by: Joel Becker <joel.becker@oracle.com>
Signed-off-by: Mark Fasheh <mfasheh@suse.com>
fs/ocfs2/xattr.c

index 123d378aba9eb0b947fd8a34a44a725f7e4e8ef2..3b059cf2eb45b31f59054da0a1263076a05a0541 100644 (file)
@@ -636,8 +636,7 @@ leave:
 }
 
 static int __ocfs2_remove_xattr_range(struct inode *inode,
-                                     struct buffer_head *root_bh,
-                                     struct ocfs2_xattr_value_root *xv,
+                                     struct ocfs2_xattr_value_buf *vb,
                                      u32 cpos, u32 phys_cpos, u32 len,
                                      struct ocfs2_xattr_set_ctxt *ctxt)
 {
@@ -645,16 +644,11 @@ static int __ocfs2_remove_xattr_range(struct inode *inode,
        u64 phys_blkno = ocfs2_clusters_to_blocks(inode->i_sb, phys_cpos);
        handle_t *handle = ctxt->handle;
        struct ocfs2_extent_tree et;
-       struct ocfs2_xattr_value_buf vb = {
-               .vb_bh = root_bh,
-               .vb_xv = xv,
-               .vb_access = ocfs2_journal_access,
-       };
 
-       ocfs2_init_xattr_value_extent_tree(&et, inode, &vb);
+       ocfs2_init_xattr_value_extent_tree(&et, inode, vb);
 
-       ret = vb.vb_access(handle, inode, vb.vb_bh,
-                          OCFS2_JOURNAL_ACCESS_WRITE);
+       ret = vb->vb_access(handle, inode, vb->vb_bh,
+                           OCFS2_JOURNAL_ACCESS_WRITE);
        if (ret) {
                mlog_errno(ret);
                goto out;
@@ -667,9 +661,9 @@ static int __ocfs2_remove_xattr_range(struct inode *inode,
                goto out;
        }
 
-       le32_add_cpu(&vb.vb_xv->xr_clusters, -len);
+       le32_add_cpu(&vb->vb_xv->xr_clusters, -len);
 
-       ret = ocfs2_journal_dirty(handle, vb.vb_bh);
+       ret = ocfs2_journal_dirty(handle, vb->vb_bh);
        if (ret) {
                mlog_errno(ret);
                goto out;
@@ -693,6 +687,11 @@ static int ocfs2_xattr_shrink_size(struct inode *inode,
        int ret = 0;
        u32 trunc_len, cpos, phys_cpos, alloc_size;
        u64 block;
+       struct ocfs2_xattr_value_buf vb = {
+               .vb_bh = root_bh,
+               .vb_xv = xv,
+               .vb_access = ocfs2_journal_access,
+       };
 
        if (old_clusters <= new_clusters)
                return 0;
@@ -701,7 +700,8 @@ static int ocfs2_xattr_shrink_size(struct inode *inode,
        trunc_len = old_clusters - new_clusters;
        while (trunc_len) {
                ret = ocfs2_xattr_get_clusters(inode, cpos, &phys_cpos,
-                                              &alloc_size, &xv->xr_list);
+                                              &alloc_size,
+                                              &vb.vb_xv->xr_list);
                if (ret) {
                        mlog_errno(ret);
                        goto out;
@@ -710,7 +710,7 @@ static int ocfs2_xattr_shrink_size(struct inode *inode,
                if (alloc_size > trunc_len)
                        alloc_size = trunc_len;
 
-               ret = __ocfs2_remove_xattr_range(inode, root_bh, xv, cpos,
+               ret = __ocfs2_remove_xattr_range(inode, &vb, cpos,
                                                 phys_cpos, alloc_size,
                                                 ctxt);
                if (ret) {