diff mbox

Btrfs: fix a compiling warning

Message ID 1347978465-20500-1-git-send-email-bo.li.liu@oracle.com (mailing list archive)
State New, archived
Headers show

Commit Message

Liu Bo Sept. 18, 2012, 2:27 p.m. UTC
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 |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)
diff mbox

Patch

diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
index 2a6e9fb..1f85d37 100644
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -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: