[Ocfs2-tools-devel] [PATCH 1/1] o2hbmonitor: Disk heartbeat monitor

[Sunil Mushran]

On 30-10-2010 07:22, Srinivas Eeda wrote:
>> A monitor for a monitor is classic over-engineering. If open/read/readdir
>> are failing, it assumes that the cluster is not up or debugfs is not mounted.
>> Occam's Razor. If former, try again after some timeout. If latter, print
>> a discreet message.
>>
>> We don't want multiple o2hbmonitors running. Easy to do using a semaphore.
>> I have left that for Srini. ;)
>>    
> is there any naming convention for creating a semaphore? Is the 
> following fine?
> key=(key_t)o2cb_crc32("o2hbmonitor")
> ret=semget(key, 2, IPC_CREAT);

A #define would be easier to recognize.