LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Robin Murphy <robin.murphy@arm.com>
To: Christoph Hellwig <hch@lst.de>, iommu@lists.linux-foundation.org
Cc: linux-arch@vger.kernel.org, linux-xtensa@linux-xtensa.org,
	Michal Simek <monstr@monstr.eu>,
	linux-snps-arc@lists.infradead.org, linux-c6x-dev@linux-c6x.org,
	linux-parisc@vger.kernel.org, linux-sh@vger.kernel.org,
	linux-hexagon@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-m68k@vger.kernel.org, openrisc@lists.librecores.org,
	Greentime Hu <green.hu@gmail.com>,
	linux-alpha@vger.kernel.org, sparclinux@vger.kernel.org,
	nios2-dev@lists.rocketboards.org,
	Vincent Chen <deanbo422@gmail.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 4/3] dma-debug: remove CONFIG_HAVE_DMA_API_DEBUG
Date: Tue, 8 May 2018 10:54:42 +0100	[thread overview]
Message-ID: <cfc6ac1e-56b7-0211-afa3-86865a01667d@arm.com> (raw)
In-Reply-To: <20180427155342.GA9232@lst.de>

On 27/04/18 16:53, Christoph Hellwig wrote:
> There is no arch specific code required for dma-debug, so there is no
> need to opt into the support either.

Makes sense, and a purely negative diffstat is always pleasing :)

Reviewed-by: Robin Murphy <robin.murphy@arm.com>

> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
>   .../io/dma-api-debug/arch-support.txt         | 31 -------------------
>   arch/Kconfig                                  |  3 --
>   arch/arm/Kconfig                              |  1 -
>   arch/arm64/Kconfig                            |  1 -
>   arch/c6x/Kconfig                              |  1 -
>   arch/ia64/Kconfig                             |  1 -
>   arch/microblaze/Kconfig                       |  1 -
>   arch/mips/Kconfig                             |  1 -
>   arch/powerpc/Kconfig                          |  1 -
>   arch/riscv/Kconfig                            |  1 -
>   arch/s390/Kconfig                             |  1 -
>   arch/sh/Kconfig                               |  1 -
>   arch/sparc/Kconfig                            |  1 -
>   arch/x86/Kconfig                              |  1 -
>   arch/xtensa/Kconfig                           |  1 -
>   lib/Kconfig.debug                             |  1 -
>   16 files changed, 48 deletions(-)
>   delete mode 100644 Documentation/features/io/dma-api-debug/arch-support.txt
> 
> diff --git a/Documentation/features/io/dma-api-debug/arch-support.txt b/Documentation/features/io/dma-api-debug/arch-support.txt
> deleted file mode 100644
> index e438ed675623..000000000000
> --- a/Documentation/features/io/dma-api-debug/arch-support.txt
> +++ /dev/null
> @@ -1,31 +0,0 @@
> -#
> -# Feature name:          dma-api-debug
> -#         Kconfig:       HAVE_DMA_API_DEBUG
> -#         description:   arch supports DMA debug facilities
> -#
> -    -----------------------
> -    |         arch |status|
> -    -----------------------
> -    |       alpha: | TODO |
> -    |         arc: | TODO |
> -    |         arm: |  ok  |
> -    |       arm64: |  ok  |
> -    |         c6x: |  ok  |
> -    |       h8300: | TODO |
> -    |     hexagon: | TODO |
> -    |        ia64: |  ok  |
> -    |        m68k: | TODO |
> -    |  microblaze: |  ok  |
> -    |        mips: |  ok  |
> -    |       nios2: | TODO |
> -    |    openrisc: | TODO |
> -    |      parisc: | TODO |
> -    |     powerpc: |  ok  |
> -    |        s390: |  ok  |
> -    |          sh: |  ok  |
> -    |       sparc: |  ok  |
> -    |          um: | TODO |
> -    |   unicore32: | TODO |
> -    |         x86: |  ok  |
> -    |      xtensa: |  ok  |
> -    -----------------------
> diff --git a/arch/Kconfig b/arch/Kconfig
> index 8e0d665c8d53..f07a1a99e5db 100644
> --- a/arch/Kconfig
> +++ b/arch/Kconfig
> @@ -278,9 +278,6 @@ config HAVE_CLK
>   	  The <linux/clk.h> calls support software clock gating and
>   	  thus are a key power management tool on many systems.
>   
> -config HAVE_DMA_API_DEBUG
> -	bool
> -
>   config HAVE_HW_BREAKPOINT
>   	bool
>   	depends on PERF_EVENTS
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 676977bdfe33..c43f5bb55ac8 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -60,7 +60,6 @@ config ARM
>   	select HAVE_CONTEXT_TRACKING
>   	select HAVE_C_RECORDMCOUNT
>   	select HAVE_DEBUG_KMEMLEAK
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_DMA_CONTIGUOUS if MMU
>   	select HAVE_DYNAMIC_FTRACE if (!XIP_KERNEL) && !CPU_ENDIAN_BE32 && MMU
>   	select HAVE_DYNAMIC_FTRACE_WITH_REGS if HAVE_DYNAMIC_FTRACE
> diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig
> index db51b6445744..b25ed7834f6c 100644
> --- a/arch/arm64/Kconfig
> +++ b/arch/arm64/Kconfig
> @@ -105,7 +105,6 @@ config ARM64
>   	select HAVE_CONTEXT_TRACKING
>   	select HAVE_DEBUG_BUGVERBOSE
>   	select HAVE_DEBUG_KMEMLEAK
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_DMA_CONTIGUOUS
>   	select HAVE_DYNAMIC_FTRACE
>   	select HAVE_EFFICIENT_UNALIGNED_ACCESS
> diff --git a/arch/c6x/Kconfig b/arch/c6x/Kconfig
> index c6b4dd1418b4..8c088b96e372 100644
> --- a/arch/c6x/Kconfig
> +++ b/arch/c6x/Kconfig
> @@ -10,7 +10,6 @@ config C6X
>   	select GENERIC_ATOMIC64
>   	select GENERIC_IRQ_SHOW
>   	select HAVE_ARCH_TRACEHOOK
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_MEMBLOCK
>   	select SPARSE_IRQ
>   	select IRQ_DOMAIN
> diff --git a/arch/ia64/Kconfig b/arch/ia64/Kconfig
> index 9485b5490eca..2067289fad4a 100644
> --- a/arch/ia64/Kconfig
> +++ b/arch/ia64/Kconfig
> @@ -29,7 +29,6 @@ config IA64
>   	select HAVE_FUNCTION_TRACER
>   	select TTY
>   	select HAVE_ARCH_TRACEHOOK
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_MEMBLOCK
>   	select HAVE_MEMBLOCK_NODE_MAP
>   	select HAVE_VIRT_CPU_ACCOUNTING
> diff --git a/arch/microblaze/Kconfig b/arch/microblaze/Kconfig
> index 3817a3e2146c..d14782100088 100644
> --- a/arch/microblaze/Kconfig
> +++ b/arch/microblaze/Kconfig
> @@ -19,7 +19,6 @@ config MICROBLAZE
>   	select HAVE_ARCH_HASH
>   	select HAVE_ARCH_KGDB
>   	select HAVE_DEBUG_KMEMLEAK
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_DYNAMIC_FTRACE
>   	select HAVE_FTRACE_MCOUNT_RECORD
>   	select HAVE_FUNCTION_GRAPH_TRACER
> diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig
> index 0f619b8c0e9e..2dcdc13cd65d 100644
> --- a/arch/mips/Kconfig
> +++ b/arch/mips/Kconfig
> @@ -42,7 +42,6 @@ config MIPS
>   	select HAVE_C_RECORDMCOUNT
>   	select HAVE_DEBUG_KMEMLEAK
>   	select HAVE_DEBUG_STACKOVERFLOW
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_DMA_CONTIGUOUS
>   	select HAVE_DYNAMIC_FTRACE
>   	select HAVE_EXIT_THREAD
> diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> index 1887f8f86a77..268fd46fc3c7 100644
> --- a/arch/powerpc/Kconfig
> +++ b/arch/powerpc/Kconfig
> @@ -181,7 +181,6 @@ config PPC
>   	select HAVE_CONTEXT_TRACKING		if PPC64
>   	select HAVE_DEBUG_KMEMLEAK
>   	select HAVE_DEBUG_STACKOVERFLOW
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_DYNAMIC_FTRACE
>   	select HAVE_DYNAMIC_FTRACE_WITH_REGS	if MPROFILE_KERNEL
>   	select HAVE_EBPF_JIT			if PPC64
> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> index 17212ba54ee3..95e2da34aaad 100644
> --- a/arch/riscv/Kconfig
> +++ b/arch/riscv/Kconfig
> @@ -23,7 +23,6 @@ config RISCV
>   	select GENERIC_ATOMIC64 if !64BIT || !RISCV_ISA_A
>   	select HAVE_MEMBLOCK
>   	select HAVE_MEMBLOCK_NODE_MAP
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_DMA_CONTIGUOUS
>   	select HAVE_GENERIC_DMA_COHERENT
>   	select IRQ_DOMAIN
> diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig
> index b794a2ab6d15..6a64287ec1da 100644
> --- a/arch/s390/Kconfig
> +++ b/arch/s390/Kconfig
> @@ -130,7 +130,6 @@ config S390
>   	select HAVE_CMPXCHG_LOCAL
>   	select HAVE_COPY_THREAD_TLS
>   	select HAVE_DEBUG_KMEMLEAK
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_DMA_CONTIGUOUS
>   	select DMA_DIRECT_OPS
>   	select HAVE_DYNAMIC_FTRACE
> diff --git a/arch/sh/Kconfig b/arch/sh/Kconfig
> index 9417f70e008e..7d521926041e 100644
> --- a/arch/sh/Kconfig
> +++ b/arch/sh/Kconfig
> @@ -13,7 +13,6 @@ config SUPERH
>   	select HAVE_OPROFILE
>   	select HAVE_GENERIC_DMA_COHERENT
>   	select HAVE_ARCH_TRACEHOOK
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_PERF_EVENTS
>   	select HAVE_DEBUG_BUGVERBOSE
>   	select ARCH_HAVE_CUSTOM_GPIO_H
> diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig
> index c1cfc17eb504..435dbc033afe 100644
> --- a/arch/sparc/Kconfig
> +++ b/arch/sparc/Kconfig
> @@ -25,7 +25,6 @@ config SPARC
>   	select RTC_CLASS
>   	select RTC_DRV_M48T59
>   	select RTC_SYSTOHC
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_ARCH_JUMP_LABEL if SPARC64
>   	select GENERIC_IRQ_SHOW
>   	select ARCH_WANT_IPC_PARSE_VERSION
> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
> index aad35c568681..3eeca48a146b 100644
> --- a/arch/x86/Kconfig
> +++ b/arch/x86/Kconfig
> @@ -135,7 +135,6 @@ config X86
>   	select HAVE_C_RECORDMCOUNT
>   	select HAVE_DEBUG_KMEMLEAK
>   	select HAVE_DEBUG_STACKOVERFLOW
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_DMA_CONTIGUOUS
>   	select HAVE_DYNAMIC_FTRACE
>   	select HAVE_DYNAMIC_FTRACE_WITH_REGS
> diff --git a/arch/xtensa/Kconfig b/arch/xtensa/Kconfig
> index c921e8bccdc8..17df332269b2 100644
> --- a/arch/xtensa/Kconfig
> +++ b/arch/xtensa/Kconfig
> @@ -19,7 +19,6 @@ config XTENSA
>   	select HAVE_ARCH_KASAN if MMU
>   	select HAVE_CC_STACKPROTECTOR
>   	select HAVE_DEBUG_KMEMLEAK
> -	select HAVE_DMA_API_DEBUG
>   	select HAVE_DMA_CONTIGUOUS
>   	select HAVE_EXIT_THREAD
>   	select HAVE_FUNCTION_TRACER
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index 685ed2dd4384..d5175eb7b917 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -1634,7 +1634,6 @@ config PROVIDE_OHCI1394_DMA_INIT
>   
>   config DMA_API_DEBUG
>   	bool "Enable debugging of DMA-API usage"
> -	depends on HAVE_DMA_API_DEBUG
>   	select NEED_DMA_MAP_STATE
>   	help
>   	  Enable this option to debug the use of the DMA API by device drivers.
> 

  reply	other threads:[~2018-05-08  9:54 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-24 14:02 dma-debug cleanups, including removing the arch hook Christoph Hellwig
2018-04-24 14:02 ` [PATCH 1/3] dma-debug: move initialization to common code Christoph Hellwig
2018-05-08  9:46   ` Robin Murphy
2018-05-08 10:22   ` Marek Szyprowski
2018-04-24 14:02 ` [PATCH 2/3] dma-debug: simplify counting of preallocated requests Christoph Hellwig
2018-05-08  9:50   ` Robin Murphy
2018-04-24 14:02 ` [PATCH 3/3] dma-debug: unexport dma_debug_resize_entries and debug_dma_dump_mappings Christoph Hellwig
2018-05-08 10:05   ` Robin Murphy
2018-05-08 10:45     ` Christoph Hellwig
2018-04-27 15:53 ` [PATCH 4/3] dma-debug: remove CONFIG_HAVE_DMA_API_DEBUG Christoph Hellwig
2018-05-08  9:54   ` Robin Murphy [this message]
2018-05-07 10:29 ` dma-debug cleanups, including removing the arch hook Christoph Hellwig

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cfc6ac1e-56b7-0211-afa3-86865a01667d@arm.com \
    --to=robin.murphy@arm.com \
    --cc=deanbo422@gmail.com \
    --cc=green.hu@gmail.com \
    --cc=hch@lst.de \
    --cc=iommu@lists.linux-foundation.org \
    --cc=linux-alpha@vger.kernel.org \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-c6x-dev@linux-c6x.org \
    --cc=linux-hexagon@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-m68k@vger.kernel.org \
    --cc=linux-parisc@vger.kernel.org \
    --cc=linux-sh@vger.kernel.org \
    --cc=linux-snps-arc@lists.infradead.org \
    --cc=linux-xtensa@linux-xtensa.org \
    --cc=monstr@monstr.eu \
    --cc=nios2-dev@lists.rocketboards.org \
    --cc=openrisc@lists.librecores.org \
    --cc=sparclinux@vger.kernel.org \
    --subject='Re: [PATCH 4/3] dma-debug: remove CONFIG_HAVE_DMA_API_DEBUG' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).