Re: linux-next: build failure after merge of the xarray tree

Leon Romanovsky
2019-02-21T12:34:42Z

[-- Attachment #1: Type: text/plain, Size: 5405 bytes --]

On Thu, Feb 21, 2019 at 05:13:32PM +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the xarray tree, today's linux-next build (powerpc
> ppc64_defconfig) failed like this:
>
> In file included from include/linux/uio.h:12,
> from include/linux/socket.h:8,
> from include/rdma/rdma_cm.h:37,
> from drivers/infiniband/core/restrack.c:6:
> drivers/infiniband/core/restrack.c: In function 'rt_xa_alloc_cyclic':
> include/linux/kernel.h:40:18: warning: passing argument 3 of '__xa_alloc' makes pointer from integer without a cast [-Wint-conversion]
> #define U32_MAX ((u32)~0U)
> ^~~~~~~~~~
> drivers/infiniband/core/restrack.c:26:27: note: in expansion of macro 'U32_MAX'
> err = __xa_alloc(xa, id, U32_MAX, entry, GFP_KERNEL);
> ^~~~~~~
> In file included from include/linux/radix-tree.h:31,
> from include/linux/fs.h:15,
> from include/linux/seq_file.h:11,
> from arch/powerpc/include/asm/machdep.h:12,
> from arch/powerpc/include/asm/archrandom.h:7,
> from include/linux/random.h:166,
> from include/linux/net.h:22,
> from include/linux/skbuff.h:29,
> from include/linux/if_arp.h:26,
> from include/rdma/ib_addr.h:39,
> from include/rdma/rdma_cm.h:39,
> from drivers/infiniband/core/restrack.c:6:
> include/linux/xarray.h:524:61: note: expected 'void *' but argument is of type 'unsigned int'
> int __must_check __xa_alloc(struct xarray *, u32 *id, void *entry,
> ~~~~~~^~~~~
> drivers/infiniband/core/restrack.c:26:36: error: incompatible type for argument 4 of '__xa_alloc'
> err = __xa_alloc(xa, id, U32_MAX, entry, GFP_KERNEL);
> ^~~~~
> In file included from include/linux/radix-tree.h:31,
> from include/linux/fs.h:15,
> from include/linux/seq_file.h:11,
> from arch/powerpc/include/asm/machdep.h:12,
> from arch/powerpc/include/asm/archrandom.h:7,
> from include/linux/random.h:166,
> from include/linux/net.h:22,
> from include/linux/skbuff.h:29,
> from include/linux/if_arp.h:26,
> from include/rdma/ib_addr.h:39,
> from include/rdma/rdma_cm.h:39,
> from drivers/infiniband/core/restrack.c:6:
> include/linux/xarray.h:525:3: note: expected 'struct xa_limit' but argument is of type 'void *'
> struct xa_limit, gfp_t);
> ^~~~~~~~~~~~~~~
> drivers/infiniband/core/restrack.c:29:28: warning: passing argument 3 of '__xa_alloc' makes pointer from integer without a cast [-Wint-conversion]
> err = __xa_alloc(xa, id, *next, entry, GFP_KERNEL);
> ^~~~~
> In file included from include/linux/radix-tree.h:31,
> from include/linux/fs.h:15,
> from include/linux/seq_file.h:11,
> from arch/powerpc/include/asm/machdep.h:12,
> from arch/powerpc/include/asm/archrandom.h:7,
> from include/linux/random.h:166,
> from include/linux/net.h:22,
> from include/linux/skbuff.h:29,
> from include/linux/if_arp.h:26,
> from include/rdma/ib_addr.h:39,
> from include/rdma/rdma_cm.h:39,
> from drivers/infiniband/core/restrack.c:6:
> include/linux/xarray.h:524:61: note: expected 'void *' but argument is of type 'u32' {aka 'unsigned int'}
> int __must_check __xa_alloc(struct xarray *, u32 *id, void *entry,
> ~~~~~~^~~~~
> drivers/infiniband/core/restrack.c:29:35: error: incompatible type for argument 4 of '__xa_alloc'
> err = __xa_alloc(xa, id, *next, entry, GFP_KERNEL);
> ^~~~~
> In file included from include/linux/radix-tree.h:31,
> from include/linux/fs.h:15,
> from include/linux/seq_file.h:11,
> from arch/powerpc/include/asm/machdep.h:12,
> from arch/powerpc/include/asm/archrandom.h:7,
> from include/linux/random.h:166,
> from include/linux/net.h:22,
> from include/linux/skbuff.h:29,
> from include/linux/if_arp.h:26,
> from include/rdma/ib_addr.h:39,
> from include/rdma/rdma_cm.h:39,
> from drivers/infiniband/core/restrack.c:6:
> include/linux/xarray.h:525:3: note: expected 'struct xa_limit' but argument is of type 'void *'
> struct xa_limit, gfp_t);
> ^~~~~~~~~~~~~~~
>
> Caused by commit
>
> fd47c2f99f04 ("RDMA/restrack: Convert internal DB from hash to XArray")
>
> from the rdma tree interacting with commit
>
> a3e4d3f97ec8 ("XArray: Redesign xa_alloc API")
>
> from the xarray tree.

Thanks Stephen, your change looks good.

Matthew, really? change of API in -rc7? And it after you pushed us to
base our -next on -rc5 after another API change? What should we do now?

Can you please ensure that you are sending your pull request to Linus,
after RDMA pull request will be successfully merged?

Thanks

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]