[Cluster-devel] Re: [PATCH] dlm: fix coverity-spotted stupidity

Patrick Caulfield pcaulfie at redhat.com
Mon Mar 26 07:35:00 UTC 2007


Abhijith Das wrote:
> Steven Whitehouse wrote:
> 
>> Hi,
>>
>> Now applied to the -nmw git tree. Thanks,
>>
>> Steve.
>>
>> On Thu, 2007-03-22 at 11:49 +0000, Patrick Caulfield wrote:
>>  
>>
>>> This patch fixes the EAGAIN checking in lowcomms-tcp.c so it actually
>>> works.
>>> Thanks to Adrian Bunk & the coverity checker for spotting that
>>>
>>> Signed-Off-By: Patrick Caulfield <pcaulfie at redhat.com>
>>>
> Hi,
> I pulled this patch along with the latest stuff from the nmw tree. Looks
> like this patch makes dlm_recv/0 use 100% CPU. I'm not able to restart
> clvmd and such. I'm guessing the kernel_recvmsg() keeps returning
> -EAGAIN and we keep rescheduling. Any clue as to what is going on?


Bugger. You're right.

-- 

patrick




More information about the Cluster-devel mailing list