Commit 00598b05 authored by Kevin Barnett's avatar Kevin Barnett Committed by Martin K. Petersen

scsi: smartpqi: Expose SAS address for SATA drives

Remove UNIQUE_WWID_IN_REPORT_PHYS_LUN PQI feature.

This feature was originally added to solve a problem with NVMe drives, but
this problem has since been solved a different way, so this PQI feature is
no longer required for any type of drive.

The kernel was not creating symbolic links in sysfs between SATA drives and
their enclosure.

The driver was enabling the UNIQUE_WWID_IN_REPORT_PHYS_LUN PQI feature,
which causes the FW to return a WWID for SATA drives that is derived from a
unique ID read from the SATA drive itself. The driver was exposing this
WWID as the drive's SAS address. However, because this SAS address does not
match the SAS address returned by an enclosure's SES Page 0xA data, the
Linux kernel was not able to match a SATA drive with its enclosure.

Link: https://lore.kernel.org/r/164375213346.440833.12379222470149882747.stgit@brunhilda.pdev.netReviewed-by: default avatarMike McGowen <mike.mcgowen@microchip.com>
Reviewed-by: default avatarScott Benesh <scott.benesh@microchip.com>
Reviewed-by: default avatarScott Teel <scott.teel@microchip.com>
Signed-off-by: default avatarKevin Barnett <kevin.barnett@microchip.com>
Signed-off-by: default avatarDon Brace <don.brace@microchip.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 5d8fbce0
...@@ -1141,7 +1141,6 @@ struct pqi_scsi_dev { ...@@ -1141,7 +1141,6 @@ struct pqi_scsi_dev {
struct pqi_stream_data stream_data[NUM_STREAMS_PER_LUN]; struct pqi_stream_data stream_data[NUM_STREAMS_PER_LUN];
atomic_t scsi_cmds_outstanding; atomic_t scsi_cmds_outstanding;
atomic_t raid_bypass_cnt; atomic_t raid_bypass_cnt;
u8 page_83_identifier[16];
}; };
/* VPD inquiry pages */ /* VPD inquiry pages */
...@@ -1331,7 +1330,6 @@ struct pqi_ctrl_info { ...@@ -1331,7 +1330,6 @@ struct pqi_ctrl_info {
u8 soft_reset_handshake_supported : 1; u8 soft_reset_handshake_supported : 1;
u8 raid_iu_timeout_supported : 1; u8 raid_iu_timeout_supported : 1;
u8 tmf_iu_timeout_supported : 1; u8 tmf_iu_timeout_supported : 1;
u8 unique_wwid_in_report_phys_lun_supported : 1;
u8 firmware_triage_supported : 1; u8 firmware_triage_supported : 1;
u8 rpl_extended_format_4_5_supported : 1; u8 rpl_extended_format_4_5_supported : 1;
u8 enable_r1_writes : 1; u8 enable_r1_writes : 1;
......
...@@ -1588,9 +1588,6 @@ static int pqi_get_physical_device_info(struct pqi_ctrl_info *ctrl_info, ...@@ -1588,9 +1588,6 @@ static int pqi_get_physical_device_info(struct pqi_ctrl_info *ctrl_info,
sizeof(device->phys_connector)); sizeof(device->phys_connector));
device->bay = id_phys->phys_bay_in_box; device->bay = id_phys->phys_bay_in_box;
memcpy(&device->page_83_identifier, &id_phys->page_83_identifier,
sizeof(device->page_83_identifier));
if ((id_phys->even_more_flags & PQI_DEVICE_PHY_MAP_SUPPORTED) && if ((id_phys->even_more_flags & PQI_DEVICE_PHY_MAP_SUPPORTED) &&
id_phys->phy_count) id_phys->phy_count)
device->phy_id = device->phy_id =
...@@ -2281,18 +2278,6 @@ static inline void pqi_mask_device(u8 *scsi3addr) ...@@ -2281,18 +2278,6 @@ static inline void pqi_mask_device(u8 *scsi3addr)
scsi3addr[3] |= 0xc0; scsi3addr[3] |= 0xc0;
} }
static inline bool pqi_is_device_with_sas_address(struct pqi_scsi_dev *device)
{
switch (device->device_type) {
case SA_DEVICE_TYPE_SAS:
case SA_DEVICE_TYPE_EXPANDER_SMP:
case SA_DEVICE_TYPE_SES:
return true;
}
return false;
}
static inline bool pqi_is_multipath_device(struct pqi_scsi_dev *device) static inline bool pqi_is_multipath_device(struct pqi_scsi_dev *device)
{ {
if (pqi_is_logical_device(device)) if (pqi_is_logical_device(device))
...@@ -2306,17 +2291,6 @@ static inline bool pqi_expose_device(struct pqi_scsi_dev *device) ...@@ -2306,17 +2291,6 @@ static inline bool pqi_expose_device(struct pqi_scsi_dev *device)
return !device->is_physical_device || !pqi_skip_device(device->scsi3addr); return !device->is_physical_device || !pqi_skip_device(device->scsi3addr);
} }
static inline void pqi_set_physical_device_wwid(struct pqi_ctrl_info *ctrl_info,
struct pqi_scsi_dev *device, struct report_phys_lun_16byte_wwid *phys_lun)
{
if (ctrl_info->unique_wwid_in_report_phys_lun_supported ||
ctrl_info->rpl_extended_format_4_5_supported ||
pqi_is_device_with_sas_address(device))
memcpy(device->wwid, phys_lun->wwid, sizeof(device->wwid));
else
memcpy(&device->wwid[8], device->page_83_identifier, 8);
}
static int pqi_update_scsi_devices(struct pqi_ctrl_info *ctrl_info) static int pqi_update_scsi_devices(struct pqi_ctrl_info *ctrl_info)
{ {
int i; int i;
...@@ -2484,7 +2458,7 @@ static int pqi_update_scsi_devices(struct pqi_ctrl_info *ctrl_info) ...@@ -2484,7 +2458,7 @@ static int pqi_update_scsi_devices(struct pqi_ctrl_info *ctrl_info)
pqi_assign_bus_target_lun(device); pqi_assign_bus_target_lun(device);
if (device->is_physical_device) { if (device->is_physical_device) {
pqi_set_physical_device_wwid(ctrl_info, device, phys_lun); memcpy(device->wwid, phys_lun->wwid, sizeof(device->wwid));
if ((phys_lun->device_flags & if ((phys_lun->device_flags &
CISS_REPORT_PHYS_DEV_FLAG_AIO_ENABLED) && CISS_REPORT_PHYS_DEV_FLAG_AIO_ENABLED) &&
phys_lun->aio_handle) { phys_lun->aio_handle) {
...@@ -2497,7 +2471,6 @@ static int pqi_update_scsi_devices(struct pqi_ctrl_info *ctrl_info) ...@@ -2497,7 +2471,6 @@ static int pqi_update_scsi_devices(struct pqi_ctrl_info *ctrl_info)
sizeof(device->volume_id)); sizeof(device->volume_id));
} }
if (pqi_is_device_with_sas_address(device))
device->sas_address = get_unaligned_be64(&device->wwid[8]); device->sas_address = get_unaligned_be64(&device->wwid[8]);
new_device_list[num_valid_devices++] = device; new_device_list[num_valid_devices++] = device;
...@@ -7087,7 +7060,7 @@ static ssize_t pqi_sas_address_show(struct device *dev, ...@@ -7087,7 +7060,7 @@ static ssize_t pqi_sas_address_show(struct device *dev,
spin_lock_irqsave(&ctrl_info->scsi_device_list_lock, flags); spin_lock_irqsave(&ctrl_info->scsi_device_list_lock, flags);
device = sdev->hostdata; device = sdev->hostdata;
if (!device || !pqi_is_device_with_sas_address(device)) { if (!device) {
spin_unlock_irqrestore(&ctrl_info->scsi_device_list_lock, flags); spin_unlock_irqrestore(&ctrl_info->scsi_device_list_lock, flags);
return -ENODEV; return -ENODEV;
} }
...@@ -7643,10 +7616,6 @@ static void pqi_ctrl_update_feature_flags(struct pqi_ctrl_info *ctrl_info, ...@@ -7643,10 +7616,6 @@ static void pqi_ctrl_update_feature_flags(struct pqi_ctrl_info *ctrl_info,
case PQI_FIRMWARE_FEATURE_TMF_IU_TIMEOUT: case PQI_FIRMWARE_FEATURE_TMF_IU_TIMEOUT:
ctrl_info->tmf_iu_timeout_supported = firmware_feature->enabled; ctrl_info->tmf_iu_timeout_supported = firmware_feature->enabled;
break; break;
case PQI_FIRMWARE_FEATURE_UNIQUE_WWID_IN_REPORT_PHYS_LUN:
ctrl_info->unique_wwid_in_report_phys_lun_supported =
firmware_feature->enabled;
break;
case PQI_FIRMWARE_FEATURE_FW_TRIAGE: case PQI_FIRMWARE_FEATURE_FW_TRIAGE:
ctrl_info->firmware_triage_supported = firmware_feature->enabled; ctrl_info->firmware_triage_supported = firmware_feature->enabled;
pqi_save_fw_triage_setting(ctrl_info, firmware_feature->enabled); pqi_save_fw_triage_setting(ctrl_info, firmware_feature->enabled);
...@@ -7744,11 +7713,6 @@ static struct pqi_firmware_feature pqi_firmware_features[] = { ...@@ -7744,11 +7713,6 @@ static struct pqi_firmware_feature pqi_firmware_features[] = {
.feature_bit = PQI_FIRMWARE_FEATURE_RAID_BYPASS_ON_ENCRYPTED_NVME, .feature_bit = PQI_FIRMWARE_FEATURE_RAID_BYPASS_ON_ENCRYPTED_NVME,
.feature_status = pqi_firmware_feature_status, .feature_status = pqi_firmware_feature_status,
}, },
{
.feature_name = "Unique WWID in Report Physical LUN",
.feature_bit = PQI_FIRMWARE_FEATURE_UNIQUE_WWID_IN_REPORT_PHYS_LUN,
.feature_status = pqi_ctrl_update_feature_flags,
},
{ {
.feature_name = "Firmware Triage", .feature_name = "Firmware Triage",
.feature_bit = PQI_FIRMWARE_FEATURE_FW_TRIAGE, .feature_bit = PQI_FIRMWARE_FEATURE_FW_TRIAGE,
...@@ -7858,7 +7822,6 @@ static void pqi_ctrl_reset_config(struct pqi_ctrl_info *ctrl_info) ...@@ -7858,7 +7822,6 @@ static void pqi_ctrl_reset_config(struct pqi_ctrl_info *ctrl_info)
ctrl_info->enable_r6_writes = false; ctrl_info->enable_r6_writes = false;
ctrl_info->raid_iu_timeout_supported = false; ctrl_info->raid_iu_timeout_supported = false;
ctrl_info->tmf_iu_timeout_supported = false; ctrl_info->tmf_iu_timeout_supported = false;
ctrl_info->unique_wwid_in_report_phys_lun_supported = false;
ctrl_info->firmware_triage_supported = false; ctrl_info->firmware_triage_supported = false;
ctrl_info->rpl_extended_format_4_5_supported = false; ctrl_info->rpl_extended_format_4_5_supported = false;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment