| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <Z6oZ9dnYrlp5djiQ@smile.fi.intel.com>
Date: Mon, 10 Feb 2025 17:23:33 +0200
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: kernel test robot <lkp@...el.com>, Arnd Bergmann <arnd@...db.de>
Cc: Raag Jadav <raag.jadav@...el.com>, gregkh@...uxfoundation.org,
rafael@...nel.org, linus.walleij@...aro.org,
mika.westerberg@...ux.intel.com, dmitry.torokhov@...il.com,
lgirdwood@...il.com, broonie@...nel.org, sre@...nel.org,
jic23@...nel.org, przemyslaw.kitszel@...el.com,
oe-kbuild-all@...ts.linux.dev, linux-gpio@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-input@...r.kernel.org,
linux-sound@...r.kernel.org, linux-pm@...r.kernel.org,
linux-iio@...r.kernel.org
Subject: Re: [PATCH v4 01/20] driver core: Split devres APIs to
device/devres.h
+Cc: Arnd
On Mon, Feb 10, 2025 at 10:30:28PM +0800, kernel test robot wrote:
> Hi Raag,
>
> kernel test robot noticed the following build errors:
>
> [auto build test ERROR on a1e062ab4a1f19bb0e94093ef90ab9a74f1f7744]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Raag-Jadav/driver-core-Split-devres-APIs-to-device-devres-h/20250210-145732
> base: a1e062ab4a1f19bb0e94093ef90ab9a74f1f7744
> patch link: https://lore.kernel.org/r/20250210064906.2181867-2-raag.jadav%40intel.com
> patch subject: [PATCH v4 01/20] driver core: Split devres APIs to device/devres.h
> config: powerpc64-randconfig-002-20250210 (https://download.01.org/0day-ci/archive/20250210/202502102201.zLWaJC6V-lkp@intel.com/config)
> compiler: powerpc64-linux-gcc (GCC) 14.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250210/202502102201.zLWaJC6V-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@...el.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202502102201.zLWaJC6V-lkp@intel.com/
>
> All error/warnings (new ones prefixed by >>):
TBH I have no quick idea how to address this. It seems that io.h includes device.h
for no reason (but I haven't checked that carefully). OTOH, we need only
IOMEM_IS_ERR() definition which can simply be moved from io.h to err.h as the
former includes the latter and the definition depends only on compiler_types.h.
Arnd?
> In file included from include/linux/device/devres.h:7,
> from include/linux/device.h:31,
> from arch/powerpc/include/asm/io.h:22,
> from include/linux/scatterlist.h:9,
> from crypto/rsassa-pkcs1.c:11:
> include/linux/io.h: In function 'pci_remap_cfgspace':
> include/linux/io.h:106:16: error: implicit declaration of function 'ioremap_np' [-Wimplicit-function-declaration]
> 106 | return ioremap_np(offset, size) ?: ioremap(offset, size);
> | ^~~~~~~~~~
> include/linux/io.h:106:44: error: implicit declaration of function 'ioremap' [-Wimplicit-function-declaration]
> 106 | return ioremap_np(offset, size) ?: ioremap(offset, size);
> | ^~~~~~~
> include/linux/io.h:106:42: error: returning 'int' from a function with return type 'void *' makes pointer from integer without a cast [-Wint-conversion]
> 106 | return ioremap_np(offset, size) ?: ioremap(offset, size);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~
> arch/powerpc/include/asm/io.h: At top level:
> >> arch/powerpc/include/asm/io.h:885:22: error: conflicting types for 'ioremap'; have 'void *(phys_addr_t, long unsigned int)' {aka 'void *(long long unsigned int, long unsigned int)'}
> 885 | extern void __iomem *ioremap(phys_addr_t address, unsigned long size);
> | ^~~~~~~
> include/linux/io.h:106:44: note: previous implicit declaration of 'ioremap' with type 'int()'
> 106 | return ioremap_np(offset, size) ?: ioremap(offset, size);
> | ^~~~~~~
> In file included from include/asm-generic/io.h:17,
> from arch/powerpc/include/asm/io.h:1031:
> >> include/asm-generic/iomap.h:106:20: error: conflicting types for 'ioremap_np'; have 'void *(phys_addr_t, size_t)' {aka 'void *(long long unsigned int, long unsigned int)'}
> 106 | #define ioremap_np ioremap_np
> | ^~~~~~~~~~
> include/asm-generic/iomap.h:107:29: note: in expansion of macro 'ioremap_np'
> 107 | static inline void __iomem *ioremap_np(phys_addr_t offset, size_t size)
> | ^~~~~~~~~~
> include/linux/io.h:106:16: note: previous implicit declaration of 'ioremap_np' with type 'int()'
> 106 | return ioremap_np(offset, size) ?: ioremap(offset, size);
> | ^~~~~~~~~~
> --
> In file included from arch/powerpc/include/asm/io.h:274,
> from include/linux/io.h:14,
> from include/linux/device/devres.h:7,
> from include/linux/device.h:31,
> from include/linux/acpi.h:14,
> from include/linux/i2c.h:13,
> from drivers/input/touchscreen/ili210x.c:5:
> >> arch/powerpc/include/asm/eeh.h:304:45: warning: 'struct iommu_group' declared inside parameter list will not be visible outside of this definition or declaration
> 304 | struct eeh_pe *eeh_iommu_group_to_pe(struct iommu_group *group);
> | ^~~~~~~~~~~
> --
> In file included from arch/powerpc/include/asm/io.h:274,
> from include/linux/io.h:14,
> from include/linux/device/devres.h:7,
> from include/linux/device.h:31,
> from include/linux/input.h:19,
> from drivers/input/touchscreen/touchright.c:17:
> >> arch/powerpc/include/asm/eeh.h:304:45: warning: 'struct iommu_group' declared inside parameter list will not be visible outside of this definition or declaration
> 304 | struct eeh_pe *eeh_iommu_group_to_pe(struct iommu_group *group);
> | ^~~~~~~~~~~
> drivers/input/touchscreen/touchright.c: In function 'tr_connect':
> drivers/input/touchscreen/touchright.c:114:49: warning: '/input0' directive output may be truncated writing 7 bytes into a region of size between 1 and 32 [-Wformat-truncation=]
> 114 | snprintf(tr->phys, sizeof(tr->phys), "%s/input0", serio->phys);
> | ^~~~~~~
> drivers/input/touchscreen/touchright.c:114:9: note: 'snprintf' output between 8 and 39 bytes into a destination of size 32
> 114 | snprintf(tr->phys, sizeof(tr->phys), "%s/input0", serio->phys);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> --
> In file included from arch/powerpc/include/asm/io.h:274,
> from include/linux/io.h:14,
> from include/linux/device/devres.h:7,
> from include/linux/device.h:31,
> from include/linux/input.h:19,
> from drivers/input/touchscreen/penmount.c:17:
> >> arch/powerpc/include/asm/eeh.h:304:45: warning: 'struct iommu_group' declared inside parameter list will not be visible outside of this definition or declaration
> 304 | struct eeh_pe *eeh_iommu_group_to_pe(struct iommu_group *group);
> | ^~~~~~~~~~~
> drivers/input/touchscreen/penmount.c: In function 'pm_connect':
> drivers/input/touchscreen/penmount.c:211:49: warning: '/input0' directive output may be truncated writing 7 bytes into a region of size between 1 and 32 [-Wformat-truncation=]
> 211 | snprintf(pm->phys, sizeof(pm->phys), "%s/input0", serio->phys);
> | ^~~~~~~
> drivers/input/touchscreen/penmount.c:211:9: note: 'snprintf' output between 8 and 39 bytes into a destination of size 32
> 211 | snprintf(pm->phys, sizeof(pm->phys), "%s/input0", serio->phys);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> --
> In file included from arch/powerpc/include/asm/io.h:274,
> from include/linux/io.h:14,
> from include/linux/device/devres.h:7,
> from include/linux/device.h:31,
> from include/linux/input.h:19,
> from drivers/input/touchscreen/inexio.c:19:
> >> arch/powerpc/include/asm/eeh.h:304:45: warning: 'struct iommu_group' declared inside parameter list will not be visible outside of this definition or declaration
> 304 | struct eeh_pe *eeh_iommu_group_to_pe(struct iommu_group *group);
> | ^~~~~~~~~~~
> drivers/input/touchscreen/inexio.c: In function 'inexio_connect':
> drivers/input/touchscreen/inexio.c:126:59: warning: '/input0' directive output may be truncated writing 7 bytes into a region of size between 1 and 32 [-Wformat-truncation=]
> 126 | snprintf(pinexio->phys, sizeof(pinexio->phys), "%s/input0", serio->phys);
> | ^~~~~~~
> drivers/input/touchscreen/inexio.c:126:9: note: 'snprintf' output between 8 and 39 bytes into a destination of size 32
> 126 | snprintf(pinexio->phys, sizeof(pinexio->phys), "%s/input0", serio->phys);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> --
> In file included from arch/powerpc/include/asm/io.h:274,
> from include/linux/io.h:14,
> from include/linux/device/devres.h:7,
> from include/linux/device.h:31,
> from include/linux/input.h:19,
> from drivers/input/touchscreen/touchwin.c:24:
> >> arch/powerpc/include/asm/eeh.h:304:45: warning: 'struct iommu_group' declared inside parameter list will not be visible outside of this definition or declaration
> 304 | struct eeh_pe *eeh_iommu_group_to_pe(struct iommu_group *group);
> | ^~~~~~~~~~~
> drivers/input/touchscreen/touchwin.c: In function 'tw_connect':
> drivers/input/touchscreen/touchwin.c:121:49: warning: '/input0' directive output may be truncated writing 7 bytes into a region of size between 1 and 32 [-Wformat-truncation=]
> 121 | snprintf(tw->phys, sizeof(tw->phys), "%s/input0", serio->phys);
> | ^~~~~~~
> drivers/input/touchscreen/touchwin.c:121:9: note: 'snprintf' output between 8 and 39 bytes into a destination of size 32
> 121 | snprintf(tw->phys, sizeof(tw->phys), "%s/input0", serio->phys);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> --
> In file included from arch/powerpc/include/asm/io.h:274,
> from include/linux/io.h:14,
> from include/linux/device/devres.h:7,
> from include/linux/device.h:31,
> from include/linux/input.h:19,
> from drivers/input/touchscreen/hampshire.c:19:
> >> arch/powerpc/include/asm/eeh.h:304:45: warning: 'struct iommu_group' declared inside parameter list will not be visible outside of this definition or declaration
> 304 | struct eeh_pe *eeh_iommu_group_to_pe(struct iommu_group *group);
> | ^~~~~~~~~~~
> drivers/input/touchscreen/hampshire.c: In function 'hampshire_connect':
> drivers/input/touchscreen/hampshire.c:122:21: warning: '/input0' directive output may be truncated writing 7 bytes into a region of size between 1 and 32 [-Wformat-truncation=]
> 122 | "%s/input0", serio->phys);
> | ^~~~~~~
> drivers/input/touchscreen/hampshire.c:121:9: note: 'snprintf' output between 8 and 39 bytes into a destination of size 32
> 121 | snprintf(phampshire->phys, sizeof(phampshire->phys),
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 122 | "%s/input0", serio->phys);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~
> --
> In file included from arch/powerpc/include/asm/io.h:274,
> from include/linux/io.h:14,
> from include/linux/device/devres.h:7,
> from include/linux/device.h:31,
> from include/linux/input.h:19,
> from drivers/input/touchscreen/tsc40.c:12:
> >> arch/powerpc/include/asm/eeh.h:304:45: warning: 'struct iommu_group' declared inside parameter list will not be visible outside of this definition or declaration
> 304 | struct eeh_pe *eeh_iommu_group_to_pe(struct iommu_group *group);
> | ^~~~~~~~~~~
> drivers/input/touchscreen/tsc40.c: In function 'tsc_connect':
> drivers/input/touchscreen/tsc40.c:95:53: warning: '/input0' directive output may be truncated writing 7 bytes into a region of size between 1 and 32 [-Wformat-truncation=]
> 95 | snprintf(ptsc->phys, sizeof(ptsc->phys), "%s/input0", serio->phys);
> | ^~~~~~~
> drivers/input/touchscreen/tsc40.c:95:9: note: 'snprintf' output between 8 and 39 bytes into a destination of size 32
> 95 | snprintf(ptsc->phys, sizeof(ptsc->phys), "%s/input0", serio->phys);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> --
> In file included from arch/powerpc/include/asm/io.h:274,
> from include/linux/io.h:14,
> from include/linux/device/devres.h:7,
> from include/linux/device.h:31,
> from include/linux/input.h:19,
> from drivers/input/touchscreen/dynapro.c:20:
> >> arch/powerpc/include/asm/eeh.h:304:45: warning: 'struct iommu_group' declared inside parameter list will not be visible outside of this definition or declaration
> 304 | struct eeh_pe *eeh_iommu_group_to_pe(struct iommu_group *group);
> | ^~~~~~~~~~~
> drivers/input/touchscreen/dynapro.c: In function 'dynapro_connect':
> drivers/input/touchscreen/dynapro.c:123:21: warning: '/input0' directive output may be truncated writing 7 bytes into a region of size between 1 and 32 [-Wformat-truncation=]
> 123 | "%s/input0", serio->phys);
> | ^~~~~~~
> drivers/input/touchscreen/dynapro.c:122:9: note: 'snprintf' output between 8 and 39 bytes into a destination of size 32
> 122 | snprintf(pdynapro->phys, sizeof(pdynapro->phys),
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 123 | "%s/input0", serio->phys);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~
>
>
> vim +885 arch/powerpc/include/asm/io.h
>
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 853
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 854 /**
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 855 * ioremap - map bus memory into CPU space
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 856 * @address: bus address of the memory
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 857 * @size: size of the resource to map
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 858 *
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 859 * ioremap performs a platform specific sequence of operations to
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 860 * make bus memory CPU accessible via the readb/readw/readl/writeb/
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 861 * writew/writel functions and the other mmio helpers. The returned
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 862 * address is not guaranteed to be usable directly as a virtual
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 863 * address.
> 4cb3cee03d558fd include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-11 864 *
> 4cb3cee03d558fd include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-11 865 * We provide a few variations of it:
> 4cb3cee03d558fd include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-11 866 *
> 4cb3cee03d558fd include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-11 867 * * ioremap is the standard one and provides non-cacheable guarded mappings
> 4cb3cee03d558fd include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-11 868 * and can be hooked by the platform via ppc_md
> 4cb3cee03d558fd include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-11 869 *
> 40f1ce7fb7e8b5d arch/powerpc/include/asm/io.h Anton Blanchard 2011-05-08 870 * * ioremap_prot allows to specify the page flags as an argument and can
> 40f1ce7fb7e8b5d arch/powerpc/include/asm/io.h Anton Blanchard 2011-05-08 871 * also be hooked by the platform via ppc_md.
> 4cb3cee03d558fd include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-11 872 *
> be135f40899cb33 arch/powerpc/include/asm/io.h Anton Blanchard 2011-05-08 873 * * ioremap_wc enables write combining
> be135f40899cb33 arch/powerpc/include/asm/io.h Anton Blanchard 2011-05-08 874 *
> 86c391bd5f47101 arch/powerpc/include/asm/io.h Christophe Leroy 2018-10-09 875 * * ioremap_wt enables write through
> 86c391bd5f47101 arch/powerpc/include/asm/io.h Christophe Leroy 2018-10-09 876 *
> 86c391bd5f47101 arch/powerpc/include/asm/io.h Christophe Leroy 2018-10-09 877 * * ioremap_coherent maps coherent cached memory
> 86c391bd5f47101 arch/powerpc/include/asm/io.h Christophe Leroy 2018-10-09 878 *
> 4cb3cee03d558fd include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-11 879 * * iounmap undoes such a mapping and can be hooked
> 4cb3cee03d558fd include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-11 880 *
> 1cdab55d8a8313f arch/powerpc/include/asm/io.h Benjamin Herrenschmidt 2009-02-22 881 * * __ioremap_caller is the same as above but takes an explicit caller
> 1cdab55d8a8313f arch/powerpc/include/asm/io.h Benjamin Herrenschmidt 2009-02-22 882 * reference rather than using __builtin_return_address(0)
> 1cdab55d8a8313f arch/powerpc/include/asm/io.h Benjamin Herrenschmidt 2009-02-22 883 *
> ^1da177e4c3f415 include/asm-ppc64/io.h Linus Torvalds 2005-04-16 884 */
> 68a64357d15ae4f include/asm-powerpc/io.h Benjamin Herrenschmidt 2006-11-13 @885 extern void __iomem *ioremap(phys_addr_t address, unsigned long size);
> 8d05554dca2af6a arch/powerpc/include/asm/io.h Christophe Leroy 2023-07-06 886 #define ioremap ioremap
> 8d05554dca2af6a arch/powerpc/include/asm/io.h Christophe Leroy 2023-07-06 887 #define ioremap_prot ioremap_prot
> be135f40899cb33 arch/powerpc/include/asm/io.h Anton Blanchard 2011-05-08 888 extern void __iomem *ioremap_wc(phys_addr_t address, unsigned long size);
> 894fa235eb4ca0b arch/powerpc/include/asm/io.h Christophe Leroy 2020-11-21 889 #define ioremap_wc ioremap_wc
> 894fa235eb4ca0b arch/powerpc/include/asm/io.h Christophe Leroy 2020-11-21 890
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists