[lvm-devel] [PATCH 0/7] dmevent mlockall() fixes

Zdenek Kabelac zkabelac at redhat.com
Mon Mar 29 15:27:18 UTC 2010


This patch set tries to fix dmeventd.

Dmeventd got actually broken with our new mlock() code.
There is no big chance to make current architecture with threads usable
with mlock() code - so mlockall() is used for daemon case.

Snapshot plugin current doesn't use cmd_context and doesn't need memory
locking.

Mirror plugin uses memlock_inc_daemon() and locks while binary to memory.

C locales are enforced to avoid loading of 100MB locale-archive file.

Few minor fixes for syslog message and resource leaks in error path.

Statistic for mlock() could be based on readable sizes.

Zdenek Kabelac (7):
  Remove mlockall() form dmeventd
  Fix resouces leak in error path
  Release pool in the same reversed order
  Updated syslog messages
  Force C locale
  Update memlock
  Count only readable size for memlock stats.

 daemons/dmeventd/dmeventd.c                       |   14 ++---
 daemons/dmeventd/plugins/lvm2/dmeventd_lvm.c      |    4 +-
 daemons/dmeventd/plugins/mirror/dmeventd_mirror.c |   28 +++++-----
 lib/mm/memlock.c                                  |   64 ++++++++++-----------
 4 files changed, 51 insertions(+), 59 deletions(-)




More information about the lvm-devel mailing list