All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Julian Vetter <jvetter@kalrayinc.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Russell King <linux@armlinux.org.uk>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>, Guo Ren <guoren@kernel.org>,
	Huacai Chen <chenhuacai@kernel.org>,
	WANG Xuerui <kernel@xen0n.name>,
	Andrew Morton <akpm@linux-foundation.org>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Richard Henderson <richard.henderson@linaro.org>,
	Ivan Kokshaysky <ink@jurassic.park.msu.ru>,
	Matt Turner <mattst88@gmail.com>,
	"James E . J . Bottomley" <James.Bottomley@hansenpartnership.com>,
	Helge Deller <deller@gmx.de>,
	Yoshinori Sato <ysato@users.sourceforge.jp>,
	Rich Felker <dalias@libc.org>,
	John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>,
	Richard Weinberger <richard@nod.at>,
	Anton Ivanov <anton.ivanov@cambridgegreys.com>,
	Johannes Berg <johannes@sipsolutions.net>,
	Michael Ellerman <mpe@ellerman.id.au>,
	Nicholas Piggin <npiggin@gmail.com>,
	Christophe Leroy <christophe.leroy@csgroup.eu>,
	Naveen N Rao <naveen@kernel.org>,
	Madhavan Srinivasan <maddy@linux.ibm.com>,
	Heiko Carstens <hca@linux.ibm.com>,
	Vasily Gorbik <gor@linux.ibm.com>,
	Alexander Gordeev <agordeev@linux.ibm.com>,
	Christian Borntraeger <borntraeger@linux.ibm.com>
Cc: oe-kbuild-all@lists.linux.dev,
	Linux Memory Management List <linux-mm@kvack.org>
Subject: Re: [PATCH v8 01/14] Consolidate IO memcpy/memset into iomap_copy.c
Date: Wed, 9 Oct 2024 10:54:53 +0800	[thread overview]
Message-ID: <202410091032.Q4NgrjoB-lkp@intel.com> (raw)
In-Reply-To: <20241008075023.3052370-2-jvetter@kalrayinc.com>

Hi Julian,

kernel test robot noticed the following build errors:

[auto build test ERROR on powerpc/next]
[also build test ERROR on powerpc/fixes arm64/for-next/core deller-parisc/for-next s390/features mtd/mtd/next mtd/mtd/fixes linus/master arm/for-next arm/fixes v6.12-rc2 next-20241008]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Julian-Vetter/Consolidate-IO-memcpy-memset-into-iomap_copy-c/20241008-163134
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
patch link:    https://lore.kernel.org/r/20241008075023.3052370-2-jvetter%40kalrayinc.com
patch subject: [PATCH v8 01/14] Consolidate IO memcpy/memset into iomap_copy.c
config: sparc-allnoconfig (https://download.01.org/0day-ci/archive/20241009/202410091032.Q4NgrjoB-lkp@intel.com/config)
compiler: sparc-linux-gcc (GCC) 14.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241009/202410091032.Q4NgrjoB-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202410091032.Q4NgrjoB-lkp@intel.com/

All errors (new ones prefixed by >>):

   In file included from arch/sparc/include/asm/io.h:7,
                    from include/linux/io.h:14,
                    from include/linux/irq.h:20,
                    from include/asm-generic/hardirq.h:17,
                    from arch/sparc/include/asm/hardirq_32.h:11,
                    from arch/sparc/include/asm/hardirq.h:7,
                    from include/linux/hardirq.h:11,
                    from include/linux/highmem.h:12,
                    from include/linux/pagemap.h:11,
                    from lib/buildid.c:7:
>> arch/sparc/include/asm/io_32.h:23:20: error: static declaration of '_memset_io' follows non-static declaration
      23 | static inline void _memset_io(volatile void __iomem *dst,
         |                    ^~~~~~~~~~
   arch/sparc/include/asm/io_32.h:10:31: note: previous declaration of '_memset_io' with type 'void(volatile void *, int,  size_t)' {aka 'void(volatile void *, int,  unsigned int)'}
      10 | #define memset_io(d,c,sz)     _memset_io(d,c,sz)
         |                               ^~~~~~~~~~
   include/asm-generic/io.h:109:13: note: in expansion of macro 'memset_io'
     109 | extern void memset_io(volatile void __iomem *dst, int c, size_t count);
         |             ^~~~~~~~~
>> arch/sparc/include/asm/io_32.h:34:20: error: static declaration of '_memcpy_fromio' follows non-static declaration
      34 | static inline void _memcpy_fromio(void *dst, const volatile void __iomem *src,
         |                    ^~~~~~~~~~~~~~
   arch/sparc/include/asm/io_32.h:11:31: note: previous declaration of '_memcpy_fromio' with type 'void(void *, const volatile void *, size_t)' {aka 'void(void *, const volatile void *, unsigned int)'}
      11 | #define memcpy_fromio(d,s,sz) _memcpy_fromio(d,s,sz)
         |                               ^~~~~~~~~~~~~~
   include/asm-generic/io.h:105:13: note: in expansion of macro 'memcpy_fromio'
     105 | extern void memcpy_fromio(void *to, const volatile void __iomem *from,
         |             ^~~~~~~~~~~~~
>> arch/sparc/include/asm/io_32.h:46:20: error: static declaration of '_memcpy_toio' follows non-static declaration
      46 | static inline void _memcpy_toio(volatile void __iomem *dst, const void *src,
         |                    ^~~~~~~~~~~~
   arch/sparc/include/asm/io_32.h:12:31: note: previous declaration of '_memcpy_toio' with type 'void(volatile void *, const void *, size_t)' {aka 'void(volatile void *, const void *, unsigned int)'}
      12 | #define memcpy_toio(d,s,sz)   _memcpy_toio(d,s,sz)
         |                               ^~~~~~~~~~~~
   include/asm-generic/io.h:107:13: note: in expansion of macro 'memcpy_toio'
     107 | extern void memcpy_toio(volatile void __iomem *to, const void *from,
         |             ^~~~~~~~~~~
--
   In file included from arch/sparc/include/asm/io.h:7,
                    from include/linux/io.h:14,
                    from include/linux/irq.h:20,
                    from include/asm-generic/hardirq.h:17,
                    from arch/sparc/include/asm/hardirq_32.h:11,
                    from arch/sparc/include/asm/hardirq.h:7,
                    from include/linux/hardirq.h:11,
                    from include/linux/interrupt.h:11,
                    from include/linux/rtc.h:17,
                    from lib/vsprintf.c:36:
>> arch/sparc/include/asm/io_32.h:23:20: error: static declaration of '_memset_io' follows non-static declaration
      23 | static inline void _memset_io(volatile void __iomem *dst,
         |                    ^~~~~~~~~~
   arch/sparc/include/asm/io_32.h:10:31: note: previous declaration of '_memset_io' with type 'void(volatile void *, int,  size_t)' {aka 'void(volatile void *, int,  unsigned int)'}
      10 | #define memset_io(d,c,sz)     _memset_io(d,c,sz)
         |                               ^~~~~~~~~~
   include/asm-generic/io.h:109:13: note: in expansion of macro 'memset_io'
     109 | extern void memset_io(volatile void __iomem *dst, int c, size_t count);
         |             ^~~~~~~~~
>> arch/sparc/include/asm/io_32.h:34:20: error: static declaration of '_memcpy_fromio' follows non-static declaration
      34 | static inline void _memcpy_fromio(void *dst, const volatile void __iomem *src,
         |                    ^~~~~~~~~~~~~~
   arch/sparc/include/asm/io_32.h:11:31: note: previous declaration of '_memcpy_fromio' with type 'void(void *, const volatile void *, size_t)' {aka 'void(void *, const volatile void *, unsigned int)'}
      11 | #define memcpy_fromio(d,s,sz) _memcpy_fromio(d,s,sz)
         |                               ^~~~~~~~~~~~~~
   include/asm-generic/io.h:105:13: note: in expansion of macro 'memcpy_fromio'
     105 | extern void memcpy_fromio(void *to, const volatile void __iomem *from,
         |             ^~~~~~~~~~~~~
>> arch/sparc/include/asm/io_32.h:46:20: error: static declaration of '_memcpy_toio' follows non-static declaration
      46 | static inline void _memcpy_toio(volatile void __iomem *dst, const void *src,
         |                    ^~~~~~~~~~~~
   arch/sparc/include/asm/io_32.h:12:31: note: previous declaration of '_memcpy_toio' with type 'void(volatile void *, const void *, size_t)' {aka 'void(volatile void *, const void *, unsigned int)'}
      12 | #define memcpy_toio(d,s,sz)   _memcpy_toio(d,s,sz)
         |                               ^~~~~~~~~~~~
   include/asm-generic/io.h:107:13: note: in expansion of macro 'memcpy_toio'
     107 | extern void memcpy_toio(volatile void __iomem *to, const void *from,
         |             ^~~~~~~~~~~
   lib/vsprintf.c: In function 'va_format':
   lib/vsprintf.c:1683:9: warning: function 'va_format' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format]
    1683 |         buf += vsnprintf(buf, end > buf ? end - buf : 0, va_fmt->fmt, va);
         |         ^~~
--
   In file included from arch/sparc/include/asm/io.h:7,
                    from include/linux/io.h:14,
                    from lib/iomap_copy.c:8:
>> arch/sparc/include/asm/io_32.h:23:20: error: static declaration of '_memset_io' follows non-static declaration
      23 | static inline void _memset_io(volatile void __iomem *dst,
         |                    ^~~~~~~~~~
   arch/sparc/include/asm/io_32.h:10:31: note: previous declaration of '_memset_io' with type 'void(volatile void *, int,  size_t)' {aka 'void(volatile void *, int,  unsigned int)'}
      10 | #define memset_io(d,c,sz)     _memset_io(d,c,sz)
         |                               ^~~~~~~~~~
   include/asm-generic/io.h:109:13: note: in expansion of macro 'memset_io'
     109 | extern void memset_io(volatile void __iomem *dst, int c, size_t count);
         |             ^~~~~~~~~
>> arch/sparc/include/asm/io_32.h:34:20: error: static declaration of '_memcpy_fromio' follows non-static declaration
      34 | static inline void _memcpy_fromio(void *dst, const volatile void __iomem *src,
         |                    ^~~~~~~~~~~~~~
   arch/sparc/include/asm/io_32.h:11:31: note: previous declaration of '_memcpy_fromio' with type 'void(void *, const volatile void *, size_t)' {aka 'void(void *, const volatile void *, unsigned int)'}
      11 | #define memcpy_fromio(d,s,sz) _memcpy_fromio(d,s,sz)
         |                               ^~~~~~~~~~~~~~
   include/asm-generic/io.h:105:13: note: in expansion of macro 'memcpy_fromio'
     105 | extern void memcpy_fromio(void *to, const volatile void __iomem *from,
         |             ^~~~~~~~~~~~~
>> arch/sparc/include/asm/io_32.h:46:20: error: static declaration of '_memcpy_toio' follows non-static declaration
      46 | static inline void _memcpy_toio(volatile void __iomem *dst, const void *src,
         |                    ^~~~~~~~~~~~
   arch/sparc/include/asm/io_32.h:12:31: note: previous declaration of '_memcpy_toio' with type 'void(volatile void *, const void *, size_t)' {aka 'void(volatile void *, const void *, unsigned int)'}
      12 | #define memcpy_toio(d,s,sz)   _memcpy_toio(d,s,sz)
         |                               ^~~~~~~~~~~~
   include/asm-generic/io.h:107:13: note: in expansion of macro 'memcpy_toio'
     107 | extern void memcpy_toio(volatile void __iomem *to, const void *from,
         |             ^~~~~~~~~~~
   lib/iomap_copy.c:10:10: fatal error: linux/unaligned.h: No such file or directory
      10 | #include <linux/unaligned.h>
         |          ^~~~~~~~~~~~~~~~~~~
   compilation terminated.


vim +/_memset_io +23 arch/sparc/include/asm/io_32.h

f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  22  
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26 @23  static inline void _memset_io(volatile void __iomem *dst,
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  24                                int c, __kernel_size_t n)
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  25  {
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  26  	volatile void __iomem *d = dst;
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  27  
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  28  	while (n--) {
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  29  		writeb(c, d);
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  30  		d++;
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  31  	}
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  32  }
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  33  
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26 @34  static inline void _memcpy_fromio(void *dst, const volatile void __iomem *src,
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  35                                    __kernel_size_t n)
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  36  {
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  37  	char *d = dst;
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  38  
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  39  	while (n--) {
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  40  		char tmp = readb(src);
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  41  		*d++ = tmp;
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  42  		src++;
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  43  	}
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  44  }
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  45  
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26 @46  static inline void _memcpy_toio(volatile void __iomem *dst, const void *src,
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  47                                  __kernel_size_t n)
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  48  {
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  49  	const char *s = src;
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  50  	volatile void __iomem *d = dst;
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  51  
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  52  	while (n--) {
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  53  		char tmp = *s++;
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  54  		writeb(tmp, d);
e1039fb426094d arch/sparc/include/asm/io_32.h Sam Ravnborg 2014-04-26  55  		d++;
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  56  	}
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  57  }
f5e706ad886b6a include/asm-sparc/io_32.h      Sam Ravnborg 2008-07-17  58  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

  parent reply	other threads:[~2024-10-09  2:55 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-08  7:50 [PATCH v8 00/14] Consolidate IO memcpy functions Julian Vetter
2024-10-08  7:50 ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 01/14] Consolidate IO memcpy/memset into iomap_copy.c Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  9:27   ` Arnd Bergmann
2024-10-08  9:27     ` Arnd Bergmann
2024-10-08  9:32     ` Christoph Hellwig
2024-10-08  9:32       ` Christoph Hellwig
2024-10-08 11:46   ` Christoph Hellwig
2024-10-08 11:46     ` Christoph Hellwig
2024-10-08 12:29     ` Arnd Bergmann
2024-10-08 12:29       ` Arnd Bergmann
2024-10-09  2:54   ` kernel test robot [this message]
2024-10-09  3:25   ` kernel test robot
2024-10-09  7:13   ` kernel test robot
2024-10-08  7:50 ` [PATCH v8 02/14] arm64: Use generic IO memcpy/memset Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 03/14] csky: " Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 04/14] loongarch: " Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 05/14] m68k: Align prototypes of " Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 06/14] alpha: " Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08 14:54   ` Richard Henderson
2024-10-08 14:54     ` Richard Henderson
2024-10-08  7:50 ` [PATCH v8 07/14] parisc: " Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 08/14] sh: " Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 09/14] arm: Align prototype of IO memset Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 10/14] powerpc: Align prototypes of IO memcpy and memset Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 11/14] s390: Add wrappers around zpci_memcpy/zpci_memset Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08 13:42   ` Niklas Schnelle
2024-10-08 13:42     ` Niklas Schnelle
2024-10-08  7:50 ` [PATCH v8 12/14] bus: mhi: ep: Add HAS_IOMEM || INDIRECT_IOMEM dependency Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  7:50 ` [PATCH v8 13/14] mtd: " Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  8:33   ` Miquel Raynal
2024-10-08  8:33     ` Miquel Raynal
2024-10-08  7:50 ` [PATCH v8 14/14] sound: Make CONFIG_SND depend on INDIRECT_IOMEM instead of UML Julian Vetter
2024-10-08  7:50   ` Julian Vetter
2024-10-08  8:16   ` Takashi Iwai
2024-10-08  8:16     ` Takashi Iwai

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=202410091032.Q4NgrjoB-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=agordeev@linux.ibm.com \
    --cc=akpm@linux-foundation.org \
    --cc=anton.ivanov@cambridgegreys.com \
    --cc=arnd@arndb.de \
    --cc=borntraeger@linux.ibm.com \
    --cc=catalin.marinas@arm.com \
    --cc=chenhuacai@kernel.org \
    --cc=christophe.leroy@csgroup.eu \
    --cc=dalias@libc.org \
    --cc=deller@gmx.de \
    --cc=geert@linux-m68k.org \
    --cc=glaubitz@physik.fu-berlin.de \
    --cc=gor@linux.ibm.com \
    --cc=guoren@kernel.org \
    --cc=hca@linux.ibm.com \
    --cc=ink@jurassic.park.msu.ru \
    --cc=johannes@sipsolutions.net \
    --cc=jvetter@kalrayinc.com \
    --cc=kernel@xen0n.name \
    --cc=linux-mm@kvack.org \
    --cc=linux@armlinux.org.uk \
    --cc=maddy@linux.ibm.com \
    --cc=mattst88@gmail.com \
    --cc=mpe@ellerman.id.au \
    --cc=naveen@kernel.org \
    --cc=npiggin@gmail.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=richard.henderson@linaro.org \
    --cc=richard@nod.at \
    --cc=will@kernel.org \
    --cc=ysato@users.sourceforge.jp \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.