[linux-lvm] clvmd-corosync should check lksb.sb_status instead of return code ?

Chrissie Caulfield ccaulfie at redhat.com
Wed Feb 25 13:52:56 UTC 2009

Xinwei Hu wrote:
> Hi Chrissie,
>   You are right. It should be returned in errno. Also, it seems errno
> should positive as lksb.sb_status too.
>   I modified the patch a little bit further.
>   Any comments ?


You're right ... this kernel-itis is catching! lksb status are indeed
positive and correct for returning as errno.

I'll commit this patch

Thank you.


> 2009/2/25 Chrissie Caulfield <ccaulfie at redhat.com>:
>> Xinwei Hu wrote:
>>> Hi all,
>>>   In clvmd-corosync, lock_resource calls dlm_ls_lock_wait with flags
>>> set to LKF_NOQUEUE.
>>>   When LKF_NOQUEUE is set, the return code of dlm_ls_lock_wait is
>>> always 0, while the actual errno is in lksb.sb_status.
>>>   This causes problem when using vgchange -aey or something like.
>>>   I attached a patch to try to fix this issue.
>>>   Please help to review. Thanks.
>> Hi
>> You are correct, the functions do need to check the LKSB. Your patch
>> isn't quite right I don't think though - the lksb status should be
>> returned in errno as shown below:
>> --
>> Chrissie

More information about the linux-lvm mailing list