From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chandra Seetharaman Subject: Re: [PATCH 0/3] Changes to Userspace multipath code to support lsi-rdac Date: Thu, 22 Mar 2007 15:45:41 -0800 Message-ID: <1174607141.14100.9.camel@linuxchandra> References: <20070321193833.24487.76682.sendpatchset@localhost.localdomain> <1174518416.6218.35.camel@localhost.localdomain> Reply-To: sekharan@us.ibm.com, device-mapper development Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <1174518416.6218.35.camel@localhost.localdomain> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Christophe Varoqui Cc: Mike Anderson , dm-devel@redhat.com, michaelc@cs.wisc.edu, agk@redhat.com List-Id: dm-devel.ids Cool, Thanks. chandra On Thu, 2007-03-22 at 00:06 +0100, Christophe Varoqui wrote: > Le mercredi 21 mars 2007 =C3=A0 11:38 -0800, Chandra Seetharaman a =C3=A9= crit : > > Hi All, > >=20 > > While working on lsi-rdac hardware handler, I found that the checkers= "tur" > > and "readsector0" are not serving the lsi-rdac properly. Also, the st= orage > > device has a active/passive state which need to be handled properly. > >=20 > > So, I wrote a new path checker for lsi-rdac, to use the C4 inquiry pa= ge to get > > the current owner of the lun and set the state as UP(active) or GHOST= (passive). > > Changes are in patch 1/3. > >=20 > > After applying the above patch, I realized the priorities are not set= properly. > > Made some changes to use the GHOST state while setting priorities. Ch= anges are > > in patch 2/3. > >=20 > > Debugging the above patches found that the wait time (5 seconds) in m= ultipathd > > was not sufficient for the devices to appear in /dev/ It was taking a= bout 20 > > seconds. Changed the wait time to be 60 seconds. Change in patch 3/3. > >=20 > > Please provide me with your feedback/comment on these changes regardi= ng > > correctness, completeness etc., > >=20 > Looks very good. I merged the 3 patches. >=20 > In 1/3, I changed the DEF_TIMEOUT name to avoid surcharging the one in > checker.h >=20 > I'm having a hard time push to kernel.org, so don't expect to see the > git tree updated soon. >=20 > Thanks, > cvaroqui >=20 --=20 ---------------------------------------------------------------------- Chandra Seetharaman | Be careful what you choose.... - sekharan@us.ibm.com | .......you may get it. ----------------------------------------------------------------------