the multipath for EMC CX700

["wuyi@xxxxxxxxxx" <wuyi@xxxxxxxxxx>]

hi:

    I have  some problems about multipath for EMC CX700,the EMC checker indicated "SG_INFO_OK_MASK".but there is no kernel info indicated that the status, driver status or host status was failed.And next checker will up the path immediately.So i think there should be a  "retry" checker to insure the path is really down.

The  message:

Mar 16 19:38:23 node7 sshd[19867]: Accepted password for root from 10.165.14.194 port 59421 ssh2
Mar 16 19:46:14 node7 : error initializing: No module named up2date_client.up2dateAuth
Mar 16 19:46:57 node7 multipathd: sdb: emc_clariion_checker: query command indicates error
Mar 16 19:46:57 node7 multipathd: checker failed path 8:16 in map mpath4
Mar 16 19:46:57 node7 kernel: [11120.924573] qla2xxx 0000:82:00.0: scsi(4:0:0): Abort command issued -- 1 4a1 2002.
Mar 16 19:46:57 node7 kernel: [11120.925307] device-mapper: multipath: Failing path 8:16.
Mar 16 19:46:57 node7 multipathd: mpath4: remaining active paths: 1
Mar 16 19:46:57 node7 multipathd: mpath4: switch to path group #2
Mar 16 19:47:02 node7 multipathd: sdb: emc_clariion_checker: Path healthy
Mar 16 19:47:02 node7 multipathd: 8:16: reinstated
Mar 16 19:47:02 node7 multipathd: mpath4: remaining active paths: 2
Mar 16 19:47:02 node7 multipathd: mpath4: switch to path group #1

The diff:

--- libcheckers/emc_clariion.c.old 2015-04-13 09:31:04.000000000 +0800
+++ libcheckers/emc_clariion.c 2015-04-13 09:34:47.000000000 +0800
@@ -40,6 +40,8 @@

int emc_clariion(struct checker * c)
{
+ int retrytimes = 3;
+
unsigned char sense_buffer[256] = { 0, };
unsigned char sb[128] = { 0, };
unsigned char inqCmdBlk[INQUIRY_CMDLEN] = {INQUIRY_CMD, 1, 0xC0, 0,
@@ -59,12 +61,18 @@
io_hdr.sbp = sb;
io_hdr.timeout = DEF_TIMEOUT;
io_hdr.pack_id = 0;
+
+retry:
if (ioctl(c->fd, SG_IO, &io_hdr) < 0) {
MSG(c, "emc_clariion_checker: sending query command failed");
return PATH_DOWN;
}
if (io_hdr.info & SG_INFO_OK_MASK) {
MSG(c, "emc_clariion_checker: query command indicates error");
+ if(retrytimes){

+ retrytimes--; 
+ goto retry;
+ }
return PATH_DOWN;
}
if (/* Verify the code page - right page & revision */

I don`t think my modification is good enough,so i hope you can give me some useful ideas to make it better.

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel