Lee Jones
6b87e435e4
scsi: pm8001: Fix some misnamed function descriptions
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/pm8001/pm8001_hwi.c:1183: warning: expecting prototype for pm8001_chip_interrupt_enable(). Prototype was for pm8001_chip_intx_interrupt_enable() instead
drivers/scsi/pm8001/pm8001_hwi.c:1257: warning: expecting prototype for pm8001_chip_intx_interrupt_disable(). Prototype was for pm8001_chip_interrupt_disable() instead
drivers/scsi/pm8001/pm8001_hwi.c:3235: warning: expecting prototype for asd_get_attached_sas_addr(). Prototype was for pm8001_get_attached_sas_addr() instead
drivers/scsi/pm8001/pm8001_hwi.c:3555: warning: expecting prototype for fw_flash_update_resp(). Prototype was for pm8001_mpi_fw_flash_update_resp() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-19-lee.jones@linaro.org
Cc: Jack Wang <jinpu.wang@cloud.ionos.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: linux-scsi@vger.kernel.org
Acked-by: Jack Wang <jinpu.wang@cloud.ionos.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:52 -04:00
Lee Jones
8514e2f1e2
scsi: lpfc: Fix a bunch of kernel-doc issues
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/lpfc/lpfc_sli.c:9654: warning: expecting prototype for lpfc_sli_iocb2wqe(). Prototype was for lpfc_sli4_iocb2wqe() instead
drivers/scsi/lpfc/lpfc_sli.c:10439: warning: Function parameter or member 'phba' not described in 'lpfc_sli_issue_fcp_io'
drivers/scsi/lpfc/lpfc_sli.c:10439: warning: Function parameter or member 'ring_number' not described in 'lpfc_sli_issue_fcp_io'
drivers/scsi/lpfc/lpfc_sli.c:10439: warning: Function parameter or member 'piocb' not described in 'lpfc_sli_issue_fcp_io'
drivers/scsi/lpfc/lpfc_sli.c:10439: warning: Function parameter or member 'flag' not described in 'lpfc_sli_issue_fcp_io'
drivers/scsi/lpfc/lpfc_sli.c:14189: warning: expecting prototype for lpfc_sli4_sp_process_cq(). Prototype was for __lpfc_sli4_sp_process_cq() instead
drivers/scsi/lpfc/lpfc_sli.c:14754: warning: expecting prototype for lpfc_sli4_hba_process_cq(). Prototype was for lpfc_sli4_dly_hba_process_cq() instead
drivers/scsi/lpfc/lpfc_sli.c:17230: warning: expecting prototype for lpfc_sli4_free_xri(). Prototype was for __lpfc_sli4_free_xri() instead
drivers/scsi/lpfc/lpfc_sli.c:18950: warning: expecting prototype for lpfc_sli4_free_rpi(). Prototype was for __lpfc_sli4_free_rpi() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-18-lee.jones@linaro.org
Cc: James Smart <james.smart@broadcom.com >
Cc: Dick Kennedy <dick.kennedy@broadcom.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: Sumit Semwal <sumit.semwal@linaro.org >
Cc: "Christian König" <christian.koenig@amd.com >
Cc: linux-scsi@vger.kernel.org
Cc: linux-media@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linaro-mm-sig@lists.linaro.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:52 -04:00
Lee Jones
53616df281
scsi: aacraid: Repair formatting issue in aac_handle_sa_aif()'s header
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/aacraid/commsup.c:334: warning: expecting prototype for fib_deallocate(). Prototype was for fib_dealloc() instead
drivers/scsi/aacraid/commsup.c:1961: warning: expecting prototype for aac_handle_sa_aif Handle a message from the firmware(). Prototype was for aac_handle_sa_aif() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-17-lee.jones@linaro.org
Cc: Adaptec OEM Raid Solutions <aacraid@microsemi.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: Sumit Semwal <sumit.semwal@linaro.org >
Cc: "Christian König" <christian.koenig@amd.com >
Cc: "PMC-Sierra, Inc" <aacraid@pmc-sierra.com >
Cc: linux-scsi@vger.kernel.org
Cc: linux-media@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linaro-mm-sig@lists.linaro.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:52 -04:00
Lee Jones
52bb80f18d
scsi: qla2xxx: Replace __qla2x00_marker()'s missing underscores
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/qla2xxx/qla_iocb.c:508: warning: expecting prototype for qla2x00_marker(). Prototype was for __qla2x00_marker() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-16-lee.jones@linaro.org
Cc: Nilesh Javali <njavali@marvell.com >
Cc: GR-QLogic-Storage-Upstream@marvell.com
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: linux-scsi@vger.kernel.org
Acked-by: Nilesh Javali <njavali@marvell.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:52 -04:00
Lee Jones
3528b9abbd
scsi: aic94xx: Fix asd_erase_nv_sector()'s header
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/aic94xx/aic94xx_sds.c:1253: warning: expecting prototype for asd_hwi_erase_nv_sector(). Prototype was for asd_erase_nv_sector() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-15-lee.jones@linaro.org
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: Jason Yan <yanaijie@huawei.com >
Cc: John Garry <john.garry@huawei.com >
Cc: "Gustavo A. R. Silva" <gustavoars@kernel.org >
Cc: Luben Tuikov <luben_tuikov@adaptec.com >
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:51 -04:00
Lee Jones
3978e59b6d
scsi: pm8001: Fix incorrectly named functions in headers
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/pm8001/pm8001_ctl.c:313: warning: expecting prototype for pm8001_ctl_sas_address_show(). Prototype was for pm8001_ctl_host_sas_address_show() instead
drivers/scsi/pm8001/pm8001_ctl.c:530: warning: expecting prototype for pm8001_ctl_aap_log_show(). Prototype was for pm8001_ctl_iop_log_show() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-14-lee.jones@linaro.org
Cc: Jack Wang <jinpu.wang@cloud.ionos.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: linux-scsi@vger.kernel.org
Acked-by: Jack Wang <jinpu.wang@cloud.ionos.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:51 -04:00
Lee Jones
ecef0c9e64
scsi: qla4xxx: Fix formatting issues - missing '-' and '_'
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/qla4xxx/ql4_os.c:631: warning: expecting prototype for qla4xxx_create chap_list(). Prototype was for qla4xxx_create_chap_list() instead
drivers/scsi/qla4xxx/ql4_os.c:9643: warning: expecting prototype for gets called if(). Prototype was for qla4xxx_pci_mmio_enabled() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-13-lee.jones@linaro.org
Cc: Nilesh Javali <njavali@marvell.com >
Cc: Manish Rangankar <mrangankar@marvell.com >
Cc: GR-QLogic-Storage-Upstream@marvell.com
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: linux-scsi@vger.kernel.org
Acked-by: Nilesh Javali <njavali@marvell.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:51 -04:00
Lee Jones
62690c056a
scsi: pm8001: Provide function name 'pm8001_I_T_nexus_reset()' in header
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/pm8001/pm8001_sas.c:989: warning: expecting prototype for and hard reset for(). Prototype was for pm8001_I_T_nexus_reset() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-12-lee.jones@linaro.org
Cc: Jack Wang <jinpu.wang@cloud.ionos.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: linux-scsi@vger.kernel.org
Acked-by: Jack Wang <jinpu.wang@cloud.ionos.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:51 -04:00
Lee Jones
52ffc08d94
scsi: bnx2fc: Fix typo in bnx2fc_indicate_kcqe()
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/bnx2fc/bnx2fc_hwi.c:1344: warning: expecting prototype for bnx2fc_indicae_kcqe(). Prototype was for bnx2fc_indicate_kcqe() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-11-lee.jones@linaro.org
Cc: Saurav Kashyap <skashyap@marvell.com >
Cc: Javed Hasan <jhasan@marvell.com >
Cc: GR-QLogic-Storage-Upstream@marvell.com
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: Prakash Gollapudi <bprakash@broadcom.com >
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:51 -04:00
Lee Jones
4c4b8cf5a1
scsi: aacraid: Fix incorrect spelling of aac_send_raw_srb()
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/aacraid/commctrl.c:480: warning: expecting prototype for aac_send_raw_scb(). Prototype was for aac_send_raw_srb() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-10-lee.jones@linaro.org
Cc: Adaptec OEM Raid Solutions <aacraid@microsemi.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: "PMC-Sierra, Inc" <aacraid@pmc-sierra.com >
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:51 -04:00
Lee Jones
bd1050e138
scsi: pm8001: Provide function name and fix a misspelling
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/pm8001/pm8001_init.c:192: warning: expecting prototype for tasklet for 64 msi(). Prototype was for pm8001_tasklet() instead
drivers/scsi/pm8001/pm8001_init.c:872: warning: expecting prototype for pm8001_set_phy_settings_ven_117c_12Gb(). Prototype was for pm8001_set_phy_settings_ven_117c_12G() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-9-lee.jones@linaro.org
Cc: Jack Wang <jinpu.wang@cloud.ionos.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: Kumar Santhanam <AnandKumar.Santhanam@pmcs.com >
Cc: Sangeetha Gnanasekaran <Sangeetha.Gnanasekaran@pmcs.com >
Cc: Nikith Ganigarakoppal <Nikith.Ganigarakoppal@pmcs.com >
Cc: linux-scsi@vger.kernel.org
Acked-by: Jack Wang <jinpu.wang@cloud.ionos.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:51 -04:00
Lee Jones
a80218c7c5
scsi: aacraid: Fix a few incorrectly named functions
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/aacraid/aachba.c:796: warning: expecting prototype for aac_probe_container(). Prototype was for aac_probe_container_callback1() instead
drivers/scsi/aacraid/aachba.c:850: warning: expecting prototype for InqStrCopy(). Prototype was for inqstrcpy() instead
drivers/scsi/aacraid/aachba.c:1814: warning: expecting prototype for Process topology change(). Prototype was for aac_get_safw_ciss_luns() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-8-lee.jones@linaro.org
Cc: Adaptec OEM Raid Solutions <aacraid@microsemi.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: "PMC-Sierra, Inc" <aacraid@pmc-sierra.com >
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:50 -04:00
Lee Jones
577c65b07a
scsi: aic94xx: Fix a couple of misnamed function names
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/aic94xx/aic94xx_hwi.c:910: warning: expecting prototype for ads_rbi_exsi_isr(). Prototype was for asd_rbi_exsi_isr() instead
drivers/scsi/aic94xx/aic94xx_hwi.c:1156: warning: expecting prototype for asd_start_timers(). Prototype was for asd_start_scb_timers() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-7-lee.jones@linaro.org
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: Lee Jones <lee.jones@linaro.org >
Cc: Luben Tuikov <luben_tuikov@adaptec.com >
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:50 -04:00
Lee Jones
dc17357509
scsi: fcoe: Fix a couple of incorrectly named functions
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/fcoe/fcoe_ctlr.c:1314: warning: expecting prototype for fcoe_ctlr_recv_els(). Prototype was for fcoe_ctlr_recv_clr_vlink() instead
drivers/scsi/fcoe/fcoe_ctlr.c:2963: warning: expecting prototype for fcoe_ctlr_vlan_disk_reply(). Prototype was for fcoe_ctlr_vlan_disc_reply() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-6-lee.jones@linaro.org
Cc: Hannes Reinecke <hare@suse.de >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:50 -04:00
Lee Jones
d4f5ae2159
scsi: megaraid_mbox: Fix function name megaraid_queue_command_lck() in description
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/megaraid/megaraid_mbox.c:1438: warning: expecting prototype for megaraid_queue_command(). Prototype was for megaraid_queue_command_lck() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-5-lee.jones@linaro.org
Cc: Kashyap Desai <kashyap.desai@broadcom.com >
Cc: Sumit Saxena <sumit.saxena@broadcom.com >
Cc: Shivasharan S <shivasharan.srikanteshwara@broadcom.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: Atul Mukker <Atul.Mukker@lsi.com >
Cc: Sreenivas Bagalkote <Sreenivas.Bagalkote@lsi.com >
Cc: Manoj Jose <Manoj.Jose@lsi.com >
Cc: megaraidlinux@lsi.com
Cc: megaraidlinux.pdl@broadcom.com
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:50 -04:00
Lee Jones
276f9aa253
scsi: fcoe: Fix function name fcoe_set_vport_symbolic_name() in description
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/fcoe/fcoe.c:2782: warning: expecting prototype for fcoe_vport_set_symbolic_name(). Prototype was for fcoe_set_vport_symbolic_name() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-4-lee.jones@linaro.org
Cc: Hannes Reinecke <hare@suse.de >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:50 -04:00
Lee Jones
616f6d8d9e
scsi: megaraid_sas: Fix a bunch of misnamed functions in their headers
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/megaraid/megaraid_sas_base.c:483: warning: expecting prototype for megasas_clear_interrupt_xscale(). Prototype was for megasas_clear_intr_xscale() instead
drivers/scsi/megaraid/megaraid_sas_base.c:666: warning: expecting prototype for megasas_clear_interrupt_ppc(). Prototype was for megasas_clear_intr_ppc() instead
drivers/scsi/megaraid/megaraid_sas_base.c:795: warning: expecting prototype for megasas_clear_interrupt_skinny(). Prototype was for megasas_clear_intr_skinny() instead
drivers/scsi/megaraid/megaraid_sas_base.c:943: warning: expecting prototype for megasas_clear_interrupt_gen2(). Prototype was for megasas_clear_intr_gen2() instead
drivers/scsi/megaraid/megaraid_sas_base.c:4902: warning: expecting prototype for opcode(). Prototype was for megasas_host_device_list_query() instead
drivers/scsi/megaraid/megaraid_sas_base.c:5173: warning: expecting prototype for megasas_get_controller_info(). Prototype was for megasas_get_ctrl_info() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-3-lee.jones@linaro.org
Cc: Kashyap Desai <kashyap.desai@broadcom.com >
Cc: Sumit Saxena <sumit.saxena@broadcom.com >
Cc: Shivasharan S <shivasharan.srikanteshwara@broadcom.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: megaraidlinux.pdl@broadcom.com
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:50 -04:00
Lee Jones
bbb2c03745
scsi: megaraid_mm: Fix incorrect function name in header
...
Fixes the following W=1 kernel build warning(s):
drivers/scsi/megaraid/megaraid_mm.c:505: warning: expecting prototype for mraid_mm_attch_buf(). Prototype was for mraid_mm_attach_buf() instead
Link: https://lore.kernel.org/r/20210303144631.3175331-2-lee.jones@linaro.org
Cc: Kashyap Desai <kashyap.desai@broadcom.com >
Cc: Sumit Saxena <sumit.saxena@broadcom.com >
Cc: Shivasharan S <shivasharan.srikanteshwara@broadcom.com >
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com >
Cc: "Martin K. Petersen" <martin.petersen@oracle.com >
Cc: megaraidlinux.pdl@broadcom.com
Cc: linux-scsi@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-15 22:14:49 -04:00
Chaitanya Kulkarni
4524a0b159
scsi: target: core: file: Don't duplicate memset(0xff)
...
The function fd_do_prot_fill() is called from two callers
fd_do_prot_unmap() and fd_format_prot(). Both callers initialize the passed
buffer to 0xff with memset().
Move the memset() call to fd_do_prot_fill() to avoid duplication.
Link: https://lore.kernel.org/r/20210228055645.22253-24-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:18 -05:00
Chaitanya Kulkarni
4db6dfe62c
scsi: target: core: pr: Initialize arrays at declaration time
...
Avoids calling memset().
Link: https://lore.kernel.org/r/20210228055645.22253-21-chaitanya.kulkarni@wdc.com
Link: https://lore.kernel.org/r/20210228055645.22253-22-chaitanya.kulkarni@wdc.com
Link: https://lore.kernel.org/r/20210228055645.22253-23-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:18 -05:00
Chaitanya Kulkarni
2d4e2daf4f
scsi: target: configfs: Initialize arrays at declaration time
...
Avoids calling memset().
Link: https://lore.kernel.org/r/20210228055645.22253-14-chaitanya.kulkarni@wdc.com
Link: https://lore.kernel.org/r/20210228055645.22253-15-chaitanya.kulkarni@wdc.com
Link: https://lore.kernel.org/r/20210228055645.22253-16-chaitanya.kulkarni@wdc.com
Link: https://lore.kernel.org/r/20210228055645.22253-17-chaitanya.kulkarni@wdc.com
Link: https://lore.kernel.org/r/20210228055645.22253-18-chaitanya.kulkarni@wdc.com
Link: https://lore.kernel.org/r/20210228055645.22253-19-chaitanya.kulkarni@wdc.com
Link: https://lore.kernel.org/r/20210228055645.22253-20-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:18 -05:00
Chaitanya Kulkarni
c22659fbb9
scsi: target: iscsi: Initialize arrays at declaration time
...
Avoids calling memset().
Link: https://lore.kernel.org/r/20210228055645.22253-13-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:18 -05:00
Chaitanya Kulkarni
c4d81e7c53
scsi: target: iscsi: Remove unused macro PRINT_BUF
...
Remove unused macro to fix the following compilation warning:
drivers/target//iscsi/iscsi_target_util.c:31: warning: macro "PRINT_BUFF" is not used [-Wunused-macros]
#define PRINT_BUFF(buff, len) \
Link: https://lore.kernel.org/r/20210228055645.22253-12-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:18 -05:00
Chaitanya Kulkarni
91ce84a3d7
scsi: target: iscsi: Remove unused macro TEXT_LEN
...
Remove unused macro to fix the following compilation warning:
drivers/target//iscsi/iscsi_target_nego.c:31: warning: macro "TEXT_LEN" is not used [-Wunused-macros]
#define TEXT_LEN 4096
Link: https://lore.kernel.org/r/20210228055645.22253-11-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:18 -05:00
Chaitanya Kulkarni
fdc1339a42
scsi: target: iscsi: Remove unused macro ISCSI_INST_LAST_FAILURE_TYPE
...
Remove unused macro to fix the following compilation warning:
drivers/target//iscsi/iscsi_target_stat.c:31: warning: macro "ISCSI_INST_LAST_FAILURE_TYPE" is not used [-Wunused-macros]
#define ISCSI_INST_LAST_FAILURE_TYPE 0
Link: https://lore.kernel.org/r/20210228055645.22253-10-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:18 -05:00
Chaitanya Kulkarni
c3a27351d6
scsi: target: core: Remove unused macros NONE and ISPRINT
...
Remove the unuseds macro to fix the following compilation warnings:
drivers/target//target_core_stat.c:34: warning: macro "NONE" is not used [-Wunused-macros]
#define NONE "None"
drivers/target//iscsi/iscsi_target_stat.c:36: warning: macro "ISPRINT" is not used [-Wunused-macros]
#define ISPRINT(a) ((a >= ' ') && (a <= '~'))
Link: https://lore.kernel.org/r/20210228055645.22253-8-chaitanya.kulkarni@wdc.com
Link: https://lore.kernel.org/r/20210228055645.22253-9-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:18 -05:00
Chaitanya Kulkarni
5cfb5b0258
scsi: target: core: Get rid of warning in compare_and_write_do_cmp()
...
Rename function local variable i to sg_cnt so we can get rid of the
shadow variable compilation warning:
unsigned int i;
^
int i;
^
Link: https://lore.kernel.org/r/20210228055645.22253-6-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:18 -05:00
Chaitanya Kulkarni
2c958a8c1f
scsi: target: pscsi: Remove unused macro ISPRINT
...
Remove unused macro to fix the following compilation warning:
drivers/target//target_core_pscsi.c:37: warning: macro "ISPRINT" is not used [-Wunused-macros]
#define ISPRINT(a) ((a >= ' ') && (a <= '~'))
Link: https://lore.kernel.org/r/20210228055645.22253-7-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
Chaitanya Kulkarni
fd48c056a3
scsi: target: pscsi: Fix warning in pscsi_complete_cmd()
...
This fixes a compilation warning in pscsi_complete_cmd():
drivers/target/target_core_pscsi.c: In function ‘pscsi_complete_cmd’:
drivers/target/target_core_pscsi.c:624:5: warning: suggest braces around empty body in an ‘if’ statement [-Wempty-body]
; /* XXX: TCM_LOGICAL_UNIT_COMMUNICATION_FAILURE */
Link: https://lore.kernel.org/r/20210228055645.22253-5-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
Chaitanya Kulkarni
c151eddbef
scsi: target: iblock: Fix type of logs_per_phys
...
According to ilog2() it expects 32/64 bit unsigned value.
" 147 * ilog2 - log base 2 of 32-bit or a 64-bit unsigned value"
Replace type of logs_per_phys from int to unsigned int.
Link: https://lore.kernel.org/r/20210228055645.22253-4-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
Chaitanya Kulkarni
a2c6c6a3b1
scsi: target: iblock: Trim line longer than 80 characters
...
Trim the line that is longer than 80 characters, which is inconsistent with
the rest of the code.
Link: https://lore.kernel.org/r/20210228055645.22253-3-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
Chaitanya Kulkarni
bc9e0e366f
scsi: target: iblock: Remove an extra argument
...
The two arguments to the functions op and opf which are REQ_OP_XXX and
REQ_XXX flags belong to the two different namespaces, i.e. they can be
combined safely given that REQ_OP_XXX takes 8 bits of the flags and rest is
available to REQ_XXX flags.
Replace op and op_flag arguments with opf.
Link: https://lore.kernel.org/r/20210228055645.22253-2-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com >
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
Bodo Stroesser
1080782f13
scsi: target: tcmu: Use GFP_NOIO while handling cmds or holding cmdr_lock
...
Especially when using tcmu with tcm_loop, memory allocations with
GFP_KERNEL for a LUN can cause write back to the same LUN.
So we have to use GFP_NOIO when allocation is done while handling commands
or while holding cmdr_lock.
Link: https://lore.kernel.org/r/20210305190009.32242-1-bostroesser@gmail.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Signed-off-by: Bodo Stroesser <bostroesser@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
Bodo Stroesser
f7c89771d0
scsi: target: tcmu: Replace radix_tree with XArray
...
An attempt from Matthew Wilcox to replace radix-tree usage by XArray in
tcmu more than 1 year ago unfortunately got lost.
I rebased that work on latest tcmu and tested it.
Link: https://lore.kernel.org/r/20210224185335.13844-3-bostroesser@gmail.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Signed-off-by: Bodo Stroesser <bostroesser@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
Bodo Stroesser
d3cbb743c3
scsi: target: tcmu: Replace IDR by XArray
...
An attempt from Matthew Wilcox to replace IDR usage by XArray in tcmu more
than 1 year ago unfortunately got lost.
I rebased that work on latest tcmu and tested it.
Link: https://lore.kernel.org/r/20210224185335.13844-2-bostroesser@gmail.com
Reviewed-by: Mike Christie <michael.christie@oracle.com >
Signed-off-by: Bodo Stroesser <bostroesser@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
zuoqilin
083d248b2d
scsi: FlashPoint: Fix typo
...
Change 'defualt' to 'default'.
Link: https://lore.kernel.org/r/20210304055120.2221-1-zuoqilin1@163.com
Acked-by: Khalid Aziz <khalid@gonehiking.org >
Signed-off-by: zuoqilin <zuoqilin@yulong.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
Jiapeng Chong
18c2a59a41
scsi: ibmvfc: Switch to using kobj_to_dev()
...
Fix the following coccicheck warnings:
./drivers/scsi/ibmvscsi/ibmvfc.c:3483:60-61: WARNING opportunity for
kobj_to_dev().
Link: https://lore.kernel.org/r/1614850124-54111-1-git-send-email-jiapeng.chong@linux.alibaba.com
Reported-by: Abaci Robot <abaci@linux.alibaba.com >
Acked-by: Tyrel Datwyler <tyreld@linux.ibm.com >
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:17 -05:00
Zhang Yunkai
b4388e3db5
scsi: ufs: Remove duplicate include in ufshcd
...
'blkdev.h' included in 'ufshcd.c' is included twice.
Link: https://lore.kernel.org/r/20210306114706.217873-1-zhang.yunkai@zte.com.cn
Reviewed-by: Bean Huo <beanhuo@micron.com >
Signed-off-by: Zhang Yunkai <zhang.yunkai@zte.com.cn >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:16 -05:00
Bhaskar Chowdhury
014ace23a5
scsi: qla4xxx: Fix a typo
...
s/circuting/circuiting/
Link: https://lore.kernel.org/r/20210301131736.14236-1-unixbhaskar@gmail.com
Acked-by: Randy Dunlap <rdunlap@infradead.org >
Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:16 -05:00
Igor Pylypiv
9ec3d4c105
scsi: pm80xx: Replace magic numbers with device state defines
...
This improves the code readability.
Link: https://lore.kernel.org/r/20210305060908.2476850-1-ipylypiv@google.com
Acked-by: Jack Wang <jinpu.wang@cloud.ionos.com >
Signed-off-by: Igor Pylypiv <ipylypiv@google.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:47:16 -05:00
Igor Pylypiv
5b1be37f77
scsi: pm80xx: Remove list entry from pm8001_ccb_info
...
List entry is not used.
Link: https://lore.kernel.org/r/20210301181847.2893199-1-ipylypiv@google.com
Acked-by: Jack Wang <jinpu.wang@cloud.ionos.com >
Signed-off-by: Igor Pylypiv <ipylypiv@google.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-09 23:22:03 -05:00
Melanie Plageman (Microsoft)
a81a38cc6d
scsi: storvsc: Parameterize number hardware queues
...
Add ability to set the number of hardware queues with new module parameter,
storvsc_max_hw_queues. The default value remains the number of CPUs. This
functionality is useful in some environments (e.g. Microsoft Azure) where
decreasing the number of hardware queues has been shown to improve
performance.
Link: https://lore.kernel.org/r/20210224232948.4651-1-melanieplageman@gmail.com
Reviewed-by: Michael Kelley <mikelley@microsoft.com >
Signed-off-by: Melanie Plageman (Microsoft) <melanieplageman@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-04 17:37:06 -05:00
James Smart
67073c69c8
scsi: lpfc: Update copyrights for 12.8.0.7 and 12.8.0.8 changes
...
For the files modified in 2021 via the 12.8.0.7 and 12.8.0.8 patch sets,
update the copyright for 2021.
Link: https://lore.kernel.org/r/20210301171821.3427-23-jsmart2021@gmail.com
Co-developed-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: James Smart <jsmart2021@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-04 17:37:06 -05:00
James Smart
ef53d5e1b8
scsi: lpfc: Update lpfc version to 12.8.0.8
...
Update lpfc version to 12.8.0.8.
Link: https://lore.kernel.org/r/20210301171821.3427-22-jsmart2021@gmail.com
Co-developed-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: James Smart <jsmart2021@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-04 17:37:05 -05:00
James Smart
69b8eff25c
scsi: lpfc: Correct function header comments related to ndlp reference counting
...
Code inspection revealed stale comments in function headers for functions
that call lpfc_prep_els_iocb(). Changes in ndlp reference counting were not
reflected in function headers.
Update the stale comments in function headers to more accurately indicate
ndlp reference counting.
Link: https://lore.kernel.org/r/20210301171821.3427-21-jsmart2021@gmail.com
Co-developed-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: James Smart <jsmart2021@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-04 17:37:05 -05:00
James Smart
cdf811606b
scsi: lpfc: Reduce LOG_TRACE_EVENT logging for vports
...
Lots of discovery messages are flooding the console log when testing NPIV.
Informational message for vports should have LOG_VPORT associated with it
as opposed to LOG_TRACE_EVENT.
Link: https://lore.kernel.org/r/20210301171821.3427-20-jsmart2021@gmail.com
Co-developed-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: James Smart <jsmart2021@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-04 17:37:05 -05:00
James Smart
a94a40eb64
scsi: lpfc: Change wording of invalid pci reset log message
...
Message 8347 Invalid device found log message is logged when an LPe12000
adapter is installed. The log message is supposed to indicate an
unsupported pci reset adapter rather than an invalid device.
Change the wording to: Incapable PCI reset device.
Link: https://lore.kernel.org/r/20210301171821.3427-19-jsmart2021@gmail.com
Co-developed-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: James Smart <jsmart2021@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-04 17:37:05 -05:00
James Smart
9628aace0d
scsi: lpfc: Fix crash caused by switch reboot
...
Driver is causing a crash in __lpfc_sli_release_iocbq_s4() when it
dereferences the els_wq which is NULL.
Validate the pring for the els_wq before dereferencing. Reorg the code to
move the pring assignment closer to where it is actually used.
Link: https://lore.kernel.org/r/20210301171821.3427-18-jsmart2021@gmail.com
Co-developed-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: James Smart <jsmart2021@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-04 17:37:05 -05:00
James Smart
debbc1e2b9
scsi: lpfc: Fix pt2pt state transition causing rmmod hang
...
On a setup with a dual port HBA and both ports direct connected, an rmmod
hangs momentarily when we log an Illegal State Transition. Once it resumes,
a nodelist not empty logic is hit, which forces rmmod to cleanup and exit.
We're missing a state transition case in the discovery engine.
Fix by adding a case for a DEVICE_RM event while in the unmapped state to
avoid illegal state transition log message.
Link: https://lore.kernel.org/r/20210301171821.3427-17-jsmart2021@gmail.com
Co-developed-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: James Smart <jsmart2021@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-04 17:37:05 -05:00
James Smart
bb6fd33e3a
scsi: lpfc: Fix nodeinfo debugfs output
...
The debugfs nodeinfo output gets jumbled when no rpri or a defer entry is
displayed. The misalignment makes it difficult to read.
Change the format to consistently print out a 4 character rpi, and turn
defer into a suffix.
Link: https://lore.kernel.org/r/20210301171821.3427-16-jsmart2021@gmail.com
Co-developed-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com >
Signed-off-by: James Smart <jsmart2021@gmail.com >
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com >
2021-03-04 17:37:05 -05:00