Commit c85f8cb9 authored by Chandra Seetharaman's avatar Chandra Seetharaman Committed by James Bottomley

[SCSI] scsi_dh: Retry mode select in rdac device handler

When the mode select sent to the controller fails with the retryable
error, it is better to retry the mode_select from the hardware handler
itself, instead of propagating the failure to dm-multipath.
Signed-off-by: default avatarChandra Seetharaman <sekharan@us.ibm.com>
Signed-off-by: default avatarJames Bottomley <James.Bottomley@HansenPartnership.com>
parent 9eece961
...@@ -24,6 +24,7 @@ ...@@ -24,6 +24,7 @@
#include <scsi/scsi_dh.h> #include <scsi/scsi_dh.h>
#define RDAC_NAME "rdac" #define RDAC_NAME "rdac"
#define RDAC_RETRY_COUNT 5
/* /*
* LSI mode page stuff * LSI mode page stuff
...@@ -478,21 +479,27 @@ static int send_mode_select(struct scsi_device *sdev, struct rdac_dh_data *h) ...@@ -478,21 +479,27 @@ static int send_mode_select(struct scsi_device *sdev, struct rdac_dh_data *h)
{ {
struct request *rq; struct request *rq;
struct request_queue *q = sdev->request_queue; struct request_queue *q = sdev->request_queue;
int err = SCSI_DH_RES_TEMP_UNAVAIL; int err, retry_cnt = RDAC_RETRY_COUNT;
retry:
err = SCSI_DH_RES_TEMP_UNAVAIL;
rq = rdac_failover_get(sdev, h); rq = rdac_failover_get(sdev, h);
if (!rq) if (!rq)
goto done; goto done;
sdev_printk(KERN_INFO, sdev, "queueing MODE_SELECT command.\n"); sdev_printk(KERN_INFO, sdev, "%s MODE_SELECT command.\n",
(retry_cnt == RDAC_RETRY_COUNT) ? "queueing" : "retrying");
err = blk_execute_rq(q, NULL, rq, 1); err = blk_execute_rq(q, NULL, rq, 1);
if (err != SCSI_DH_OK) blk_put_request(rq);
if (err != SCSI_DH_OK) {
err = mode_select_handle_sense(sdev, h->sense); err = mode_select_handle_sense(sdev, h->sense);
if (err == SCSI_DH_RETRY && retry_cnt--)
goto retry;
}
if (err == SCSI_DH_OK) if (err == SCSI_DH_OK)
h->state = RDAC_STATE_ACTIVE; h->state = RDAC_STATE_ACTIVE;
blk_put_request(rq);
done: done:
return err; return err;
} }
......
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