On 09/18/2012 10:05 PM, Fengguang Wu wrote:
> Hi Liu,
> 
> FYI, there are new compile warnings show up in
> 
> tree:   git://git.kernel.org/pub/scm/linux/kernel/git/josef/btrfs-next.git 
> master
> head:   6a47d2266699f2d7903d26f178ed135dd9a109cf
> commit: 434728b4678badfad6e8a35df26034a081014254 [66/68] Btrfs: 
> snapshot-aware defrag
> config: x86_64-randconfig-s345 (attached as .config)
> 

Thanks for the report :)

thanks,
liubo

> All error/warnings:
> 
> 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]
> In file included from fs/btrfs/inode.c:43:0:
> 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]
> In file included from fs/btrfs/inode.c:43:0:
> 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]
> In file included from fs/btrfs/inode.c:43:0:
> fs/btrfs/ctree.h:3340:5: note: expected 'u64' but argument is of type 'u64 *'
> 
> vim +2185 fs/btrfs/inode.c
>   2175                start = backref->file_pos;
>   2176                if (backref->extent_offset < old->extent_offset + 
> old->offset)
>   2177                        start += old->extent_offset + old->offset -
>   2178                                 backref->extent_offset;
>   2179        
>   2180                len = min(backref->extent_offset + backref->num_bytes,
>   2181                          old->extent_offset + old->offset + old->len);
>   2182                len -= max(backref->extent_offset, old->extent_offset + 
> old->offset);
>   2183        
>   2184                ret = btrfs_drop_extents(trans, inode, start,
>> 2185                                  start + len, &hint_byte, 1);
>   2186                if (ret)
>   2187                        goto out_free_path;
>   2188        again:
>   2189                key.objectid = btrfs_ino(inode);
>   2190                key.type = BTRFS_EXTENT_DATA_KEY;
>   2191                key.offset = start;
>   2192        
>   2193                if (merge) {
>   2194                        struct btrfs_file_extent_item *fi;
>   2195                        u64 extent_len;
> 
> ---
> 0-DAY kernel build testing backend         Open Source Technology Centre
> Fengguang Wu, Yuanhan Liu                              Intel Corporation
> 

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

Reply via email to