diff options
| author | Stephen M. Cameron <scameron@beardog.cce.hp.com> | 2012-01-19 14:01:09 -0600 |
|---|---|---|
| committer | James Bottomley <JBottomley@Parallels.com> | 2012-02-19 08:08:56 -0600 |
| commit | bbef6c0ce8046114c07f5b6a62ce6a239503dd61 (patch) | |
| tree | a408dddb332f794cb451affa9aced3db970ca6c7 /drivers/scsi/hpsa.c | |
| parent | 9bc3711cbb67ac620bf09b4a147cbab45b2c36c0 (diff) | |
[SCSI] hpsa: make target and lun match what SCSI REPORT LUNs returns
Some distros have a "rescan-scsi-bus.sh" script which depends on
SCSI REPORT LUNs not reporting something different than what the
driver tells the kernel, even if the driver uses scan_start and
scan_finished methods of the SCSI host template to override the
usual SCSI midlayer discovery code. Previously, 1 was added to
the LUN to make room to insert the RAID controller device at
LUN 0. Now, the RAID controller is moved to bus 3, and 1 is no
longer added to the LUN. However, SCSI REPORT LUNS on Smart Array
doesn't report physical devices like tape drives or auto-loaders
as it turns out, so those particular device types still won't match.
Generally the logical drives are reported first however, so at
least those should match.
Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/hpsa.c')
| -rw-r--r-- | drivers/scsi/hpsa.c | 25 |
1 files changed, 5 insertions, 20 deletions
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 4e7df2e5701..264c1764bef 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c @@ -1647,30 +1647,15 @@ static void figure_bus_target_lun(struct ctlr_info *h, *target = (lunid >> 16) & 0x3fff; *lun = lunid & 0x00ff; } else { - if (likely(is_scsi_rev_5(h))) { - /* All current smart arrays (circa 2011) */ - *bus = 0; - *target = 0; - *lun = (lunid & 0x3fff) + 1; - } else { - /* Traditional old smart array way. */ - *bus = 0; - *target = lunid & 0x3fff; - *lun = 0; - } + *bus = 0; + *target = 0; + *lun = (lunid & 0x3fff); } } else { - /* physical device */ if (is_hba_lunid(lunaddrbytes)) - if (unlikely(is_scsi_rev_5(h))) { - *bus = 0; /* put p1210m ctlr at 0,0,0 */ - *target = 0; - *lun = 0; - return; - } else - *bus = 3; /* traditional smartarray */ + *bus = 3; /* controller */ else - *bus = 2; /* physical disk */ + *bus = 2; /* physical device */ *target = -1; *lun = -1; /* we will fill these in later. */ } |