linux-kernel - Re: [PATCH v2 06/26] lib/dma-noop: Only build dma_noop

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] [thread-next>] [day] [month] [year] [list]

Message-Id: <33170c0b-abc4-005d-d992-cc86a71f07a2@de.ibm.com>
Date:   Fri, 13 Jan 2017 09:38:29 +0100
From:   Christian Borntraeger <borntraeger@...ibm.com>
To:     Bart Van Assche <bart.vanassche@...disk.com>,
        Doug Ledford <dledford@...hat.com>
Cc:     linux-rdma@...r.kernel.org, linux-kernel@...r.kernel.org,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Joerg Roedel <jroedel@...e.de>,
        Andy Lutomirski <luto@...nel.org>,
        "Michael S . Tsirkin" <mst@...hat.com>,
        Christoph Hellwig <hch@....de>,
        linux-s390 <linux-s390@...r.kernel.org>
Subject: Re: [PATCH v2 06/26] lib/dma-noop: Only build dma_noop_ops for m32r
 and s390

On 01/12/2017 08:06 PM, Bart Van Assche wrote:
> This was suggested by Christoph Hellwig.
> 
> Signed-off-by: Bart Van Assche <bart.vanassche@...disk.com>
> Cc: Christian Borntraeger <borntraeger@...ibm.com>
> Cc: Joerg Roedel <jroedel@...e.de>
> Cc: Andy Lutomirski <luto@...nel.org>
> Cc: Michael S. Tsirkin <mst@...hat.com>
> Cc: Christoph Hellwig <hch@....de>

Should be ok for s390  (as we do have ARCH_DMA_ADDR_T_64BIT).


> ---
>  arch/m32r/Kconfig | 1 +
>  arch/s390/Kconfig | 1 +
>  lib/Kconfig       | 5 +++++
>  lib/Makefile      | 2 +-
>  4 files changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/m32r/Kconfig b/arch/m32r/Kconfig
> index d227a6988d6b..95474460b367 100644
> --- a/arch/m32r/Kconfig
> +++ b/arch/m32r/Kconfig
> @@ -18,6 +18,7 @@ config M32R
>  	select MODULES_USE_ELF_RELA
>  	select HAVE_DEBUG_STACKOVERFLOW
>  	select CPU_NO_EFFICIENT_FFS
> +	select DMA_NOOP_OPS
> 
>  config SBUS
>  	bool
> diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig
> index c6722112527d..38ad42358c41 100644
> --- a/arch/s390/Kconfig
> +++ b/arch/s390/Kconfig
> @@ -137,6 +137,7 @@ config S390
>  	select HAVE_DEBUG_KMEMLEAK
>  	select HAVE_DMA_API_DEBUG
>  	select HAVE_DMA_CONTIGUOUS
> +	select DMA_NOOP_OPS
>  	select HAVE_DYNAMIC_FTRACE
>  	select HAVE_DYNAMIC_FTRACE_WITH_REGS
>  	select HAVE_EFFICIENT_UNALIGNED_ACCESS
> diff --git a/lib/Kconfig b/lib/Kconfig
> index 260a80e313b9..b6baf0609e8b 100644
> --- a/lib/Kconfig
> +++ b/lib/Kconfig
> @@ -395,6 +395,11 @@ config HAS_DMA
>  	depends on !NO_DMA
>  	default y
> 
> +config DMA_NOOP_OPS
> +	bool
> +	depends on HAS_DMA && (!64BIT || ARCH_DMA_ADDR_T_64BIT)
> +	default n
> +
>  config CHECK_SIGNATURE
>  	bool
> 
> diff --git a/lib/Makefile b/lib/Makefile
> index bc4073a8cd08..ada06d5652e8 100644
> --- a/lib/Makefile
> +++ b/lib/Makefile
> @@ -26,7 +26,7 @@ lib-y := ctype.o string.o vsprintf.o cmdline.o \
> 
>  lib-$(CONFIG_MMU) += ioremap.o
>  lib-$(CONFIG_SMP) += cpumask.o
> -lib-$(CONFIG_HAS_DMA) += dma-noop.o
> +lib-$(CONFIG_DMA_NOOP_OPS) += dma-noop.o
> 
>  lib-y	+= kobject.o klist.o
>  obj-y	+= lockref.o
>