diff options
author | Mike Christie <michaelc@cs.wisc.edu> | 2010-04-09 23:07:37 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-04-11 15:04:02 -0400 |
commit | 5738d4449c1baf05e8345684d12371f76296473d (patch) | |
tree | 00710629fd26aff9dc7c55c1aebc9d32c692e2a0 /drivers/scsi | |
parent | ef7d17a907d89d590831a3995ed8638a9ad394bc (diff) |
[SCSI] scsi_dh_emc: request flag cleanup
blk_get_request sets the cmd_flags, so we should not and do not
need to set them. If we did set them to a different value then
it can cause a oops in the elevator code.
Signed-off-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/device_handler/scsi_dh_emc.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/scsi/device_handler/scsi_dh_emc.c b/drivers/scsi/device_handler/scsi_dh_emc.c index e8a0bc3efd49..6faf472f7537 100644 --- a/drivers/scsi/device_handler/scsi_dh_emc.c +++ b/drivers/scsi/device_handler/scsi_dh_emc.c | |||
@@ -285,13 +285,11 @@ static struct request *get_req(struct scsi_device *sdev, int cmd, | |||
285 | switch (cmd) { | 285 | switch (cmd) { |
286 | case MODE_SELECT: | 286 | case MODE_SELECT: |
287 | len = sizeof(short_trespass); | 287 | len = sizeof(short_trespass); |
288 | rq->cmd_flags |= REQ_RW; | ||
289 | rq->cmd[1] = 0x10; | 288 | rq->cmd[1] = 0x10; |
290 | rq->cmd[4] = len; | 289 | rq->cmd[4] = len; |
291 | break; | 290 | break; |
292 | case MODE_SELECT_10: | 291 | case MODE_SELECT_10: |
293 | len = sizeof(long_trespass); | 292 | len = sizeof(long_trespass); |
294 | rq->cmd_flags |= REQ_RW; | ||
295 | rq->cmd[1] = 0x10; | 293 | rq->cmd[1] = 0x10; |
296 | rq->cmd[8] = len; | 294 | rq->cmd[8] = len; |
297 | break; | 295 | break; |