All of lore.kernel.org
 help / color / mirror / Atom feed
* [lee-linaro:tb-fix-w1-warnings 194/194] drivers/scsi/be2iscsi/be_main.c:4935: warning: expecting prototype for beiscsi_show_boot_tgt_info(). Prototype was for BEISCSI_SYSFS_ISCSI_BOOT_FLAGS() instead
@ 2021-02-26  2:31 kernel test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2021-02-26  2:31 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 2472 bytes --]

tree:   https://git.linaro.org/people/lee.jones/linux.git tb-fix-w1-warnings
head:   88ee6a99115ba595216378f4f3f1eaf160547d1f
commit: 88ee6a99115ba595216378f4f3f1eaf160547d1f [194/194] scsi: be2iscsi: be_main: Provide missing function name in header
config: x86_64-randconfig-a001-20210225 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project a921aaf789912d981cbb2036bdc91ad7289e1523)
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
        # install x86_64 cross compiling tool for clang build
        # apt-get install binutils-x86-64-linux-gnu
        git remote add lee-linaro https://git.linaro.org/people/lee.jones/linux.git
        git fetch --no-tags lee-linaro tb-fix-w1-warnings
        git checkout 88ee6a99115ba595216378f4f3f1eaf160547d1f
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 

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

All warnings (new ones prefixed by >>):

>> drivers/scsi/be2iscsi/be_main.c:4935: warning: expecting prototype for beiscsi_show_boot_tgt_info(). Prototype was for BEISCSI_SYSFS_ISCSI_BOOT_FLAGS() instead


vim +4935 drivers/scsi/be2iscsi/be_main.c

50a4b824be9e4a Jitendra Bhivare 2016-08-19  4928  
8d9ecd49426c95 Jitendra Bhivare 2017-10-10  4929  /**
88ee6a99115ba5 Lee Jones        2021-02-25  4930   * beiscsi_show_boot_tgt_info()
8d9ecd49426c95 Jitendra Bhivare 2017-10-10  4931   * Boot flag info for iscsi-utilities
8d9ecd49426c95 Jitendra Bhivare 2017-10-10  4932   * Bit 0 Block valid flag
8d9ecd49426c95 Jitendra Bhivare 2017-10-10  4933   * Bit 1 Firmware booting selected
8d9ecd49426c95 Jitendra Bhivare 2017-10-10  4934   */
8d9ecd49426c95 Jitendra Bhivare 2017-10-10 @4935  #define BEISCSI_SYSFS_ISCSI_BOOT_FLAGS	3
8d9ecd49426c95 Jitendra Bhivare 2017-10-10  4936  

:::::: The code at line 4935 was first introduced by commit
:::::: 8d9ecd49426c95ccb986d8111bbbf19920a231fe scsi: be2iscsi: Fix boot flags in sysfs

:::::: TO: Jitendra Bhivare <jitendra.bhivare@broadcom.com>
:::::: CC: Martin K. Petersen <martin.petersen@oracle.com>

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 36794 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [lee-linaro:tb-fix-w1-warnings 194/194] drivers/scsi/be2iscsi/be_main.c:4935: warning: expecting prototype for beiscsi_show_boot_tgt_info(). Prototype was for BEISCSI_SYSFS_ISCSI_BOOT_FLAGS() instead
@ 2021-02-26  9:45 kernel test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2021-02-26  9:45 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 9958 bytes --]

tree:   https://git.linaro.org/people/lee.jones/linux.git tb-fix-w1-warnings
head:   88ee6a99115ba595216378f4f3f1eaf160547d1f
commit: 88ee6a99115ba595216378f4f3f1eaf160547d1f [194/194] scsi: be2iscsi: be_main: Provide missing function name in header
config: s390-randconfig-p002-20210225 (attached as .config)
compiler: s390-linux-gcc (GCC) 9.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
        git remote add lee-linaro https://git.linaro.org/people/lee.jones/linux.git
        git fetch --no-tags lee-linaro tb-fix-w1-warnings
        git checkout 88ee6a99115ba595216378f4f3f1eaf160547d1f
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=s390 

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

All error/warnings (new ones prefixed by >>):

>> drivers/scsi/be2iscsi/be_main.c:4935: warning: expecting prototype for beiscsi_show_boot_tgt_info(). Prototype was for BEISCSI_SYSFS_ISCSI_BOOT_FLAGS() instead
--
   drivers/scsi/bfa/bfa_fcs_lport.c: In function 'bfa_fcs_lport_fdmi_build_rhba_pyld':
>> drivers/scsi/bfa/bfa_fcs_lport.c:1911:28: error: invalid type argument of unary '*' (have 'struct bfa_fcs_fdmi_hba_attr_s')
    1911 |  hba_attr = kzalloc(sizeof(*hba_attr), GFP_KERNEL);
         |                            ^~~~~~~~~
>> drivers/scsi/bfa/bfa_fcs_lport.c:1912:6: error: wrong type argument to unary exclamation mark
    1912 |  if (!hba_attr)
         |      ^
>> drivers/scsi/bfa/bfa_fcs_lport.c:2156:8: error: incompatible type for argument 1 of 'kfree'
    2156 |  kfree(hba_attr);
         |        ^~~~~~~~
         |        |
         |        struct bfa_fcs_fdmi_hba_attr_s
   In file included from arch/s390/include/asm/pci_io.h:8,
                    from arch/s390/include/asm/io.h:16,
                    from include/linux/io.h:13,
                    from include/linux/pci.h:39,
                    from drivers/scsi/bfa/bfad_drv.h:23,
                    from drivers/scsi/bfa/bfa_fcs_lport.c:11:
   include/linux/slab.h:185:12: note: expected 'const void *' but argument is of type 'struct bfa_fcs_fdmi_hba_attr_s'
     185 | void kfree(const void *);
         |            ^~~~~~~~~~~~
--
   drivers/scsi/mpt3sas/mpt3sas_scsih.c: In function '_scsih_scan_for_devices_after_reset':
>> drivers/scsi/mpt3sas/mpt3sas_scsih.c:10355:6: error: passing argument 3 of 'mpt3sas_config_get_raid_volume_pg1' from incompatible pointer type [-Werror=incompatible-pointer-types]
   10355 |      &volume_pg1, MPI2_RAID_VOLUME_PGAD_FORM_GET_NEXT_HANDLE, handle))) {
         |      ^~~~~~~~~~~
         |      |
         |      Mpi2RaidVolPage1_t ** {aka struct _MPI2_CONFIG_PAGE_RAID_VOL_1 **}
   In file included from drivers/scsi/mpt3sas/mpt3sas_scsih.c:60:
   drivers/scsi/mpt3sas/mpt3sas_base.h:1856:52: note: expected 'Mpi2RaidVolPage1_t *' {aka 'struct _MPI2_CONFIG_PAGE_RAID_VOL_1 *'} but argument is of type 'Mpi2RaidVolPage1_t **' {aka 'struct _MPI2_CONFIG_PAGE_RAID_VOL_1 **'}
    1856 |  Mpi2ConfigReply_t *mpi_reply, Mpi2RaidVolPage1_t *config_page, u32 form,
         |                                ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~
   In file included from include/linux/swab.h:5,
                    from include/uapi/linux/byteorder/big_endian.h:13,
                    from include/linux/byteorder/big_endian.h:5,
                    from arch/s390/include/uapi/asm/byteorder.h:5,
                    from include/asm-generic/bitops/le.h:6,
                    from arch/s390/include/asm/bitops.h:392,
                    from include/linux/bitops.h:32,
                    from include/linux/kernel.h:11,
                    from include/linux/list.h:9,
                    from include/linux/module.h:12,
                    from drivers/scsi/mpt3sas/mpt3sas_scsih.c:45:
>> drivers/scsi/mpt3sas/mpt3sas_scsih.c:10363:34: error: 'volume_pg1' is a pointer; did you mean to use '->'?
   10363 |   handle = le16_to_cpu(volume_pg1.DevHandle);
         |                                  ^
   include/uapi/linux/swab.h:102:54: note: in definition of macro '__swab16'
     102 | #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
         |                                                      ^
   include/linux/byteorder/generic.h:91:21: note: in expansion of macro '__le16_to_cpu'
      91 | #define le16_to_cpu __le16_to_cpu
         |                     ^~~~~~~~~~~~~
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10363:12: note: in expansion of macro 'le16_to_cpu'
   10363 |   handle = le16_to_cpu(volume_pg1.DevHandle);
         |            ^~~~~~~~~~~
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10366:29: error: 'volume_pg1' is a pointer; did you mean to use '->'?
   10366 |       le64_to_cpu(volume_pg1.WWID));
         |                             ^
   include/uapi/linux/swab.h:128:54: note: in definition of macro '__swab64'
     128 | #define __swab64(x) (__u64)__builtin_bswap64((__u64)(x))
         |                                                      ^
   include/linux/byteorder/generic.h:87:21: note: in expansion of macro '__le64_to_cpu'
      87 | #define le64_to_cpu __le64_to_cpu
         |                     ^~~~~~~~~~~~~
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10366:7: note: in expansion of macro 'le64_to_cpu'
   10366 |       le64_to_cpu(volume_pg1.WWID));
         |       ^~~~~~~~~~~
>> drivers/scsi/mpt3sas/mpt3sas_scsih.c:10371:7: error: passing argument 3 of 'mpt3sas_config_get_raid_volume_pg0' from incompatible pointer type [-Werror=incompatible-pointer-types]
   10371 |       &volume_pg0, MPI2_RAID_VOLUME_PGAD_FORM_HANDLE, handle,
         |       ^~~~~~~~~~~
         |       |
         |       Mpi2RaidVolPage0_t ** {aka struct _MPI2_CONFIG_PAGE_RAID_VOL_0 **}
   In file included from drivers/scsi/mpt3sas/mpt3sas_scsih.c:60:
   drivers/scsi/mpt3sas/mpt3sas_base.h:1861:52: note: expected 'Mpi2RaidVolPage0_t *' {aka 'struct _MPI2_CONFIG_PAGE_RAID_VOL_0 *'} but argument is of type 'Mpi2RaidVolPage0_t **' {aka 'struct _MPI2_CONFIG_PAGE_RAID_VOL_0 **'}
    1861 |  Mpi2ConfigReply_t *mpi_reply, Mpi2RaidVolPage0_t *config_page, u32 form,
         |                                ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~
>> drivers/scsi/mpt3sas/mpt3sas_scsih.c:10381:17: error: 'volume_pg0' is a pointer; did you mean to use '->'?
   10381 |   if (volume_pg0.VolumeState == MPI2_RAID_VOL_STATE_OPTIMAL ||
         |                 ^
         |                 ->
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10382:17: error: 'volume_pg0' is a pointer; did you mean to use '->'?
   10382 |       volume_pg0.VolumeState == MPI2_RAID_VOL_STATE_ONLINE ||
         |                 ^
         |                 ->
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10383:17: error: 'volume_pg0' is a pointer; did you mean to use '->'?
   10383 |       volume_pg0.VolumeState == MPI2_RAID_VOL_STATE_DEGRADED) {
         |                 ^
         |                 ->
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10386:37: error: 'volume_pg1' is a pointer; did you mean to use '->'?
   10386 |    element.VolDevHandle = volume_pg1.DevHandle;
         |                                     ^
         |                                     ->
   In file included from include/linux/kernel.h:16,
                    from include/linux/list.h:9,
                    from include/linux/module.h:12,
                    from drivers/scsi/mpt3sas/mpt3sas_scsih.c:45:
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10388:16: error: 'volume_pg1' is a pointer; did you mean to use '->'?
   10388 |      volume_pg1.DevHandle);
         |                ^
   include/linux/printk.h:373:34: note: in definition of macro 'pr_info'
     373 |  printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
         |                                  ^~~~~~~~~~~
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10387:4: note: in expansion of macro 'ioc_info'
   10387 |    ioc_info(ioc, "\tBEFORE adding volume: handle (0x%04x)\n",
         |    ^~~~~~~~
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10391:16: error: 'volume_pg1' is a pointer; did you mean to use '->'?
   10391 |      volume_pg1.DevHandle);
         |                ^
   include/linux/printk.h:373:34: note: in definition of macro 'pr_info'
     373 |  printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
         |                                  ^~~~~~~~~~~
   drivers/scsi/mpt3sas/mpt3sas_scsih.c:10390:4: note: in expansion of macro 'ioc_info'
   10390 |    ioc_info(ioc, "\tAFTER adding volume: handle (0x%04x)\n",
         |    ^~~~~~~~
   cc1: some warnings being treated as errors


vim +4935 drivers/scsi/be2iscsi/be_main.c

50a4b824be9e4a01 Jitendra Bhivare 2016-08-19  4928  
8d9ecd49426c95cc Jitendra Bhivare 2017-10-10  4929  /**
88ee6a99115ba595 Lee Jones        2021-02-25  4930   * beiscsi_show_boot_tgt_info()
8d9ecd49426c95cc Jitendra Bhivare 2017-10-10  4931   * Boot flag info for iscsi-utilities
8d9ecd49426c95cc Jitendra Bhivare 2017-10-10  4932   * Bit 0 Block valid flag
8d9ecd49426c95cc Jitendra Bhivare 2017-10-10  4933   * Bit 1 Firmware booting selected
8d9ecd49426c95cc Jitendra Bhivare 2017-10-10  4934   */
8d9ecd49426c95cc Jitendra Bhivare 2017-10-10 @4935  #define BEISCSI_SYSFS_ISCSI_BOOT_FLAGS	3
8d9ecd49426c95cc Jitendra Bhivare 2017-10-10  4936  

:::::: The code at line 4935 was first introduced by commit
:::::: 8d9ecd49426c95ccb986d8111bbbf19920a231fe scsi: be2iscsi: Fix boot flags in sysfs

:::::: TO: Jitendra Bhivare <jitendra.bhivare@broadcom.com>
:::::: CC: Martin K. Petersen <martin.petersen@oracle.com>

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 33776 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-02-26  9:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-26  2:31 [lee-linaro:tb-fix-w1-warnings 194/194] drivers/scsi/be2iscsi/be_main.c:4935: warning: expecting prototype for beiscsi_show_boot_tgt_info(). Prototype was for BEISCSI_SYSFS_ISCSI_BOOT_FLAGS() instead kernel test robot
2021-02-26  9:45 kernel test robot

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.