Skip to content

Commit b81b182

Browse files
committed
Merge tag 'scsi-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi
Pull SCSI fixes from James Bottomley: "Two minor fixes in the lpfc driver. One changing the classification of trace messages and the other fixing a build issue when NVME_FC is disabled" * tag 'scsi-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi: scsi: lpfc: Reduce log messages seen after firmware download scsi: lpfc: Remove NVMe support if kernel has NVME_FC disabled
2 parents 080eba7 + 5852ed2 commit b81b182

File tree

4 files changed

+20
-7
lines changed

4 files changed

+20
-7
lines changed

drivers/scsi/lpfc/lpfc.h

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1161,6 +1161,16 @@ struct lpfc_hba {
11611161
uint32_t cfg_hostmem_hgp;
11621162
uint32_t cfg_log_verbose;
11631163
uint32_t cfg_enable_fc4_type;
1164+
#define LPFC_ENABLE_FCP 1
1165+
#define LPFC_ENABLE_NVME 2
1166+
#define LPFC_ENABLE_BOTH 3
1167+
#if (IS_ENABLED(CONFIG_NVME_FC))
1168+
#define LPFC_MAX_ENBL_FC4_TYPE LPFC_ENABLE_BOTH
1169+
#define LPFC_DEF_ENBL_FC4_TYPE LPFC_ENABLE_BOTH
1170+
#else
1171+
#define LPFC_MAX_ENBL_FC4_TYPE LPFC_ENABLE_FCP
1172+
#define LPFC_DEF_ENBL_FC4_TYPE LPFC_ENABLE_FCP
1173+
#endif
11641174
uint32_t cfg_aer_support;
11651175
uint32_t cfg_sriov_nr_virtfn;
11661176
uint32_t cfg_request_firmware_upgrade;
@@ -1182,9 +1192,6 @@ struct lpfc_hba {
11821192
uint32_t cfg_ras_fwlog_func;
11831193
uint32_t cfg_enable_bbcr; /* Enable BB Credit Recovery */
11841194
uint32_t cfg_enable_dpp; /* Enable Direct Packet Push */
1185-
#define LPFC_ENABLE_FCP 1
1186-
#define LPFC_ENABLE_NVME 2
1187-
#define LPFC_ENABLE_BOTH 3
11881195
uint32_t cfg_enable_pbde;
11891196
uint32_t cfg_enable_mi;
11901197
struct nvmet_fc_target_port *targetport;

drivers/scsi/lpfc/lpfc_attr.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3978,8 +3978,8 @@ LPFC_ATTR_R(nvmet_mrq_post,
39783978
* 3 - register both FCP and NVME
39793979
* Supported values are [1,3]. Default value is 3
39803980
*/
3981-
LPFC_ATTR_R(enable_fc4_type, LPFC_ENABLE_BOTH,
3982-
LPFC_ENABLE_FCP, LPFC_ENABLE_BOTH,
3981+
LPFC_ATTR_R(enable_fc4_type, LPFC_DEF_ENBL_FC4_TYPE,
3982+
LPFC_ENABLE_FCP, LPFC_MAX_ENBL_FC4_TYPE,
39833983
"Enable FC4 Protocol support - FCP / NVME");
39843984

39853985
/*

drivers/scsi/lpfc/lpfc_init.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2104,7 +2104,7 @@ lpfc_handle_eratt_s4(struct lpfc_hba *phba)
21042104
}
21052105
if (reg_err1 == SLIPORT_ERR1_REG_ERR_CODE_2 &&
21062106
reg_err2 == SLIPORT_ERR2_REG_FW_RESTART) {
2107-
lpfc_printf_log(phba, KERN_ERR, LOG_TRACE_EVENT,
2107+
lpfc_printf_log(phba, KERN_ERR, LOG_SLI,
21082108
"3143 Port Down: Firmware Update "
21092109
"Detected\n");
21102110
en_rn_msg = false;

drivers/scsi/lpfc/lpfc_sli.c

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13363,6 +13363,7 @@ lpfc_sli4_eratt_read(struct lpfc_hba *phba)
1336313363
uint32_t uerr_sta_hi, uerr_sta_lo;
1336413364
uint32_t if_type, portsmphr;
1336513365
struct lpfc_register portstat_reg;
13366+
u32 logmask;
1336613367

1336713368
/*
1336813369
* For now, use the SLI4 device internal unrecoverable error
@@ -13413,7 +13414,12 @@ lpfc_sli4_eratt_read(struct lpfc_hba *phba)
1341313414
readl(phba->sli4_hba.u.if_type2.ERR1regaddr);
1341413415
phba->work_status[1] =
1341513416
readl(phba->sli4_hba.u.if_type2.ERR2regaddr);
13416-
lpfc_printf_log(phba, KERN_ERR, LOG_TRACE_EVENT,
13417+
logmask = LOG_TRACE_EVENT;
13418+
if (phba->work_status[0] ==
13419+
SLIPORT_ERR1_REG_ERR_CODE_2 &&
13420+
phba->work_status[1] == SLIPORT_ERR2_REG_FW_RESTART)
13421+
logmask = LOG_SLI;
13422+
lpfc_printf_log(phba, KERN_ERR, logmask,
1341713423
"2885 Port Status Event: "
1341813424
"port status reg 0x%x, "
1341913425
"port smphr reg 0x%x, "

0 commit comments

Comments
 (0)