linux-kernel - [tglx-devel:irq/msi 4/8] include/asm-generic/io.h:173:15: error: too many arguments to function 'readl'

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

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Date:   Wed, 21 Jul 2021 10:53:36 +0800
From:   kernel test robot <lkp@...el.com>
To:     Thomas Gleixner <tglx@...utronix.de>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org
Subject: [tglx-devel:irq/msi 4/8] include/asm-generic/io.h:173:15: error: too
 many arguments to function 'readl'

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel.git irq/msi
head:   148a9616eb053a8c0d0c9859fde99fa2aee3ccea
commit: abefd748ab60997c05fcd79e7b4072cd4fd47c19 [4/8] PCI/MSI: Enforce MSI[X] entry updates to be visible
config: ia64-randconfig-r025-20210720 (attached as .config)
compiler: ia64-linux-gcc (GCC) 10.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel.git/commit/?id=abefd748ab60997c05fcd79e7b4072cd4fd47c19
        git remote add tglx-devel https://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel.git
        git fetch --no-tags tglx-devel irq/msi
        git checkout abefd748ab60997c05fcd79e7b4072cd4fd47c19
        # save the attached .config to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-10.3.0 make.cross O=build_dir ARCH=ia64 SHELL=/bin/bash drivers/pci/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>

All errors (new ones prefixed by >>):

   drivers/pci/msi.c: In function '__pci_write_msi_msg':
   drivers/pci/msi.c:316:12: warning: passing argument 1 of 'readl' makes pointer from integer without a cast [-Wint-conversion]
     316 |   readl(msg->data, base + PCI_MSIX_ENTRY_DATA);
         |         ~~~^~~~~~
         |            |
         |            u32 {aka unsigned int}
   In file included from arch/ia64/include/asm/io.h:281,
                    from arch/ia64/include/asm/uaccess.h:41,
                    from include/linux/uaccess.h:11,
                    from include/linux/sched/task.h:11,
                    from include/linux/sched/signal.h:9,
                    from include/linux/rcuwait.h:6,
                    from include/linux/percpu-rwsem.h:7,
                    from include/linux/fs.h:33,
                    from include/linux/huge_mm.h:8,
                    from include/linux/mm.h:728,
                    from drivers/pci/msi.c:11:
   include/asm-generic/io.h:174:54: note: expected 'const volatile void *' but argument is of type 'u32' {aka 'unsigned int'}
     174 | static inline u32 readl(const volatile void __iomem *addr)
         |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~
>> include/asm-generic/io.h:173:15: error: too many arguments to function 'readl'
     173 | #define readl readl
         |               ^~~~~
   drivers/pci/msi.c:316:3: note: in expansion of macro 'readl'
     316 |   readl(msg->data, base + PCI_MSIX_ENTRY_DATA);
         |   ^~~~~
   include/asm-generic/io.h:173:15: note: declared here
     173 | #define readl readl
         |               ^~~~~
   include/asm-generic/io.h:174:19: note: in expansion of macro 'readl'
     174 | static inline u32 readl(const volatile void __iomem *addr)
         |                   ^~~~~


vim +/readl +173 include/asm-generic/io.h

7dc59bdde70633 GuanXuetao     2011-02-22  171  
9216efafc52ff9 Thierry Reding 2014-10-01  172  #ifndef readl
9216efafc52ff9 Thierry Reding 2014-10-01 @173  #define readl readl
9216efafc52ff9 Thierry Reding 2014-10-01  174  static inline u32 readl(const volatile void __iomem *addr)
3f7e212df82ca0 Arnd Bergmann  2009-05-13  175  {
032d59e1cde9dd Sinan Kaya     2018-04-05  176  	u32 val;
032d59e1cde9dd Sinan Kaya     2018-04-05  177  
032d59e1cde9dd Sinan Kaya     2018-04-05  178  	__io_br();
c1d55d50139bea Stafford Horne 2020-07-29  179  	val = __le32_to_cpu((__le32 __force)__raw_readl(addr));
abbbbc83a210e9 Will Deacon    2019-02-22  180  	__io_ar(val);
032d59e1cde9dd Sinan Kaya     2018-04-05  181  	return val;
3f7e212df82ca0 Arnd Bergmann  2009-05-13  182  }
9216efafc52ff9 Thierry Reding 2014-10-01  183  #endif
3f7e212df82ca0 Arnd Bergmann  2009-05-13  184  

:::::: The code at line 173 was first introduced by commit
:::::: 9216efafc52ff99e9351ef60de5fcafc2bc8adb6 asm-generic/io.h: Reconcile I/O accessor overrides

:::::: TO: Thierry Reding <treding@...dia.com>
:::::: CC: Thierry Reding <treding@...dia.com>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Download attachment ".config.gz" of type "application/gzip" (26645 bytes)