From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F007FC47091 for ; Thu, 27 May 2021 15:13:18 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 99F28613AF for ; Thu, 27 May 2021 15:13:18 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 99F28613AF Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 879058D0002; Thu, 27 May 2021 11:13:17 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 828B08D0001; Thu, 27 May 2021 11:13:17 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6A2278D0002; Thu, 27 May 2021 11:13:17 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0071.hostedemail.com [216.40.44.71]) by kanga.kvack.org (Postfix) with ESMTP id 35F6D8D0001 for ; Thu, 27 May 2021 11:13:17 -0400 (EDT) Received: from smtpin23.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id C8DE7181AEF0B for ; Thu, 27 May 2021 15:13:16 +0000 (UTC) X-FDA: 78187354392.23.853374A Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by imf18.hostedemail.com (Postfix) with ESMTP id C887B20009A3 for ; Thu, 27 May 2021 15:13:08 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 8B3AE613BA; Thu, 27 May 2021 15:13:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1622128395; bh=v4Y5CgfBMzp5T0CX+Epz9rKs73Qi5rHOJJTJ8WdHbRY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=TpgEYLQa6fww76pD+EXqPpG4tWw4RVeiOxm1IKDenxYeudi6q9QrsMPBj8vWQKGHF V7/AI4UcdJsv4V6xu6Hb60ZhUJrk6fiYerMLikOSbIqSBxzhi3g7f2UU+7FR9Gqt9V C3Ne5JKZLMQDRYjaw8c8GWIEPtvXypd/75O7SfurJguzt92R4evulYOP3jYxwMk/E8 HZLWeWb5dUdXXuTLVqxBjcStrY3S9YlmYJhjfnYYjB+C/OSSHD9RmNC0VDFJpsIM6w Pnmu3HDTg3AIUWWdpgI7DQ0ETS3eDvwT+YkmrmCWtuEnOu7h1QNKbK4vk/AJe8ndtI rvDl0qboF0+VQ== Date: Thu, 27 May 2021 18:13:03 +0300 From: Mike Rapoport To: Kefeng Wang Cc: Andrew Morton , linux-alpha@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-ia64@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-mips@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-riscv@lists.infradead.org, linux-s390@vger.kernel.org, sparclinux@vger.kernel.org, linux-mm@kvack.org, Catalin Marinas , Will Deacon , Geert Uytterhoeven , Thomas Bogendoerfer , "David S. Miller" , Ingo Molnar , Borislav Petkov , Palmer Dabbelt , Richard Henderson , Russell King Subject: Re: [PATCH] mm: generalize ZONE_[DMA|DMA32] Message-ID: References: <20210527143047.123611-1-wangkefeng.wang@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210527143047.123611-1-wangkefeng.wang@huawei.com> Authentication-Results: imf18.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=TpgEYLQa; spf=pass (imf18.hostedemail.com: domain of rppt@kernel.org designates 198.145.29.99 as permitted sender) smtp.mailfrom=rppt@kernel.org; dmarc=pass (policy=none) header.from=kernel.org X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: C887B20009A3 X-Stat-Signature: 88sh8ykqwt76xbjkjntyus991g6tmojw X-HE-Tag: 1622128388-858204 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Thu, May 27, 2021 at 10:30:47PM +0800, Kefeng Wang wrote: > ZONE_[DMA|DMA32] configs have duplicate definitions on platforms > that subscribe them. Instead, just make them generic options which > can be selected on applicable platforms. > > Also only x86/arm64 architectures could enable both ZONE_DMA and > ZONE_DMA32 if EXPERT, add ARCH_HAS_ZONE_DMA_SET to make dma zone > configurable and visible on the two architectures. > > Cc: Andrew Morton > Cc: Catalin Marinas > Cc: Will Deacon > Cc: Geert Uytterhoeven > Cc: Thomas Bogendoerfer > Cc: "David S. Miller" > Cc: Ingo Molnar > Cc: Borislav Petkov > Cc: Palmer Dabbelt > Cc: Richard Henderson > Cc: Russell King > Signed-off-by: Kefeng Wang Acked-by: Mike Rapoport > --- > arch/alpha/Kconfig | 5 +---- > arch/arm/Kconfig | 3 --- > arch/arm64/Kconfig | 9 +-------- > arch/ia64/Kconfig | 4 +--- > arch/m68k/Kconfig | 5 +---- > arch/microblaze/Kconfig | 4 +--- > arch/mips/Kconfig | 7 ------- > arch/powerpc/Kconfig | 4 ---- > arch/powerpc/platforms/Kconfig.cputype | 1 + > arch/riscv/Kconfig | 5 +---- > arch/s390/Kconfig | 4 +--- > arch/sparc/Kconfig | 5 +---- > arch/x86/Kconfig | 15 ++------------- > mm/Kconfig | 11 +++++++++++ > 14 files changed, 22 insertions(+), 60 deletions(-) > > diff --git a/arch/alpha/Kconfig b/arch/alpha/Kconfig > index 5998106faa60..6a69a14c4825 100644 > --- a/arch/alpha/Kconfig > +++ b/arch/alpha/Kconfig > @@ -40,6 +40,7 @@ config ALPHA > select MMU_GATHER_NO_RANGE > select SET_FS > select SPARSEMEM_EXTREME if SPARSEMEM > + select ZONE_DMA > help > The Alpha is a 64-bit general-purpose processor designed and > marketed by the Digital Equipment Corporation of blessed memory, > @@ -65,10 +66,6 @@ config GENERIC_CALIBRATE_DELAY > bool > default y > > -config ZONE_DMA > - bool > - default y > - > config GENERIC_ISA_DMA > bool > default y > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index 24804f11302d..000c3f80b58e 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -218,9 +218,6 @@ config GENERIC_CALIBRATE_DELAY > config ARCH_MAY_HAVE_PC_FDC > bool > > -config ZONE_DMA > - bool > - > config ARCH_SUPPORTS_UPROBES > def_bool y > > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > index 9f1d8566bbf9..42794474f37f 100644 > --- a/arch/arm64/Kconfig > +++ b/arch/arm64/Kconfig > @@ -42,6 +42,7 @@ config ARM64 > select ARCH_HAS_SYSCALL_WRAPPER > select ARCH_HAS_TEARDOWN_DMA_OPS if IOMMU_SUPPORT > select ARCH_HAS_TICK_BROADCAST if GENERIC_CLOCKEVENTS_BROADCAST > + select ARCH_HAS_ZONE_DMA_SET if EXPERT > select ARCH_HAVE_ELF_PROT > select ARCH_HAVE_NMI_SAFE_CMPXCHG > select ARCH_INLINE_READ_LOCK if !PREEMPTION > @@ -307,14 +308,6 @@ config GENERIC_CSUM > config GENERIC_CALIBRATE_DELAY > def_bool y > > -config ZONE_DMA > - bool "Support DMA zone" if EXPERT > - default y > - > -config ZONE_DMA32 > - bool "Support DMA32 zone" if EXPERT > - default y > - > config ARCH_MHP_MEMMAP_ON_MEMORY_ENABLE > def_bool y > > diff --git a/arch/ia64/Kconfig b/arch/ia64/Kconfig > index 279252e3e0f7..fd8503a0088a 100644 > --- a/arch/ia64/Kconfig > +++ b/arch/ia64/Kconfig > @@ -60,6 +60,7 @@ config IA64 > select NUMA if !FLATMEM > select PCI_MSI_ARCH_FALLBACKS if PCI_MSI > select SET_FS > + select ZONE_DMA32 > default y > help > The Itanium Processor Family is Intel's 64-bit successor to > @@ -72,9 +73,6 @@ config 64BIT > select ATA_NONSTANDARD if ATA > default y > > -config ZONE_DMA32 > - def_bool y > - > config MMU > bool > default y > diff --git a/arch/m68k/Kconfig b/arch/m68k/Kconfig > index 372e4e69c43a..05a729c6ad7f 100644 > --- a/arch/m68k/Kconfig > +++ b/arch/m68k/Kconfig > @@ -34,6 +34,7 @@ config M68K > select SET_FS > select UACCESS_MEMCPY if !MMU > select VIRT_TO_BUS > + select ZONE_DMA > > config CPU_BIG_ENDIAN > def_bool y > @@ -62,10 +63,6 @@ config TIME_LOW_RES > config NO_IOPORT_MAP > def_bool y > > -config ZONE_DMA > - bool > - default y > - > config HZ > int > default 1000 if CLEOPATRA > diff --git a/arch/microblaze/Kconfig b/arch/microblaze/Kconfig > index 0660f47012bc..14a67a42fcae 100644 > --- a/arch/microblaze/Kconfig > +++ b/arch/microblaze/Kconfig > @@ -43,6 +43,7 @@ config MICROBLAZE > select MMU_GATHER_NO_RANGE > select SPARSE_IRQ > select SET_FS > + select ZONE_DMA > > # Endianness selection > choice > @@ -60,9 +61,6 @@ config CPU_LITTLE_ENDIAN > > endchoice > > -config ZONE_DMA > - def_bool y > - > config ARCH_HAS_ILOG2_U32 > def_bool n > > diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig > index ed51970c08e7..430d5324f1af 100644 > --- a/arch/mips/Kconfig > +++ b/arch/mips/Kconfig > @@ -3277,13 +3277,6 @@ config I8253 > select CLKSRC_I8253 > select CLKEVT_I8253 > select MIPS_EXTERNAL_TIMER > - > -config ZONE_DMA > - bool > - > -config ZONE_DMA32 > - bool > - > endmenu > > config TRAD_SIGNALS > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig > index 088dd2afcfe4..0f78bb383a12 100644 > --- a/arch/powerpc/Kconfig > +++ b/arch/powerpc/Kconfig > @@ -403,10 +403,6 @@ config PPC_ADV_DEBUG_DAC_RANGE > config PPC_DAWR > bool > > -config ZONE_DMA > - bool > - default y if PPC_BOOK3E_64 > - > config PGTABLE_LEVELS > int > default 2 if !PPC64 > diff --git a/arch/powerpc/platforms/Kconfig.cputype b/arch/powerpc/platforms/Kconfig.cputype > index f998e655b570..7d271de8fcbd 100644 > --- a/arch/powerpc/platforms/Kconfig.cputype > +++ b/arch/powerpc/platforms/Kconfig.cputype > @@ -111,6 +111,7 @@ config PPC_BOOK3E_64 > select PPC_FPU # Make it a choice ? > select PPC_SMP_MUXED_IPI > select PPC_DOORBELL > + select ZONE_DMA > > endchoice > > diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig > index a8ad8eb76120..d29643dee126 100644 > --- a/arch/riscv/Kconfig > +++ b/arch/riscv/Kconfig > @@ -103,6 +103,7 @@ config RISCV > select SYSCTL_EXCEPTION_TRACE > select THREAD_INFO_IN_TASK > select UACCESS_MEMCPY if !MMU > + select ZONE_DMA32 if 64BIT > > config ARCH_MMAP_RND_BITS_MIN > default 18 if 64BIT > @@ -132,10 +133,6 @@ config MMU > Select if you want MMU-based virtualised addressing space > support by paged memory management. If unsure, say 'Y'. > > -config ZONE_DMA32 > - bool > - default y if 64BIT > - > config VA_BITS > int > default 32 if 32BIT > diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig > index b4c7c34069f8..daab9d56957a 100644 > --- a/arch/s390/Kconfig > +++ b/arch/s390/Kconfig > @@ -2,9 +2,6 @@ > config MMU > def_bool y > > -config ZONE_DMA > - def_bool y > - > config CPU_BIG_ENDIAN > def_bool y > > @@ -210,6 +207,7 @@ config S390 > select THREAD_INFO_IN_TASK > select TTY > select VIRT_CPU_ACCOUNTING > + select ZONE_DMA > # Note: keep the above list sorted alphabetically > > config SCHED_OMIT_FRAME_POINTER > diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig > index 164a5254c91c..39679664cc9a 100644 > --- a/arch/sparc/Kconfig > +++ b/arch/sparc/Kconfig > @@ -59,6 +59,7 @@ config SPARC32 > select CLZ_TAB > select HAVE_UID16 > select OLD_SIGACTION > + select ZONE_DMA > > config SPARC64 > def_bool 64BIT > @@ -141,10 +142,6 @@ config HIGHMEM > default y if SPARC32 > select KMAP_LOCAL > > -config ZONE_DMA > - bool > - default y if SPARC32 > - > config GENERIC_ISA_DMA > bool > default y if SPARC32 > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index 0045e1b44190..11cf8a0d6800 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -33,6 +33,7 @@ config X86_64 > select NEED_DMA_MAP_STATE > select SWIOTLB > select ARCH_HAS_ELFCORE_COMPAT > + select ZONE_DMA32 > > config FORCE_DYNAMIC_FTRACE > def_bool y > @@ -93,6 +94,7 @@ config X86 > select ARCH_HAS_SYSCALL_WRAPPER > select ARCH_HAS_UBSAN_SANITIZE_ALL > select ARCH_HAS_DEBUG_WX > + select ARCH_HAS_ZONE_DMA_SET if EXPERT > select ARCH_HAVE_NMI_SAFE_CMPXCHG > select ARCH_MIGHT_HAVE_ACPI_PDC if ACPI > select ARCH_MIGHT_HAVE_PC_PARPORT > @@ -343,9 +345,6 @@ config ARCH_SUSPEND_POSSIBLE > config ARCH_WANT_GENERAL_HUGETLB > def_bool y > > -config ZONE_DMA32 > - def_bool y if X86_64 > - > config AUDIT_ARCH > def_bool y if X86_64 > > @@ -393,16 +392,6 @@ config CC_HAS_SANE_STACKPROTECTOR > > menu "Processor type and features" > > -config ZONE_DMA > - bool "DMA memory allocation support" if EXPERT > - default y > - help > - DMA memory allocation support allows devices with less than 32-bit > - addressing to allocate within the first 16MB of address space. > - Disable if no such devices will be used. > - > - If unsure, say Y. > - > config SMP > bool "Symmetric multi-processing support" > help > diff --git a/mm/Kconfig b/mm/Kconfig > index 02d44e3420f5..68b0cbdc7968 100644 > --- a/mm/Kconfig > +++ b/mm/Kconfig > @@ -789,6 +789,17 @@ config ARCH_HAS_CACHE_LINE_SIZE > config ARCH_HAS_PTE_DEVMAP > bool > > +config ARCH_HAS_ZONE_DMA_SET > + bool > + > +config ZONE_DMA > + bool "Support DMA zone" if ARCH_HAS_ZONE_DMA_SET > + default y if ARM64 > + > +config ZONE_DMA32 > + bool "Support DMA32 zone" if ARCH_HAS_ZONE_DMA_SET > + default y if ARM64 > + > config ZONE_DEVICE > bool "Device memory (pmem, HMM, etc...) hotplug support" > depends on MEMORY_HOTPLUG > -- > 2.26.2 > > > _______________________________________________ > linux-riscv mailing list > linux-riscv@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-riscv -- Sincerely yours, Mike.