From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Ellerman Date: Tue, 01 Jun 2021 02:56:10 +0000 Subject: Re: [PATCH v2] mm: generalize ZONE_[DMA|DMA32] Message-Id: <87k0negunp.fsf@mpe.ellerman.id.au> List-Id: References: <20210527143047.123611-1-wangkefeng.wang@huawei.com> <20210528074557.17768-1-wangkefeng.wang@huawei.com> In-Reply-To: <20210528074557.17768-1-wangkefeng.wang@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Kefeng Wang , Andrew Morton Cc: 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, Kefeng Wang , Catalin Marinas , Will Deacon , Geert Uytterhoeven , Thomas Bogendoerfer , "David S. Miller" , Ingo Molnar , Borislav Petkov , Palmer Dabbelt , Richard Henderson , Russell King , Mike Rapoport Kefeng Wang writes: > 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 > Acked-by: Catalin Marinas # for arm64 > Acked-by: Geert Uytterhoeven # for m68k > Acked-by: Mike Rapoport > Signed-off-by: Kefeng Wang > --- > v2: > -i386 can't enable ZONE_DMA32, fix it. > -make ZONE_DMA default y on X86 as before. > -collect ACKs > > 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 + Acked-by: Michael Ellerman (powerpc) cheers