Re: linux-next: manual merge of the dma-mapping tree with Linus' tree

Stephen Rothwell
2019-07-09T00:16:48Z

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

Hi all,

On Mon, 1 Jul 2019 08:42:13 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> Today's linux-next merge of the dma-mapping tree got a conflict in:
>
> include/linux/genalloc.h
>
> between commit:
>
> 795ee30648c7 ("lib/genalloc: introduce chunk owners")
>
> from Linus' tree and commit:
>
> cf394fc5f715 ("lib/genalloc.c: Add algorithm, align and zeroed family of DMA allocators")
>
> from the dma-mapping tree.
>
> I fixed it up (see below) and can carry the fix as necessary. This
> is now fixed as far as linux-next is concerned, but any non trivial
> conflicts should be mentioned to your upstream maintainer when your tree
> is submitted for merging. You may also want to consider cooperating
> with the maintainer of the conflicting tree to minimise any particularly
> complex conflicts.
>
> --
> Cheers,
> Stephen Rothwell
>
> diff --cc include/linux/genalloc.h
> index 205f62b8d291,ed641337df87..000000000000
> --- a/include/linux/genalloc.h
> +++ b/include/linux/genalloc.h
> @@@ -122,47 -116,21 +122,56 @@@ static inline int gen_pool_add(struct g
> return gen_pool_add_virt(pool, addr, -1, size, nid);
> }
> extern void gen_pool_destroy(struct gen_pool *);
> -extern unsigned long gen_pool_alloc(struct gen_pool *, size_t);
> -extern unsigned long gen_pool_alloc_algo(struct gen_pool *, size_t,
> - genpool_algo_t algo, void *data);
> +unsigned long gen_pool_alloc_algo_owner(struct gen_pool *pool, size_t size,
> + genpool_algo_t algo, void *data, void **owner);
> +
> +static inline unsigned long gen_pool_alloc_owner(struct gen_pool *pool,
> + size_t size, void **owner)
> +{
> + return gen_pool_alloc_algo_owner(pool, size, pool->algo, pool->data,
> + owner);
> +}
> +
> +static inline unsigned long gen_pool_alloc_algo(struct gen_pool *pool,
> + size_t size, genpool_algo_t algo, void *data)
> +{
> + return gen_pool_alloc_algo_owner(pool, size, algo, data, NULL);
> +}
> +
> +/**
> + * gen_pool_alloc - allocate special memory from the pool
> + * @pool: pool to allocate from
> + * @size: number of bytes to allocate from the pool
> + *
> + * Allocate the requested number of bytes from the specified pool.
> + * Uses the pool allocation function (with first-fit algorithm by default).
> + * Can not be used in NMI handler on architectures without
> + * NMI-safe cmpxchg implementation.
> + */
> +static inline unsigned long gen_pool_alloc(struct gen_pool *pool, size_t size)
> +{
> + return gen_pool_alloc_algo(pool, size, pool->algo, pool->data);
> +}
> +
> extern void *gen_pool_dma_alloc(struct gen_pool *pool, size_t size,
> dma_addr_t *dma);
> + extern void *gen_pool_dma_alloc_algo(struct gen_pool *pool, size_t size,
> + dma_addr_t *dma, genpool_algo_t algo, void *data);
> + extern void *gen_pool_dma_alloc_align(struct gen_pool *pool, size_t size,
> + dma_addr_t *dma, int align);
> + extern void *gen_pool_dma_zalloc(struct gen_pool *pool, size_t size, dma_addr_t *dma);
> + extern void *gen_pool_dma_zalloc_algo(struct gen_pool *pool, size_t size,
> + dma_addr_t *dma, genpool_algo_t algo, void *data);
> + extern void *gen_pool_dma_zalloc_align(struct gen_pool *pool, size_t size,
> + dma_addr_t *dma, int align);
> -extern void gen_pool_free(struct gen_pool *, unsigned long, size_t);
> +extern void gen_pool_free_owner(struct gen_pool *pool, unsigned long addr,
> + size_t size, void **owner);
> +static inline void gen_pool_free(struct gen_pool *pool, unsigned long addr,
> + size_t size)
> +{
> + gen_pool_free_owner(pool, addr, size, NULL);
> +}
> +
> extern void gen_pool_for_each_chunk(struct gen_pool *,
> void (*)(struct gen_pool *, struct gen_pool_chunk *, void *), void *);
> extern size_t gen_pool_avail(struct gen_pool *);

I am still getting this conflict (the commit ids may have changed).
Just a reminder in case you think Linus may need to know.