On Wed, Mar 01, 2017 at 09:36:38AM +0200, Nikolay Borisov wrote:
> Kbuild reported the following warning:
>
> fs/btrfs/scrub.c: In function 'check_extent_to_block':
> >> fs/btrfs/scrub.c:4259:24: error: passing argument 1 of 'btrfs_get_extent'
> >> from incompatible pointer type [-Werror=incompatible-pointer-types]
> em = btrfs_get_extent(inode, NULL, 0, start, len, 0);
> ^~~~~
> In file included from fs/btrfs/scrub.c:21:0:
> fs/btrfs/ctree.h:3169:20: note: expected 'struct inode *' but argument is
> of type 'struct btrfs_inode *'
> struct extent_map *btrfs_get_extent(struct inode *inode, struct page
> *page,
> ^~~~~~~~~~~~~~~~
> cc1: some warnings being treated as errors
>
> vim +/btrfs_get_extent +4259 fs/btrfs/scrub.c
>
> 32159242 Gui Hecheng 2014-11-10 4253 if (ordered) {
> 32159242 Gui Hecheng 2014-11-10 4254
> btrfs_put_ordered_extent(ordered);
> 32159242 Gui Hecheng 2014-11-10 4255 ret = 1;
> 32159242 Gui Hecheng 2014-11-10 4256 goto out_unlock;
> 32159242 Gui Hecheng 2014-11-10 4257 }
> 32159242 Gui Hecheng 2014-11-10 4258
> 32159242 Gui Hecheng 2014-11-10 @4259 em = btrfs_get_extent(inode,
> NULL, 0, start, len, 0);
> 32159242 Gui Hecheng 2014-11-10 4260 if (IS_ERR(em)) {
> 32159242 Gui Hecheng 2014-11-10 4261 ret = PTR_ERR(em);
> 32159242 Gui Hecheng 2014-11-10 4262 goto out_unlock;
>
> I guess changing the definition of btrfs_get_extent in ctree.h got missed to
> being converted to struct btrfs_inode. Could you be able to fix it up?
Seems the problem got fixed by some following patch. The exact commit
1c8c9c5216295711c79 fails to build but the whole branch is ok. As it's a
minor annoyance I'd rather skip redoing the pull request.
--
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
