Wang Shilong
2013-Jul-03 16:24 UTC
[PATCH RESEND] Btrfs-progs: fix compile warning in btrfs_free_block_groups()
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;
if (cache->free_space_ctl) {
btrfs_remove_free_space_cache(cache);
kfree(cache->free_space_ctl);
--
1.7.11.7
--
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
Stefan Behrens
2013-Jul-03 16:31 UTC
Re: [PATCH RESEND] Btrfs-progs: fix compile warning in btrfs_free_block_groups()
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