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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8C3AAC61DA4 for ; Mon, 13 Mar 2023 17:55:45 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id C4F7D6B0072; Mon, 13 Mar 2023 13:55:44 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id C00916B0074; Mon, 13 Mar 2023 13:55:44 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id AC73D6B0075; Mon, 13 Mar 2023 13:55:44 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 9B2F36B0072 for ; Mon, 13 Mar 2023 13:55:44 -0400 (EDT) Received: from smtpin02.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 5D527A0D8D for ; Mon, 13 Mar 2023 17:55:44 +0000 (UTC) X-FDA: 80564627808.02.CADE0B7 Received: from elvis.franken.de (elvis.franken.de [193.175.24.41]) by imf05.hostedemail.com (Postfix) with ESMTP id 07734100007 for ; Mon, 13 Mar 2023 17:55:41 +0000 (UTC) Authentication-Results: imf05.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf05.hostedemail.com: domain of tsbogend@alpha.franken.de designates 193.175.24.41 as permitted sender) smtp.mailfrom=tsbogend@alpha.franken.de ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1678730142; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=3fNT6tPLJ7yhud5AtKryTATETrMMpctjTC8PUWOgkYI=; b=tsvJ12CFBCYecfSfzGH29PhBa2wQYyvei0qvoFE9kFFYR0VVF0Wn74qitYZwW6QsfpMLK6 WD9vhTKJmcXYD2MhoJvKqPdnFOm56P3Z+Kd9Gt4wBSjokajoiY96/ffhD3NfaXk/vpjBYk pYTbkZcclzsvQyL78UIIeaiz0KRcQVQ= ARC-Authentication-Results: i=1; imf05.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf05.hostedemail.com: domain of tsbogend@alpha.franken.de designates 193.175.24.41 as permitted sender) smtp.mailfrom=tsbogend@alpha.franken.de ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1678730142; a=rsa-sha256; cv=none; b=L4p/ljNVi2wS+BD9BqdOQ2UcDt6I04o4KLLEOmeVDXwtPqhk5JyvZfUz9cOSUJyBcVwias 6rcgykqZiZVveXS0ytjNXgAXlbZ3uluwsNagZKXEbSVBWYFxi3wbgiGYAyEG1mSo6jYLo5 FYlfx1RoWPzCMjPZK9+1YdRp4Vog/zo= Received: from uucp (helo=alpha) by elvis.franken.de with local-bsmtp (Exim 3.36 #1) id 1pbmOT-0002OB-00; Mon, 13 Mar 2023 18:55:33 +0100 Received: by alpha.franken.de (Postfix, from userid 1000) id ABE78C1252; Mon, 13 Mar 2023 18:55:21 +0100 (CET) Date: Mon, 13 Mar 2023 18:55:21 +0100 From: Thomas Bogendoerfer To: Baoquan He Cc: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, linux-mm@kvack.org, arnd@arndb.de, mpe@ellerman.id.au, geert@linux-m68k.org, mcgrof@kernel.org, hch@infradead.org, Helge Deller , Serge Semin , Florian Fainelli , Huacai Chen , Jiaxun Yang , linux-mips@vger.kernel.org Subject: Re: [PATCH v4 2/4] mips: add including Message-ID: <20230313175521.GA14404@alpha.franken.de> References: <20230308130710.368085-1-bhe@redhat.com> <20230308130710.368085-3-bhe@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230308130710.368085-3-bhe@redhat.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-Rspam-User: X-Rspamd-Server: rspam02 X-Rspamd-Queue-Id: 07734100007 X-Stat-Signature: 8d46apr9fi8ho7o6u6jcipwzgej4rybo X-HE-Tag: 1678730141-12078 X-HE-Meta: U2FsdGVkX19zLU8ZVshUS0a8eTvQoUpg/eCp3LX/QsvaLzzneSReVf0fZkDloVddMjt8QC4ugBkJJXRW/j/QgJIcRv3OMuCJJkp2jh9l4P16DNkKz1KHOW3eMfRCphYsgzVy36rMFK2q3Qnaf+uIn4WswimDBe+At/+l7n4BGd5P/75xxBJoR+giRUUujlRPbFOuy1Y9nIX+iANCzjryMSrxdeZB+hDNwBFr0LcyoFhlHW/48dsn+I3gwtvDyx+8C6s3B4xD2z7un8DkmChbyPo9sWegFNkFNddJRFGQrf875907FC+yarCW1DsbC8Z1oXA1iAHown0UdPYqGjygsXHymbx8ZuDLM2VVFj+zl2xz6XA8sOpjMSJK58gkCTY0PD783/5Kq7HTrEzxf3vAokW4I8wmYeezf9pq1o2NMa1kbebM++Ug7sTnJCT0P195ZImpqoZsVOif8453a9ue3uksIPfioW8h01StkVLtgJg2cOLMYBjbAb76a1tyJA2kVBDR7RPbhCFmrW3fmlpaBU78c37apTUtOQ/p3Lf3hAvHoqp/+uJnp56TXslRTTjZjZLZry19Hr47tt8AAcibMZNyYrcmZ0OugoigSw4hCOhr3LTfi3TDqyK9QY0sCXB5llZCOGh7gGXCLRznIQcL/dFiZjhx3X2vyTtMV01a7LcLg0CWuZiFIJGmUTWzF+JhOm/hiu5YkXt1YE0fKsiHUYgjeaUH+AvIQawvgAXoVCF2aMVklWeURVJSmuQ11ipqopPKHIC0/xPI8kNY1OoBHh1vVFUwEeQVEcTu58Awfz18lCWDgl9zTRVal1xQscf1dCiAy9++/UFszm182PsMqOcr8ogiol/+xQ9Vr0ClJVnswkugE/1XGzGxYpErvHOZFLjQH+zRWIk31xmULX3LM1pJCsPvQANKVy4OALOZ7NwqFi4BBRPm/FztI9foIReP04bId/aRN7j0R45OPzH H+02FI3F uAggXw1WnSx1cZbHrNAEsn72Z2xuHQgabjWa/E+1H1go+uC86GAuR2TIwjn9m7dw9hjEoLtLqeUzIHQ4hTaigyxJRnC/eqXEYz6I7ns0nyInmah3voRzqwFcxZOEX1IJlPKmwfQ+NVuMkXVUGudxnKdZm2u5X/V5+jK/tZgnpCl/sfGel7v4AL6oehNQ7cXKqSimnOFg4IE7IWcX+0WM/jWlWPVhgOQ2YtI/VEXaWYxVgeR+II1u6uwBbKivOr+eTx7F+p8LMO8ObsNSjObDU2xmGe1Mcqlg5YKFZLfDkuQc0OeCUEE2vWvM95WudxL0NimiA1ttF7wtqi+S/Hzb73Ho59A== 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 Wed, Mar 08, 2023 at 09:07:08PM +0800, Baoquan He wrote: > With the adding, some default ioremap_xx methods defined in > asm-generic/io.h can be used. E.g the default ioremap_uc() returning > NULL. > > Signed-off-by: Baoquan He > Reviewed-by: Arnd Bergmann > Cc: Thomas Bogendoerfer > Cc: Geert Uytterhoeven > Cc: Helge Deller > Cc: Serge Semin > Cc: Florian Fainelli > Cc: Huacai Chen > Cc: Jiaxun Yang > Cc: linux-mips@vger.kernel.org > --- > arch/mips/include/asm/io.h | 78 ++++++++++++++++++++++++++++++++++---- > 1 file changed, 70 insertions(+), 8 deletions(-) > > diff --git a/arch/mips/include/asm/io.h b/arch/mips/include/asm/io.h > index cec8347f0b85..6756baadba6c 100644 > --- a/arch/mips/include/asm/io.h > +++ b/arch/mips/include/asm/io.h > @@ -126,6 +126,7 @@ static inline phys_addr_t virt_to_phys(const volatile void *x) > * almost all conceivable cases a device driver should not be using > * this function > */ > +#define phys_to_virt phys_to_virt > static inline void * phys_to_virt(unsigned long address) > { > return __va(address); > @@ -359,6 +360,27 @@ __BUILD_MEMORY_PFX(__raw_, q, u64, 0) > __BUILD_MEMORY_PFX(__mem_, q, u64, 0) > #endif > > +#define readb readb > +#define readw readw > +#define readl readl > +#define writeb writeb > +#define writew writew > +#define writel writel > + > +#ifdef CONFIG_64BIT > +#define readq readq > +#define writeq writeq > +#define __raw_readq __raw_readq > +#define __raw_writeq __raw_writeq > +#endif > + > +#define __raw_readb __raw_readb > +#define __raw_readw __raw_readw > +#define __raw_readl __raw_readl > +#define __raw_writeb __raw_writeb > +#define __raw_writew __raw_writew > +#define __raw_writel __raw_writel > + > #define __BUILD_IOPORT_PFX(bus, bwlq, type) \ > __BUILD_IOPORT_SINGLE(bus, bwlq, type, 1, 0,) \ > __BUILD_IOPORT_SINGLE(bus, bwlq, type, 1, 0, _p) > @@ -374,6 +396,27 @@ BUILDIO_IOPORT(l, u32) > BUILDIO_IOPORT(q, u64) > #endif > > +#define inb inb > +#define inw inw > +#define inl inl > +#define inb_p inb_p > +#define inw_p inw_p > +#define inl_p inl_p > + > +#define outb outb > +#define outw outw > +#define outl outl > +#define outb_p outb_p > +#define outw_p outw_p > +#define outl_p outl_p > + > +#ifdef CONFIG_64BIT > +#define inq inq > +#define outq outq > +#define inq_p inq_p > +#define outq_p outq_p > +#endif > + > #define __BUILDIO(bwlq, type) \ > \ > __BUILD_MEMORY_SINGLE(____raw_, bwlq, type, 1, 0, 0) > @@ -412,14 +455,6 @@ __BUILDIO(q, u64) > #define writeq_be(val, addr) \ > __raw_writeq(cpu_to_be64((val)), (__force unsigned *)(addr)) > > -/* > - * Some code tests for these symbols > - */ > -#ifdef CONFIG_64BIT > -#define readq readq > -#define writeq writeq > -#endif > - > #define __BUILD_MEMORY_STRING(bwlq, type) \ > \ > static inline void writes##bwlq(volatile void __iomem *mem, \ > @@ -480,14 +515,39 @@ BUILDSTRING(l, u32) > BUILDSTRING(q, u64) > #endif > > +#define insb insb > +#define insw insw > +#define insl insl > +#define outsb outsb > +#define outsw outsw > +#define outsl outsl > + > +#define readsb readsb > +#define readsw readsw > +#define readsl readsl > +#define writesb writesb > +#define writesw writesw > +#define writesl writesl > + > +#ifdef CONFIG_64BIT > +#define insq insq > +#define readsq readsq > +#define readsq readsq > +#define writesq writesq > +#endif > + > + > +#define memset_io memset_io > static inline void memset_io(volatile void __iomem *addr, unsigned char val, int count) > { > memset((void __force *) addr, val, count); > } > +#define memcpy_fromio memcpy_fromio > static inline void memcpy_fromio(void *dst, const volatile void __iomem *src, int count) > { > memcpy(dst, (void __force *) src, count); > } > +#define memcpy_toio memcpy_toio > static inline void memcpy_toio(volatile void __iomem *dst, const void *src, int count) > { > memcpy((void __force *) dst, src, count); > @@ -556,4 +616,6 @@ extern void (*_dma_cache_inv)(unsigned long start, unsigned long size); > > void __ioread64_copy(void *to, const void __iomem *from, size_t count); > > +#include this #include blows up builds with: GEN Makefile Checking missing-syscalls for N32 CALL /local/tbogendoerfer/korg/linux/scripts/checksyscalls.sh Checking missing-syscalls for O32 CALL /local/tbogendoerfer/korg/linux/scripts/checksyscalls.sh CALL /local/tbogendoerfer/korg/linux/scripts/checksyscalls.sh CC init/version.o In file included from /local/tbogendoerfer/korg/linux/include/linux/spinlock.h:311:0, from /local/tbogendoerfer/korg/linux/include/linux/vmalloc.h:5, from /local/tbogendoerfer/korg/linux/include/asm-generic/io.h:994, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/io.h:618, from /local/tbogendoerfer/korg/linux/include/linux/io.h:13, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/mips-cps.h:11, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/smp-ops.h:16, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/smp.h:21, from /local/tbogendoerfer/korg/linux/include/linux/smp.h:113, from /local/tbogendoerfer/korg/linux/include/linux/lockdep.h:14, from /local/tbogendoerfer/korg/linux/include/linux/rcupdate.h:29, from /local/tbogendoerfer/korg/linux/include/linux/rculist.h:11, from /local/tbogendoerfer/korg/linux/include/linux/pid.h:5, from /local/tbogendoerfer/korg/linux/include/linux/sched.h:14, from /local/tbogendoerfer/korg/linux/include/linux/utsname.h:6, from /local/tbogendoerfer/korg/linux/init/version.c:17: /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h: In function ‘__raw_spin_trylock’: /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:90:3: error: implicit declaration of function ‘spin_acquire’ [-Werror=implicit-function-declaration] spin_acquire(&lock->dep_map, 0, 1, _RET_IP_); ^~~~~~~~~~~~ /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:90:21: error: ‘raw_spinlock_t {aka struct raw_spinlock}’ has no member named ‘dep_map’ spin_acquire(&lock->dep_map, 0, 1, _RET_IP_); ^~ /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h: In function ‘__raw_spin_lock_irqsave’: /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:110:20: error: ‘raw_spinlock_t {aka struct raw_spinlock}’ has no member named ‘dep_map’ spin_acquire(&lock->dep_map, 0, 0, _RET_IP_); ^~ /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:111:2: error: implicit declaration of function ‘LOCK_CONTENDED’ [-Werror=implicit-function-declaration] LOCK_CONTENDED(lock, do_raw_spin_trylock, do_raw_spin_lock); ^~~~~~~~~~~~~~ GEN Makefile Checking missing-syscalls for N32 CALL /local/tbogendoerfer/korg/linux/scripts/checksyscalls.sh Checking missing-syscalls for O32 CALL /local/tbogendoerfer/korg/linux/scripts/checksyscalls.sh CALL /local/tbogendoerfer/korg/linux/scripts/checksyscalls.sh CC init/version.o In file included from /local/tbogendoerfer/korg/linux/include/linux/spinlock.h:311:0, from /local/tbogendoerfer/korg/linux/include/linux/vmalloc.h:5, from /local/tbogendoerfer/korg/linux/include/asm-generic/io.h:994, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/io.h:618, from /local/tbogendoerfer/korg/linux/include/linux/io.h:13, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/mips-cps.h:11, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/smp-ops.h:16, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/smp.h:21, from /local/tbogendoerfer/korg/linux/include/linux/smp.h:113, from /local/tbogendoerfer/korg/linux/include/linux/lockdep.h:14, from /local/tbogendoerfer/korg/linux/include/linux/rcupdate.h:29, from /local/tbogendoerfer/korg/linux/include/linux/rculist.h:11, from /local/tbogendoerfer/korg/linux/include/linux/pid.h:5, from /local/tbogendoerfer/korg/linux/include/linux/sched.h:14, from /local/tbogendoerfer/korg/linux/include/linux/utsname.h:6, from /local/tbogendoerfer/korg/linux/init/version.c:17: /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h: In function ‘__raw_spin_trylock’: /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:90:3: error: implicit declaration of function ‘spin_acquire’ [-Werror=implicit-function-declaration] spin_acquire(&lock->dep_map, 0, 1, _RET_IP_); ^~~~~~~~~~~~ /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:90:21: error: ‘raw_spinlock_t {aka struct raw_spinlock}’ has no member named ‘dep_map’ spin_acquire(&lock->dep_map, 0, 1, _RET_IP_); ^~ /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h: In function ‘__raw_spin_lock_irqsave’: /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:110:20: error: ‘raw_spinlock_t {aka struct raw_spinlock}’ has no member named ‘dep_map’ spin_acquire(&lock->dep_map, 0, 0, _RET_IP_); ^~ /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:111:2: error: implicit declaration of function ‘LOCK_CONTENDED’ [-Werror=implicit-function-declaration] LOCK_CONTENDED(lock, do_raw_spin_trylock, do_raw_spin_lock); ^~~~~~~~~~~~~~ GEN Makefile Checking missing-syscalls for N32 CALL /local/tbogendoerfer/korg/linux/scripts/checksyscalls.sh Checking missing-syscalls for O32 CALL /local/tbogendoerfer/korg/linux/scripts/checksyscalls.sh CALL /local/tbogendoerfer/korg/linux/scripts/checksyscalls.sh CC init/version.o In file included from /local/tbogendoerfer/korg/linux/include/linux/spinlock.h:311:0, from /local/tbogendoerfer/korg/linux/include/linux/vmalloc.h:5, from /local/tbogendoerfer/korg/linux/include/asm-generic/io.h:994, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/io.h:618, from /local/tbogendoerfer/korg/linux/include/linux/io.h:13, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/mips-cps.h:11, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/smp-ops.h:16, from /local/tbogendoerfer/korg/linux/arch/mips/include/asm/smp.h:21, from /local/tbogendoerfer/korg/linux/include/linux/smp.h:113, from /local/tbogendoerfer/korg/linux/include/linux/lockdep.h:14, from /local/tbogendoerfer/korg/linux/include/linux/rcupdate.h:29, from /local/tbogendoerfer/korg/linux/include/linux/rculist.h:11, from /local/tbogendoerfer/korg/linux/include/linux/pid.h:5, from /local/tbogendoerfer/korg/linux/include/linux/sched.h:14, from /local/tbogendoerfer/korg/linux/include/linux/utsname.h:6, from /local/tbogendoerfer/korg/linux/init/version.c:17: /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h: In function ‘__raw_spin_trylock’: /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:90:3: error: implicit declaration of function ‘spin_acquire’ [-Werror=implicit-function-declaration] spin_acquire(&lock->dep_map, 0, 1, _RET_IP_); ^~~~~~~~~~~~ /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:90:21: error: ‘raw_spinlock_t {aka struct raw_spinlock}’ has no member named ‘dep_map’ spin_acquire(&lock->dep_map, 0, 1, _RET_IP_); ^~ /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h: In function ‘__raw_spin_lock_irqsave’: /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:110:20: error: ‘raw_spinlock_t {aka struct raw_spinlock}’ has no member named ‘dep_map’ spin_acquire(&lock->dep_map, 0, 0, _RET_IP_); ^~ /local/tbogendoerfer/korg/linux/include/linux/spinlock_api_smp.h:111:2: error: implicit declaration of function ‘LOCK_CONTENDED’ [-Werror=implicit-function-declaration] LOCK_CONTENDED(lock, do_raw_spin_trylock, do_raw_spin_lock); ^~~~~~~~~~~~~~ [...] I've cut the compiler output. Removing the asm-generic doesn't show this problem, but so far I fail to see the reason... Thomas. -- Crap can work. Given enough thrust pigs will fly, but it's not necessarily a good idea. [ RFC1925, 2.3 ]