]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
Btrfs: fix a compiling warning
authorLiu Bo <bo.li.liu@oracle.com>
Tue, 18 Sep 2012 14:27:45 +0000 (08:27 -0600)
committerChris Mason <chris.mason@fusionio.com>
Thu, 4 Oct 2012 13:40:02 +0000 (09:40 -0400)
NOTE:
This is against btrfs-next git repo.

fs/btrfs/inode.c: In function ‘relink_extent_backref’:
fs/btrfs/inode.c:2185:6: warning: passing argument 2 of
‘btrfs_drop_extents’ from incompatible pointer type [enabled by default]
fs/btrfs/ctree.h:3340:5: note: expected ‘struct btrfs_root *’ but argument is of type ‘struct inode *’
fs/btrfs/inode.c:2185:6: warning: passing argument 3 of ‘btrfs_drop_extents’ makes pointer from integer without a cast [enabled by default]
fs/btrfs/ctree.h:3340:5: note: expected ‘struct inode *’ but argument is of type ‘u64’
fs/btrfs/inode.c:2185:6: warning: passing argument 5 of ‘btrfs_drop_extents’ makes integer from pointer without a cast [enabled by default]
fs/btrfs/ctree.h:3340:5: note: expected ‘u64’ but argument is of type ‘u64 *’

We didn't update the related functions after someone changed the API.

This fixes it.

Reported-by: <fengguang.wu@intel.com>
Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
fs/btrfs/inode.c

index 2a6e9fbcd02b32d4ae4ddc3d10df6e93762f283e..1f85d37aaec755474aea90cffdb78cc0e11e12f3 100644 (file)
@@ -2095,7 +2095,6 @@ static noinline int relink_extent_backref(struct btrfs_path *path,
        struct inode *inode;
        struct extent_state *cached = NULL;
        int ret = 0;
-       u64 hint_byte;
        u64 start;
        u64 len;
        bool merge = false;
@@ -2181,8 +2180,8 @@ static noinline int relink_extent_backref(struct btrfs_path *path,
                  old->extent_offset + old->offset + old->len);
        len -= max(backref->extent_offset, old->extent_offset + old->offset);
 
-       ret = btrfs_drop_extents(trans, inode, start,
-                                start + len, &hint_byte, 1);
+       ret = btrfs_drop_extents(trans, BTRFS_I(inode)->root, inode, start,
+                                start + len, 1);
        if (ret)
                goto out_free_path;
 again: