libmultipath: move remove_map waiter code to multipathd
authorBenjamin Marzinski <bmarzins@redhat.com>
Wed, 14 Mar 2018 17:46:40 +0000 (12:46 -0500)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Tue, 27 Mar 2018 20:20:22 +0000 (22:20 +0200)
Only multipathd needs to worry about the multipath waiter code. There is
no point in having remove_map_and_stop_waiter() or
remove_maps_and_stop_waiters() in libmultipath, since they should never
be use outside of multipathd.

Reviewed-by: Martin Wilck <mwilck@suse.com>
Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
libmultipath/structs_vec.c
libmultipath/structs_vec.h
multipathd/main.c

index f9dc8a8..41e5d88 100644 (file)
@@ -117,24 +117,15 @@ set_multipath_wwid (struct multipath * mpp)
        dm_get_uuid(mpp->alias, mpp->wwid);
 }
 
-#define KEEP_WAITER 0
-#define STOP_WAITER 1
 #define PURGE_VEC 1
 
-static void
-_remove_map (struct multipath * mpp, struct vectors * vecs,
-           int stop_waiter, int purge_vec)
+void
+remove_map(struct multipath * mpp, struct vectors * vecs, int purge_vec)
 {
        int i;
 
        condlog(4, "%s: remove multipath map", mpp->alias);
 
-       /*
-        * stop the DM event waiter thread
-        */
-       if (stop_waiter)
-               stop_waiter_thread(mpp, vecs);
-
        /*
         * clear references to this map
         */
@@ -150,19 +141,8 @@ _remove_map (struct multipath * mpp, struct vectors * vecs,
        free_multipath(mpp, KEEP_PATHS);
 }
 
-void remove_map(struct multipath *mpp, struct vectors *vecs, int purge_vec)
-{
-       _remove_map(mpp, vecs, KEEP_WAITER, purge_vec);
-}
-
-void remove_map_and_stop_waiter(struct multipath *mpp, struct vectors *vecs,
-                               int purge_vec)
-{
-       _remove_map(mpp, vecs, STOP_WAITER, purge_vec);
-}
-
-static void
-_remove_maps (struct vectors * vecs, int stop_waiter)
+void
+remove_maps(struct vectors * vecs)
 {
        int i;
        struct multipath * mpp;
@@ -171,7 +151,7 @@ _remove_maps (struct vectors * vecs, int stop_waiter)
                return;
 
        vector_foreach_slot (vecs->mpvec, mpp, i) {
-               _remove_map(mpp, vecs, stop_waiter, 1);
+               remove_map(mpp, vecs, 1);
                i--;
        }
 
@@ -179,16 +159,6 @@ _remove_maps (struct vectors * vecs, int stop_waiter)
        vecs->mpvec = NULL;
 }
 
-void remove_maps(struct vectors *vecs)
-{
-       _remove_maps(vecs, KEEP_WAITER);
-}
-
-void remove_maps_and_stop_waiters(struct vectors *vecs)
-{
-       _remove_maps(vecs, STOP_WAITER);
-}
-
 void
 extract_hwe_from_path(struct multipath * mpp)
 {
index 3749eb6..2a0be93 100644 (file)
@@ -27,9 +27,7 @@ int update_multipath_strings (struct multipath *mpp, vector pathvec,
 void extract_hwe_from_path(struct multipath * mpp);
 
 void remove_map (struct multipath * mpp, struct vectors * vecs, int purge_vec);
-void remove_map_and_stop_waiter (struct multipath * mpp, struct vectors * vecs, int purge_vec);
 void remove_maps (struct vectors * vecs);
-void remove_maps_and_stop_waiters (struct vectors * vecs);
 
 void sync_map_state (struct multipath *);
 int update_map (struct multipath *mpp, struct vectors *vecs);
index ce914ab..32cd6fe 100644 (file)
@@ -293,6 +293,29 @@ switch_pathgroup (struct multipath * mpp)
                 mpp->alias, mpp->bestpg);
 }
 
+static void
+remove_map_and_stop_waiter(struct multipath *mpp, struct vectors *vecs,
+                          int purge_vec)
+{
+       stop_waiter_thread(mpp, vecs);
+       remove_map(mpp, vecs, purge_vec);
+}
+
+static void
+remove_maps_and_stop_waiters(struct vectors *vecs)
+{
+       int i;
+       struct multipath * mpp;
+
+       if (!vecs)
+               return;
+
+       vector_foreach_slot(vecs->mpvec, mpp, i)
+               stop_waiter_thread(mpp, vecs);
+
+       remove_maps(vecs);
+}
+
 static int
 coalesce_maps(struct vectors *vecs, vector nmpv)
 {