diff mbox

[RESEND] Btrfs-progs: fix compile warning in btrfs_free_block_groups()

Message ID 1372868685-4973-1-git-send-email-wangshilong1991@gmail.com (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Wang Shilong July 3, 2013, 4:24 p.m. UTC
From: Wang Shilong <wangsl-fnst@cn.fujitsu.com>

extent-tree.c: In function 'btrfs_free_block_groups':
extent-tree.c:3189:12: warning: cast to pointer from
integer of different size [-Wint-to-pointer-cast]

Signed-off-by: Wang Shilong <wangsl-fnst@cn.fujitsu.com>
---
 extent-tree.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Stefan Behrens July 3, 2013, 4:31 p.m. UTC | #1
On Thu,  4 Jul 2013 00:24:45 +0800, Wang Shilong wrote:
> From: Wang Shilong <wangsl-fnst@cn.fujitsu.com>
> 
> extent-tree.c: In function 'btrfs_free_block_groups':
> extent-tree.c:3189:12: warning: cast to pointer from
> integer of different size [-Wint-to-pointer-cast]
> 
> Signed-off-by: Wang Shilong <wangsl-fnst@cn.fujitsu.com>
> ---
>  extent-tree.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/extent-tree.c b/extent-tree.c
> index b0cfe0a..7561bbb 100644
> --- a/extent-tree.c
> +++ b/extent-tree.c
> @@ -3186,7 +3186,8 @@ int btrfs_free_block_groups(struct btrfs_fs_info *info)
>  			break;
>  		ret = get_state_private(&info->block_group_cache, start, &ptr);
>  		if (!ret) {
> -			cache = (struct btrfs_block_group_cache *)ptr;
> +			cache = (struct btrfs_block_group_cache *)
> +				(unsigned long)ptr;

Shouldn't this be "(uintptr_t)" instead of "(unsigned long)"?

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/extent-tree.c b/extent-tree.c
index b0cfe0a..7561bbb 100644
--- a/extent-tree.c
+++ b/extent-tree.c
@@ -3186,7 +3186,8 @@  int btrfs_free_block_groups(struct btrfs_fs_info *info)
 			break;
 		ret = get_state_private(&info->block_group_cache, start, &ptr);
 		if (!ret) {
-			cache = (struct btrfs_block_group_cache *)ptr;
+			cache = (struct btrfs_block_group_cache *)
+				(unsigned long)ptr;
 			if (cache->free_space_ctl) {
 				btrfs_remove_free_space_cache(cache);
 				kfree(cache->free_space_ctl);