From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: [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
Date: Fri, 26 Feb 2021 17:45:11 +0800 [thread overview]
Message-ID: <202102261706.7r4NMGdB-lkp@intel.com> (raw)
[-- 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 --]
next reply other threads:[~2021-02-26 9:45 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-26 9:45 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
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
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=202102261706.7r4NMGdB-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/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.