diff options
| author | Moger, Babu <Babu.Moger@netapp.com> | 2011-08-29 12:24:37 -0400 |
|---|---|---|
| committer | Christophe Varoqui <christophe.varoqui@opensvc.com> | 2011-09-01 08:41:50 +0200 |
| commit | 4050f1edc37ae5b327be51c4dffb071955c45cb2 (patch) | |
| tree | 48757fee6dd6f2027145dae51d6028dabc732e90 | |
| parent | fa7074f35c67ff1ef29f1aea5a484c5dea25863a (diff) | |
| download | multipath-tools-4050f1edc37ae5b327be51c4dffb071955c45cb2.tar.gz | |
multipath-tools: service mode changes for RDAC storage
This patch was not picked up yet, so resubmitting it again. There no change except it was generated on top of the latest file.
This patch handles the recent changes in NetApp RDAC storage firmware to report service mode. Firmware changed the inquiry page 0xc9 to report service mode. Purpose this change is to avoid DMMP going into infinite loop of switching back and forth between controllers when a controller is placed in service mode. This fixes the problem and reports the path as failed if the controller is placed in service mode.
Signed-off-by: Babu Moger <babu.moger@netapp.com>
Reviewed-by: Yanling Qi <yanling.qi@netapp.com>
Reviewed-by: Somasundaram Krishnasamy <Somasundaram.Krishnasamy@netapp.com>
| -rw-r--r-- | libmultipath/checkers/rdac.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/libmultipath/checkers/rdac.c b/libmultipath/checkers/rdac.c index 989d373..d7dfb08 100644 --- a/libmultipath/checkers/rdac.c +++ b/libmultipath/checkers/rdac.c @@ -198,7 +198,10 @@ struct volume_access_inq char PQ_PDT; char dontcare0[7]; char avtcvp; - char dontcare1[39]; + char dontcare1; + char asym_access_state_cur; + char vendor_specific_cur; + char dontcare2[36]; }; extern int @@ -218,6 +221,14 @@ libcheck_check (struct checker * c) goto done; } + /* check if controller is in service mode */ + if ((inq.avtcvp & 0x10) && + ((inq.asym_access_state_cur & 0x0F) == 0x3) && + (inq.vendor_specific_cur == 0x7)) { + ret = PATH_DOWN; + goto done; + } + /* If owner set or ioship mode is enabled return PATH_UP always */ if ((inq.avtcvp & 0x1) || ((inq.avtcvp >> 5) & 0x1)) ret = PATH_UP; |