libmultipath: __setup_multipath param cleanup
authorBenjamin Marzinski <bmarzins@redhat.com>
Thu, 7 Dec 2017 18:49:02 +0000 (12:49 -0600)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Sat, 13 Jan 2018 09:14:57 +0000 (10:14 +0100)
setup multipath is only called by the daemon, so there is no point in
the is_daemon parameter.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
libmultipath/structs_vec.c
libmultipath/structs_vec.h

index 70fb005..814a4b9 100644 (file)
@@ -336,7 +336,7 @@ void set_no_path_retry(struct config *conf, struct multipath *mpp)
 }
 
 int __setup_multipath(struct vectors *vecs, struct multipath *mpp,
 }
 
 int __setup_multipath(struct vectors *vecs, struct multipath *mpp,
-                     int reset, int is_daemon)
+                     int reset)
 {
        struct config *conf;
 
 {
        struct config *conf;
 
@@ -352,7 +352,7 @@ int __setup_multipath(struct vectors *vecs, struct multipath *mpp,
                goto out;
        }
 
                goto out;
        }
 
-       if (update_multipath_strings(mpp, vecs->pathvec, is_daemon)) {
+       if (update_multipath_strings(mpp, vecs->pathvec, 1)) {
                condlog(0, "%s: failed to setup multipath", mpp->alias);
                goto out;
        }
                condlog(0, "%s: failed to setup multipath", mpp->alias);
                goto out;
        }
@@ -594,7 +594,7 @@ int update_multipath (struct vectors *vecs, char *mapname, int reset)
                return 2;
        }
 
                return 2;
        }
 
-       if (__setup_multipath(vecs, mpp, reset, 1))
+       if (__setup_multipath(vecs, mpp, reset))
                return 1; /* mpp freed in setup_multipath */
 
        /*
                return 1; /* mpp freed in setup_multipath */
 
        /*
index 54865d7..04fa0dd 100644 (file)
@@ -20,8 +20,8 @@ void orphan_path (struct path * pp, const char *reason);
 int verify_paths(struct multipath * mpp, struct vectors * vecs);
 int update_mpp_paths(struct multipath * mpp, vector pathvec);
 int __setup_multipath (struct vectors * vecs, struct multipath * mpp,
 int verify_paths(struct multipath * mpp, struct vectors * vecs);
 int update_mpp_paths(struct multipath * mpp, vector pathvec);
 int __setup_multipath (struct vectors * vecs, struct multipath * mpp,
-                      int reset, int is_daemon);
-#define setup_multipath(vecs, mpp) __setup_multipath(vecs, mpp, 1, 1)
+                      int reset);
+#define setup_multipath(vecs, mpp) __setup_multipath(vecs, mpp, 1)
 int update_multipath_strings (struct multipath *mpp, vector pathvec,
                              int is_daemon);
 void extract_hwe_from_path(struct multipath * mpp);
 int update_multipath_strings (struct multipath *mpp, vector pathvec,
                              int is_daemon);
 void extract_hwe_from_path(struct multipath * mpp);