github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-05-29 17:43:52 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

scsi: Fix sas_user_scan() to handle wildcard and multi-channel scans

Browse Source 
sas_user_scan() did not fully process wildcard channel scans
(SCAN_WILD_CARD) when a transport-specific user_scan() callback was
present. Only channel 0 would be scanned via user_scan(), while the
remaining channels were skipped, potentially missing devices.

user_scan() invokes updated sas_user_scan() for channel 0, and if
successful, iteratively scans remaining channels (1 to
shost->max_channel) via scsi_scan_host_selected().  This ensures complete
wildcard scanning without affecting transport-specific scanning behavior.

Signed-off-by: Ranjan Kumar <ranjan.kumar@broadcom.com>
Link: https://lore.kernel.org/r/20250624061649.17990-1-ranjan.kumar@broadcom.com
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
Ranjan Kumar 2025-06-24 11:46:49 +05:30 committed by Martin K. Petersen
parent 6e0f6aa44b
commit 37c4e72b06
2 changed files with 50 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/scsi/scsi_scan.c
View File

@ -1900,7 +1900,7 @@ int scsi_scan_host_selected(struct Scsi_Host *shost, unsigned int channel,
return 0; return 0;
} }
EXPORT_SYMBOL(scsi_scan_host_selected);
static void scsi_sysfs_add_devices(struct Scsi_Host *shost) static void scsi_sysfs_add_devices(struct Scsi_Host *shost)
{ {
struct scsi_device *sdev; struct scsi_device *sdev;

62
drivers/scsi/scsi_transport_sas.c
View File

@ -40,6 +40,8 @@
#include <scsi/scsi_transport_sas.h> #include <scsi/scsi_transport_sas.h>
#include "scsi_sas_internal.h" #include "scsi_sas_internal.h"
#include "scsi_priv.h"
struct sas_host_attrs { struct sas_host_attrs {
struct list_head rphy_list; struct list_head rphy_list;
struct mutex lock; struct mutex lock;
@ -1683,6 +1685,22 @@ int scsi_is_sas_rphy(const struct device *dev)
} }
EXPORT_SYMBOL(scsi_is_sas_rphy); EXPORT_SYMBOL(scsi_is_sas_rphy);
static void scan_channel_zero(struct Scsi_Host *shost, uint id, u64 lun)
{
struct sas_host_attrs *sas_host = to_sas_host_attrs(shost);
struct sas_rphy *rphy;
list_for_each_entry(rphy, &sas_host->rphy_list, list) {
if (rphy->identify.device_type != SAS_END_DEVICE ||
rphy->scsi_target_id == -1)
continue;
if (id == SCAN_WILD_CARD || id == rphy->scsi_target_id) {
scsi_scan_target(&rphy->dev, 0, rphy->scsi_target_id,
lun, SCSI_SCAN_MANUAL);
}
}
}
/* /*
* SCSI scan helper * SCSI scan helper
@ -1692,23 +1710,41 @@ static int sas_user_scan(struct Scsi_Host *shost, uint channel,
uint id, u64 lun) uint id, u64 lun)
{ {
struct sas_host_attrs *sas_host = to_sas_host_attrs(shost); struct sas_host_attrs *sas_host = to_sas_host_attrs(shost);
struct sas_rphy *rphy; int res = 0;
int i;
mutex_lock(&sas_host->lock); switch (channel) {
list_for_each_entry(rphy, &sas_host->rphy_list, list) { case 0:
if (rphy->identify.device_type != SAS_END_DEVICE || mutex_lock(&sas_host->lock);
rphy->scsi_target_id == -1) scan_channel_zero(shost, id, lun);
continue; mutex_unlock(&sas_host->lock);
break;
if ((channel == SCAN_WILD_CARD || channel == 0) && case SCAN_WILD_CARD:
(id == SCAN_WILD_CARD || id == rphy->scsi_target_id)) { mutex_lock(&sas_host->lock);
scsi_scan_target(&rphy->dev, 0, rphy->scsi_target_id, scan_channel_zero(shost, id, lun);
lun, SCSI_SCAN_MANUAL); mutex_unlock(&sas_host->lock);
for (i = 1; i <= shost->max_channel; i++) {
res = scsi_scan_host_selected(shost, i, id, lun,
SCSI_SCAN_MANUAL);
if (res)
goto exit_scan;
} }
} break;
mutex_unlock(&sas_host->lock);
return 0; default:
if (channel < shost->max_channel) {
res = scsi_scan_host_selected(shost, channel, id, lun,
SCSI_SCAN_MANUAL);
} else {
res = -EINVAL;
}
break;
}
exit_scan:
return res;
} }