Chandra Seetharaman wrote:
On Wed, 2008-04-16 at 11:29 -0500, Mike Christie wrote:
Chandra Seetharaman wrote:
+
+static int send_cmd(struct scsi_device *sdev, int cmd)
+{
+ struct request *rq = get_req(sdev, cmd);
+
+ if (!rq)
+ return SCSI_DH_RES_TEMP_UNAVAIL;
+
+ return blk_execute_rq(sdev->request_queue, NULL, rq, 1);
+}
+
My only concerns are:
1. EMC and HP need to send a command to every device to transition them.
Because we do blk_execute_rq from the dm multipath workqueue we can now
only failover/failback for a couple devices at a time.
I am not sure if this is a big deal, because this the error handler path
so it is going to be slower than the normal path. But it seems like
Yes. But...
pg_init() due to failover/failback will be sent only when I/O is
sent/resent to a multipath device, isn't it ? and we don't expect I/Os
to be sent to all the devices at the same time (all the time), do we ?
I am not sure what you mean by all the time, because I am talking about
failover times above. And for failover I think I said yes in the
previous mail. For EMC we are currently sending failover commands to all
the devices at the same time, because EMC does not do the controller
failover RDAC does.
So, as you pointed, is it a big deal ? :)
In the previous mail I specifically said users might care, because they
are picky about failover times, so the answer is to your question is
what I said before, maybe :) I said I am not sure, because I do not have
any numbers for the failover times.
BTW, As you know, it was originally coded that way(patchset posted on
Jan 23, 2008) and later changed as per James comments (through you) that
the code was overusing blk_execute_rq_nowait().
Yes, I know, and as you know I did not agree with James. The reason I
bring it up again is that Ed is not doing dm-multipath stuff, so EMC
does not have a good reviewer right now, and I want to make sure these
issues are raised on the list during the review so we can all discuss
them together.
If James thinks it is a big enough problem he can offer some of this
alternatives. If not then he will merge it and we can see if people even
notice and handle it later. I just want to make sure we all know what is
going on, because Alasdair is not a scsi guy. James does not know all
the fun details of every box. And the EMC guys are not up to speed on
linux. I am just worried we are going to get a bad review.
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel
