Minki/linux
1
0
Fork 1
mirror of https://github.com/torvalds/linux.git synced 2024-11-24 13:11:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

scsi: hpsa: use bus '3' for legacy HBA devices

Browse Source 
Older controllers use SCSI target id '0' for the first internal disk. As
the controllers are now placed on the same bus as the internal disks
this leads to a clash with the SCSI target id of controller.  This patch
checks the SCSI revision, and moves older controller to bus '3' to be
compatible with older releases and avoid this problem.

[mkp: fixed uninitialized variable]

Fixes: 09371d623c ("hpsa: Change SAS transport devices to bus 0.")
Cc: <stable@vger.kernel.org> # v4.5+
Signed-off-by: Hannes Reinecke <hare@suse.com>
Acked-by: Don Brace <don.brace@microsemi.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
Hannes Reinecke 2016-11-17 12:15:56 +01:00 committed by Martin K. Petersen
parent 208da78e8e
commit 7630b3a599
2 changed files with 13 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
drivers/scsi/hpsa.c
View File

@ -2009,7 +2009,7 @@ static struct hpsa_scsi_dev_t *lookup_hpsa_scsi_dev(struct ctlr_info *h,
static int hpsa_slave_alloc(struct scsi_device *sdev) static int hpsa_slave_alloc(struct scsi_device *sdev)
{ {
struct hpsa_scsi_dev_t *sd; struct hpsa_scsi_dev_t *sd = NULL;
unsigned long flags; unsigned long flags;
struct ctlr_info *h; struct ctlr_info *h;
@ -2026,7 +2026,8 @@ static int hpsa_slave_alloc(struct scsi_device *sdev)
sd->target = sdev_id(sdev); sd->target = sdev_id(sdev);
sd->lun = sdev->lun; sd->lun = sdev->lun;
} }
} else }
if (!sd)
sd = lookup_hpsa_scsi_dev(h, sdev_channel(sdev), sd = lookup_hpsa_scsi_dev(h, sdev_channel(sdev),
sdev_id(sdev), sdev->lun); sdev_id(sdev), sdev->lun);
@ -3840,6 +3841,7 @@ static int hpsa_update_device_info(struct ctlr_info *h,
sizeof(this_device->vendor)); sizeof(this_device->vendor));
memcpy(this_device->model, &inq_buff[16], memcpy(this_device->model, &inq_buff[16],
sizeof(this_device->model)); sizeof(this_device->model));
this_device->rev = inq_buff[2];
memset(this_device->device_id, 0, memset(this_device->device_id, 0,
sizeof(this_device->device_id)); sizeof(this_device->device_id));
if (hpsa_get_device_id(h, scsi3addr, this_device->device_id, 8, if (hpsa_get_device_id(h, scsi3addr, this_device->device_id, 8,
@ -3929,10 +3931,14 @@ static void figure_bus_target_lun(struct ctlr_info *h,
if (!is_logical_dev_addr_mode(lunaddrbytes)) { if (!is_logical_dev_addr_mode(lunaddrbytes)) {
/* physical device, target and lun filled in later */ /* physical device, target and lun filled in later */
if (is_hba_lunid(lunaddrbytes)) if (is_hba_lunid(lunaddrbytes)) {
int bus = HPSA_HBA_BUS;
if (!device->rev)
bus = HPSA_LEGACY_HBA_BUS;
hpsa_set_bus_target_lun(device, hpsa_set_bus_target_lun(device,
HPSA_HBA_BUS, 0, lunid & 0x3fff); bus, 0, lunid & 0x3fff);
else } else
/* defer target, lun assignment for physical devices */ /* defer target, lun assignment for physical devices */
hpsa_set_bus_target_lun(device, hpsa_set_bus_target_lun(device,
HPSA_PHYSICAL_DEVICE_BUS, -1, -1); HPSA_PHYSICAL_DEVICE_BUS, -1, -1);

2
drivers/scsi/hpsa.h
View File

@ -69,6 +69,7 @@ struct hpsa_scsi_dev_t {
u64 sas_address; u64 sas_address;
unsigned char vendor[8]; /* bytes 8-15 of inquiry data */ unsigned char vendor[8]; /* bytes 8-15 of inquiry data */
unsigned char model[16]; /* bytes 16-31 of inquiry data */ unsigned char model[16]; /* bytes 16-31 of inquiry data */
unsigned char rev; /* byte 2 of inquiry data */
unsigned char raid_level; /* from inquiry page 0xC1 */ unsigned char raid_level; /* from inquiry page 0xC1 */
unsigned char volume_offline; /* discovered via TUR or VPD */ unsigned char volume_offline; /* discovered via TUR or VPD */
u16 queue_depth; /* max queue_depth for this device */ u16 queue_depth; /* max queue_depth for this device */
@ -402,6 +403,7 @@ struct offline_device_entry {
#define HPSA_RAID_VOLUME_BUS 1 #define HPSA_RAID_VOLUME_BUS 1
#define HPSA_EXTERNAL_RAID_VOLUME_BUS 2 #define HPSA_EXTERNAL_RAID_VOLUME_BUS 2
#define HPSA_HBA_BUS 0 #define HPSA_HBA_BUS 0
#define HPSA_LEGACY_HBA_BUS 3
/* /*
Send the command to the hardware Send the command to the hardware