[lvm-devel] [PATCH] pid files cleanup take 2

Fabio M. Di Nitto fdinitto at redhat.com
Mon Jul 12 09:37:50 UTC 2010


Hi all,

this new patch moves create_lockfile to dm_create_lockfile (dm-file.c),
addresses the return code issue and switch also dmeventd to use the
generic implementation.

autoreconf executed on F-12.

NOTE: dmeventd has, in its current implementation, a slightly more
detailed error handling of the pid file. See the 2 return codes that are
not necessary any longer by using dm_create_lock.
If current behavior of dmeventd has to be retained, then we can simply
modify dm_create_lockfile to return different values depending on the
error and dmeventd can simply re-map them for its needs.

Cheers
Fabio
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: pid_files2.diff
URL: <http://listman.redhat.com/archives/lvm-devel/attachments/20100712/3ad8768a/attachment.ksh>


More information about the lvm-devel mailing list