| Message ID | 5188E3EF.3000807@cn.fujitsu.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
oops, i miswrite the patch titile Btrfs-progs to Btrfs.. Sorry my bad, will resend. Thanks, Wang > See the warnings below: > > [CC] btrfs-list.o > btrfs-list.c: In function 'filter_by_parent': > btrfs-list.c:1183:34: warning: cast to pointer from integer of different size > [-Wint-to-pointer-cast] > [CC] cmds-subvolume.o > cmds-subvolume.c: In function 'cmd_subvol_show': > cmds-subvolume.c:917:5: warning: cast from pointer to integer of different size > [-Wpointer-to-int-cast] > [CC] cmds-restore.o > cmds-restore.c: In function 'decompress_lzo': > cmds-restore.c:116:10: warning: passing argument 4 of 'lzo1x_decompress_safe' > from incompatible pointer type [enabled by default] > In file included from cmds-restore.c:31:0: > /usr/include/lzo/lzo1x.h:77:1: note: expected 'lzo_uint *' but argument is of > type 'size_t *' > > Reported-by: Russell Coker <russell@coker.com.au> > Signed-off-by: Wang Shilong <wangsl-fnst@cn.fujitsu.com> > --- > btrfs-list.c | 2 +- > cmds-restore.c | 3 ++- > cmds-subvolume.c | 2 +- > 3 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/btrfs-list.c b/btrfs-list.c > index a748d5e..e4fc994 100644 > --- a/btrfs-list.c > +++ b/btrfs-list.c > @@ -1180,7 +1180,7 @@ static int filter_full_path(struct root_info *ri, u64 data) > > static int filter_by_parent(struct root_info *ri, u64 data) > { > - return !uuid_compare(ri->puuid, (u8 *)data); > + return !uuid_compare(ri->puuid, (u8 *)(unsigned long)data); > } > > static btrfs_list_filter_func all_filter_funcs[] = { > diff --git a/cmds-restore.c b/cmds-restore.c > index c75e187..3098f8d 100644 > --- a/cmds-restore.c > +++ b/cmds-restore.c > @@ -113,7 +113,8 @@ static int decompress_lzo(unsigned char *inbuf, char *outbuf, u64 compress_len, > > new_len = lzo1x_worst_compress(PAGE_CACHE_SIZE); > ret = lzo1x_decompress_safe((const unsigned char *)inbuf, in_len, > - (unsigned char *)outbuf, &new_len, NULL); > + (unsigned char *)outbuf, > + (void *)&new_len, NULL); > if (ret != LZO_E_OK) { > fprintf(stderr, "failed to inflate: %d\n", ret); > return -1; > diff --git a/cmds-subvolume.c b/cmds-subvolume.c > index 74e2130..1bc7867 100644 > --- a/cmds-subvolume.c > +++ b/cmds-subvolume.c > @@ -914,7 +914,7 @@ static int cmd_subvol_show(int argc, char **argv) > printf("\tSnapshot(s):\n"); > filter_set = btrfs_list_alloc_filter_set(); > btrfs_list_setup_filter(&filter_set, BTRFS_LIST_FILTER_BY_PARENT, > - (u64)get_ri.uuid); > + (u64)(unsigned long)get_ri.uuid); > btrfs_list_setup_print_column(BTRFS_LIST_PATH); > btrfs_list_subvols_print(fd, filter_set, NULL, BTRFS_LIST_LAYOUT_RAW, > 1, raw_prefix); -- 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 --git a/btrfs-list.c b/btrfs-list.c index a748d5e..e4fc994 100644 --- a/btrfs-list.c +++ b/btrfs-list.c @@ -1180,7 +1180,7 @@ static int filter_full_path(struct root_info *ri, u64 data) static int filter_by_parent(struct root_info *ri, u64 data) { - return !uuid_compare(ri->puuid, (u8 *)data); + return !uuid_compare(ri->puuid, (u8 *)(unsigned long)data); } static btrfs_list_filter_func all_filter_funcs[] = { diff --git a/cmds-restore.c b/cmds-restore.c index c75e187..3098f8d 100644 --- a/cmds-restore.c +++ b/cmds-restore.c @@ -113,7 +113,8 @@ static int decompress_lzo(unsigned char *inbuf, char *outbuf, u64 compress_len, new_len = lzo1x_worst_compress(PAGE_CACHE_SIZE); ret = lzo1x_decompress_safe((const unsigned char *)inbuf, in_len, - (unsigned char *)outbuf, &new_len, NULL); + (unsigned char *)outbuf, + (void *)&new_len, NULL); if (ret != LZO_E_OK) { fprintf(stderr, "failed to inflate: %d\n", ret); return -1; diff --git a/cmds-subvolume.c b/cmds-subvolume.c index 74e2130..1bc7867 100644 --- a/cmds-subvolume.c +++ b/cmds-subvolume.c @@ -914,7 +914,7 @@ static int cmd_subvol_show(int argc, char **argv) printf("\tSnapshot(s):\n"); filter_set = btrfs_list_alloc_filter_set(); btrfs_list_setup_filter(&filter_set, BTRFS_LIST_FILTER_BY_PARENT, - (u64)get_ri.uuid); + (u64)(unsigned long)get_ri.uuid); btrfs_list_setup_print_column(BTRFS_LIST_PATH); btrfs_list_subvols_print(fd, filter_set, NULL, BTRFS_LIST_LAYOUT_RAW, 1, raw_prefix);
See the warnings below: [CC] btrfs-list.o btrfs-list.c: In function 'filter_by_parent': btrfs-list.c:1183:34: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] [CC] cmds-subvolume.o cmds-subvolume.c: In function 'cmd_subvol_show': cmds-subvolume.c:917:5: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] [CC] cmds-restore.o cmds-restore.c: In function 'decompress_lzo': cmds-restore.c:116:10: warning: passing argument 4 of 'lzo1x_decompress_safe' from incompatible pointer type [enabled by default] In file included from cmds-restore.c:31:0: /usr/include/lzo/lzo1x.h:77:1: note: expected 'lzo_uint *' but argument is of type 'size_t *' Reported-by: Russell Coker <russell@coker.com.au> Signed-off-by: Wang Shilong <wangsl-fnst@cn.fujitsu.com> --- btrfs-list.c | 2 +- cmds-restore.c | 3 ++- cmds-subvolume.c | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-)