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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Tue, 9 Jul 2019 10:16:43 +1000
From:   Stephen Rothwell <sfr@...b.auug.org.au>
To:     Christoph Hellwig <hch@....de>
Cc:     Linux Next Mailing List <linux-next@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Dan Williams <dan.j.williams@...el.com>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Fredrik Noring <noring@...rew.org>
Subject: Re: linux-next: manual merge of the dma-mapping tree with Linus'
 tree

Hi all,

On Mon, 1 Jul 2019 08:42:13 +1000 Stephen Rothwell <sfr@...b.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.

-- 
Cheers,
Stephen Rothwell

Content of type "application/pgp-signature" skipped