Skip to content

Commit 06b6fa3

Browse files
jsmart-ghmartinkpetersen
authored andcommitted
scsi: lpfc: Remove lpfc_enable_pbde as module parameter
Enablement of the PBDE optimization brought out some incompatible behaviors under error scenarios. Best to disable and remove the PBDE optimization. Signed-off-by: Dick Kennedy <[email protected]> Signed-off-by: James Smart <[email protected]> Signed-off-by: Martin K. Petersen <[email protected]>
1 parent 24bc311 commit 06b6fa3

File tree

1 file changed

+2
-10
lines changed

1 file changed

+2
-10
lines changed

drivers/scsi/lpfc/lpfc_attr.c

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5387,14 +5387,6 @@ LPFC_BBCR_ATTR_RW(enable_bbcr, 1, 0, 1, "Enable BBC Recovery");
53875387
*/
53885388
LPFC_ATTR_RW(enable_dpp, 1, 0, 1, "Enable Direct Packet Push");
53895389

5390-
/*
5391-
* lpfc_enable_pbde: Enable PBDE on PRISM - G7
5392-
* 0 = PBDE on G7 disabled
5393-
* 1 = PBDE on G7 enabled (default)
5394-
* Value range is [0,1]. Default value is 1
5395-
*/
5396-
LPFC_ATTR_R(enable_pbde, 1, 0, 1, "Enable PBDE support on PRISM");
5397-
53985390
struct device_attribute *lpfc_hba_attrs[] = {
53995391
&dev_attr_nvme_info,
54005392
&dev_attr_bg_info,
@@ -5506,7 +5498,6 @@ struct device_attribute *lpfc_hba_attrs[] = {
55065498
&dev_attr_lpfc_enable_mds_diags,
55075499
&dev_attr_lpfc_enable_bbcr,
55085500
&dev_attr_lpfc_enable_dpp,
5509-
&dev_attr_lpfc_enable_pbde,
55105501
NULL,
55115502
};
55125503

@@ -6541,7 +6532,6 @@ lpfc_get_cfgparam(struct lpfc_hba *phba)
65416532
lpfc_nvme_io_channel_init(phba, lpfc_nvme_io_channel);
65426533
lpfc_enable_bbcr_init(phba, lpfc_enable_bbcr);
65436534
lpfc_enable_dpp_init(phba, lpfc_enable_dpp);
6544-
lpfc_enable_pbde_init(phba, lpfc_enable_pbde);
65456535

65466536
if (phba->sli_rev != LPFC_SLI_REV4) {
65476537
/* NVME only supported on SLI4 */
@@ -6558,6 +6548,8 @@ lpfc_get_cfgparam(struct lpfc_hba *phba)
65586548
phba->cfg_auto_imax = 0;
65596549
phba->initial_imax = phba->cfg_fcp_imax;
65606550

6551+
phba->cfg_enable_pbde = 0;
6552+
65616553
/* A value of 0 means use the number of CPUs found in the system */
65626554
if (phba->cfg_fcp_io_channel == 0)
65636555
phba->cfg_fcp_io_channel = phba->sli4_hba.num_present_cpu;

0 commit comments

Comments
 (0)