From: Martin Wilck <mwilck@xxxxxxxx> Hi Christophe, hi Ben, hi lixiaokeng, this series was inspired by lixiaokeng's recent posting "[QUESTION] memory leak in main (multipath)". It implements my first idea, registering cleanup handlers with atexit(). However it turned out to be quite complex. In particular multipathd has a lot of things to clean up. This series is based on the previous series "multipath-tools: shutdown, libdevmapper races, globals" (v3). While the bulk of the series is the cleanup handling, it also contains some bug fixes for issues that I found while working on this. Changes v1 -> v2: Patches 24..29 have been added; they are logging cleanups, 24/29 and 29/29 are fixes to issues Ben had mentioned during his review. 01/29 "multipathd: uxlsnr: avoid deadlock on exit" Fix invalid array element reference (Ben) 07/29 "multipathd: move conf destruction into separate function" Don't reset logink when the log thread is stopped (Ben) Logging via logsink after shutting down the log thread is racy (no worse than before); the race will be fixed in 29/29. 14/29 "libmultipath: add libmp_dm_exit()": Fixed the skip_libmp_dm_init() case (Ben). 18/29 "libmultipath: fix log_thread startup and teardown" No need to wait before joining the log thread in log_thread_stop() (Ben) -> "libmultipath: fix race between log_safe and log_thread_stop()" 23/29: "multipath-tools: mpath-tools.supp: file with valgrind suppressions" Remove empty line at end of file (Ben) 24/29 "libmultipath: use libmp_verbosity to track verbosity" This fixes 13/21 "libmultipath: provide defaults for {get,put}_multipath_config" (from the previous patch series "multipath-tools: shutdown, libdevmapper races, globals"; control of verbosity during config file loading) 29/29 "libmultipath: fix race between log_safe and log_thread_stop()" (see 18/29) Regards Martin Martin Wilck (29): multipathd: uxlsnr: avoid deadlock on exit multipathd: Fix liburcu memory leak multipathd: move handling of io_err_stat_attr into libmultipath multipathd: move vecs desctruction into cleanup function multipathd: make some globals static multipathd: move threads destruction into separate function multipathd: move conf destruction into separate function multipathd: move pid destruction into separate function multipathd: close pidfile on exit multipathd: add helper for systemd notification at exit multipathd: child(): call cleanups in failure case, too multipathd: unwatch_all_dmevents: check if waiter is initialized multipathd: print error message if config can't be loaded libmultipath: add libmp_dm_exit() multipathd: fixup libdm deinitialization libmultipath: log_thread_stop(): check if logarea is initialized multipathd: add cleanup_child() exit handler libmultipath: fix log_thread startup and teardown multipathd: move cleanup_{prio,checkers,foreign} to libmultipath_exit multipath: use atexit() for cleanup handlers mpathpersist: use atexit() for cleanup handlers multipath: fix leaks in check_path_valid() multipath-tools: mpath-tools.supp: file with valgrind suppressions libmultipath: use libmp_verbosity to track verbosity libmultipath: introduce symbolic values for logsink libmultipath: simplify dlog() multipathd: common code for "-k" and command args multipathd: sanitize uxsock_listen() libmultipath: fix race between log_safe and log_thread_stop() libmpathpersist/mpath_persist.c | 7 +- libmultipath/config.c | 14 +- libmultipath/config.h | 2 + libmultipath/debug.c | 38 +-- libmultipath/debug.h | 27 +- libmultipath/devmapper.c | 23 +- libmultipath/devmapper.h | 1 + libmultipath/io_err_stat.c | 7 +- libmultipath/libmultipath.version | 31 +- libmultipath/log_pthread.c | 114 +++++--- mpathpersist/main.c | 5 +- multipath/main.c | 119 ++++---- multipathd/dmevents.c | 2 + multipathd/main.c | 396 ++++++++++++++++---------- multipathd/uxlsnr.c | 80 ++++-- tests/alias.c | 1 + tests/blacklist.c | 2 + tests/globals.c | 3 +- tests/hwtable.c | 2 +- tests/test-log.c | 4 +- tests/test-log.h | 3 +- third-party/valgrind/mpath-tools.supp | 32 +++ 22 files changed, 585 insertions(+), 328 deletions(-) create mode 100644 third-party/valgrind/mpath-tools.supp -- 2.28.0 -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel