Hi Vijay,
Sorry for a late report, but I was busy with other stuff and didn't test
as much as I wanted, but anyway here is what I have by now.
1. Patches apply against the 63704387009443bdb37d9deaaafa9ab121d45bfb
without any problems. Everything builds correctly (tested it on centos
6.2 and fedora 16).
2. multipathd runs correctly.
3. mpathpersist is missing --no-inquiry option. It would be nice to be
compatible with sg_persist. (breaks most of my test scripts.)
4. mpathpersist -d /dev/mapper/blah --in --read-status
mpathpersist: unrecognized option '--read-status'
unrecognised switch code 0x3f ??
5. registration works
6. reservation, reserves only a single path. (is this by design?)
mpathpersist --out --reserve --param-rk=123abc --prout-type=8 -d
/dev/mapper/blah
sg_persist -i -s /dev/mapper/blah
QNAP iSCSI Storage 3.1
Peripheral device type: disk
PR generation=0x8
Key=0x123abc
All target ports bit clear
Relative port address: 0x1
not reservation holder
Transport Id of initiator:
iSCSI name and session id: iqn.2009-11.com.adriaticsolutions:blah
Key=0x123abc
All target ports bit clear
Relative port address: 0x1
<< Reservation holder >>
scope: LU_SCOPE, type: Exclusive Access, all registrants
Transport Id of initiator:
iSCSI name and session id: iqn.2009-11.com.adriaticsolutions:blah
multipath -ll
blah (36001405c55fc03cd8193d491eda0d4d7) dm-4 QNAP,iSCSI Storage
size=10G features='0' hwhandler='0' wp=rw
`-+- policy='round-robin 0' prio=1 status=active
|- 16:0:0:0 sda 8:0 active ready running
`- 17:0:0:0 sdb 8:16 failed faulty running
7. mpathpersist -d /dev/mapper/blah --in --read-reservation
Persistent Reserve IN command failed
8. release 'fails' because reservation registered only a single path and
the other one is in a failed state.
mpathpersist -d /dev/mapper/blah --out --release --param-rk=123abc
--prout-type=8
Jan 24 00:43:53 | 36001405c55fc03cd8193d491eda0d4d7: pr in read
reservation command failed.
PR out: command failed
9. de-registration works....
I haven't had time to test path fail over and weird cases when multiple
nodes try to do destructive things to the shared luns...
thanks
fil
On 01/23/2012 11:09 PM, Chauhan, Vijay wrote:
> Thanks for these info Fil. Can you please also share your feedback on your in-depth testing.
>
> I appreciate your effort.
>
> On January 20, 2012, Fil wrote:
>> Thanks for the patches Vijay,
>>
>> I tested the new set of patches against the git repo, and it applied
>> cleanly compiled and is running as expected. I will do more indepth
>> testing over the weekend.
>>
>> thanks
>> fil
>>
>> On 01/19/2012 02:20 PM, Chauhan, Vijay wrote:
>>> Resending this patch with the fix mentioned by Mike
>> Snitzer<snitzer@xxxxxxxxxx> and Fil<lists@xxxxxxxxxxxx>.
>>> Thanks to all for sending me your review comments. Fil, Thanks for
>> trying out mpathpersist and providing your
>>> feedback. Also thanking Yanling Qi<Yanling.Qi@xxxxxxxxxx> for
>> proposing this design for pr management feature.
>>> These series of patch is made on commit
>> 864162b08067d3e55d52996846d10a046956c6d6.
>
>
> Regards,
> Vijay
>
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel
