diff options
| author | Moore, Eric <Eric.Moore@lsil.com> | 2006-04-13 16:08:17 -0600 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2006-04-14 09:35:12 -0500 |
| commit | 3c0c25b97c7d020ef07f6366cf1d668a8e980c7c (patch) | |
| tree | 229cd04ca4836ebdf7b12bf71540d5b92d32f170 /drivers/message | |
| parent | 1e08dcb39c6b95ec84fc8bad97d5bdce879b76c9 (diff) | |
| download | linux-3.10-3c0c25b97c7d020ef07f6366cf1d668a8e980c7c.tar.gz linux-3.10-3c0c25b97c7d020ef07f6366cf1d668a8e980c7c.tar.bz2 linux-3.10-3c0c25b97c7d020ef07f6366cf1d668a8e980c7c.zip | |
[SCSI] mptfusion - fix panic in mptsas_slave_configure
Driver panic when RAID logical volume was present when driver
loaded, or when a RAID logical volume was created on the fly.
This issue was created in due to recent scsi_transport_sas change,
when sas_read_port_mode_page was added into the mptsas drivers
slave_config entry point.
This new API expects that all sdev's to be assocated to an rphy, however
that is not the case for logical volumes, as they are created using
scsi_add_device, instead of sas_rphy_add().
Signed-off-by: Eric Moore <Eric.Moore@lsil.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/message')
| -rw-r--r-- | drivers/message/fusion/mptsas.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/drivers/message/fusion/mptsas.c b/drivers/message/fusion/mptsas.c index 010d4a39269..e9716b10ace 100644 --- a/drivers/message/fusion/mptsas.c +++ b/drivers/message/fusion/mptsas.c @@ -366,7 +366,15 @@ mptsas_sas_enclosure_pg0(MPT_ADAPTER *ioc, struct mptsas_enclosure *enclosure, static int mptsas_slave_configure(struct scsi_device *sdev) { - sas_read_port_mode_page(sdev); + struct Scsi_Host *host = sdev->host; + MPT_SCSI_HOST *hd = (MPT_SCSI_HOST *)host->hostdata; + + /* + * RAID volumes placed beyond the last expected port. + * Ignore sending sas mode pages in that case.. + */ + if (sdev->channel < hd->ioc->num_ports) + sas_read_port_mode_page(sdev); return mptscsih_slave_configure(sdev); } |