Commit 6d73c851 authored by Al Viro's avatar Al Viro Committed by James Bottomley

[SCSI] scsi_lib: fix recognition of cache type of Initio SBP-2 bridges

Regardless what mode page was asked for, Initio INIC-14x0 and
INIC-2430 always return page 6 without mode page headers.  Try to
recognise this as a special case in scsi_mode_sense and setting the
mode sense headers accordingly.
Signed-off-by: default avatarAl Viro <viro@ftp.linux.org.uk>
Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
parent fc25307d
...@@ -1892,8 +1892,16 @@ scsi_mode_sense(struct scsi_device *sdev, int dbd, int modepage, ...@@ -1892,8 +1892,16 @@ scsi_mode_sense(struct scsi_device *sdev, int dbd, int modepage,
} }
if(scsi_status_is_good(result)) { if(scsi_status_is_good(result)) {
data->header_length = header_length; if (unlikely(buffer[0] == 0x86 && buffer[1] == 0x0b &&
if(use_10_for_ms) { (modepage == 6 || modepage == 8))) {
/* Initio breakage? */
header_length = 0;
data->length = 13;
data->medium_type = 0;
data->device_specific = 0;
data->longlba = 0;
data->block_descriptor_length = 0;
} else if(use_10_for_ms) {
data->length = buffer[0]*256 + buffer[1] + 2; data->length = buffer[0]*256 + buffer[1] + 2;
data->medium_type = buffer[2]; data->medium_type = buffer[2];
data->device_specific = buffer[3]; data->device_specific = buffer[3];
...@@ -1906,6 +1914,7 @@ scsi_mode_sense(struct scsi_device *sdev, int dbd, int modepage, ...@@ -1906,6 +1914,7 @@ scsi_mode_sense(struct scsi_device *sdev, int dbd, int modepage,
data->device_specific = buffer[2]; data->device_specific = buffer[2];
data->block_descriptor_length = buffer[3]; data->block_descriptor_length = buffer[3];
} }
data->header_length = header_length;
} }
return result; return result;
......
...@@ -1333,6 +1333,12 @@ sd_read_cache_type(struct scsi_disk *sdkp, char *diskname, ...@@ -1333,6 +1333,12 @@ sd_read_cache_type(struct scsi_disk *sdkp, char *diskname,
if (!scsi_status_is_good(res)) if (!scsi_status_is_good(res))
goto bad_sense; goto bad_sense;
if (!data.header_length) {
modepage = 6;
printk(KERN_ERR "%s: missing header in MODE_SENSE response\n",
diskname);
}
/* that went OK, now ask for the proper length */ /* that went OK, now ask for the proper length */
len = data.length; len = data.length;
......
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