aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/scsi
diff options
context:
space:
mode:
authorAlan Stern <stern@rowland.harvard.edu>2008-10-10 16:03:20 -0400
committerJames Bottomley <James.Bottomley@HansenPartnership.com>2008-10-23 12:42:19 -0400
commitd4c9b736080056ae3ba81dcf2ac418193c57dbb1 (patch)
treee27d56d2cb3ee0ffcab0499654105cad5efa1627 /drivers/scsi
parent36a529202b334d0513cfeabeadca940ec41c8165 (diff)
[SCSI] sd: remove command-size switching code
This patch (as1138) removes from sd.c some old code for switching from 10-byte commands to 6-byte commands. This code is redundant -- the switching for READ and WRITE is already handled in scsi_io_completion() and the switching for MODE SENSE is already handled in scsi_mode_sense(). (There is no comparable switch for MODE SELECT, but I doubt one is needed.) Furthermore the other handlers do a better job; they check for appropriate ASC and ASCQ values before blindly switching the size. The code in sd.c is known to cause problems with some devices by switching when it shouldn't. Signed-off-by: Alan Stern <stern@rowland.harvard.edu> Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/scsi')
-rw-r--r--drivers/scsi/sd.c9
1 files changed, 0 insertions, 9 deletions
diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c
index 55e6ed4b886d..7e22aa7b8b8a 100644
--- a/drivers/scsi/sd.c
+++ b/drivers/scsi/sd.c
@@ -1084,15 +1084,6 @@ static int sd_done(struct scsi_cmnd *SCpnt)
1084 scsi_print_sense("sd", SCpnt); 1084 scsi_print_sense("sd", SCpnt);
1085 good_bytes = sd_completed_bytes(SCpnt); 1085 good_bytes = sd_completed_bytes(SCpnt);
1086 } 1086 }
1087 if (!scsi_device_protection(SCpnt->device) &&
1088 SCpnt->device->use_10_for_rw &&
1089 (SCpnt->cmnd[0] == READ_10 ||
1090 SCpnt->cmnd[0] == WRITE_10))
1091 SCpnt->device->use_10_for_rw = 0;
1092 if (SCpnt->device->use_10_for_ms &&
1093 (SCpnt->cmnd[0] == MODE_SENSE_10 ||
1094 SCpnt->cmnd[0] == MODE_SELECT_10))
1095 SCpnt->device->use_10_for_ms = 0;
1096 break; 1087 break;
1097 default: 1088 default:
1098 break; 1089 break;