[dm-devel] [RFC PATCH 4/6] multipathd: cancel threads early during shutdown

[Martin Wilck]

Cancel the other threads before taking vecs->lock. This avoids
delays during shutdown caused e.g. by the checker thread holding
the vecs lock.

Signed-off-by: Martin Wilck <mwi...@suse.com>
---
 multipathd/main.c | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/multipathd/main.c b/multipathd/main.c
index 413beee0..569a27ac 100644
--- a/multipathd/main.c
+++ b/multipathd/main.c
@@ -2872,23 +2872,24 @@ child (void * param)
                }
        }
 
-       lock(&vecs->lock);
+       pthread_cancel(check_thr);
+       pthread_cancel(uevent_thr);
+       pthread_cancel(uxlsnr_thr);
+       pthread_cancel(uevq_thr);
+       if (poll_dmevents)
+               pthread_cancel(dmevent_thr);
+
        conf = get_multipath_config();
        queue_without_daemon = conf->queue_without_daemon;
        put_multipath_config(conf);
+
+       lock(&vecs->lock);
        if (queue_without_daemon == QUE_NO_DAEMON_OFF)
                vector_foreach_slot(vecs->mpvec, mpp, i)
                        dm_queue_if_no_path(mpp->alias, 0);
        remove_maps_and_stop_waiters(vecs);
        unlock(&vecs->lock);
 
-       pthread_cancel(check_thr);
-       pthread_cancel(uevent_thr);
-       pthread_cancel(uxlsnr_thr);
-       pthread_cancel(uevq_thr);
-       if (poll_dmevents)
-               pthread_cancel(dmevent_thr);
-
        pthread_join(check_thr, NULL);
        pthread_join(uevent_thr, NULL);
        pthread_join(uxlsnr_thr, NULL);
-- 
2.19.2

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel