Message ID | 1518043787-7066-5-git-send-email-bmarzins@redhat.com (mailing list archive) |
---|---|
State | Not Applicable, archived |
Delegated to: | Mike Snitzer |
Headers | show |
On Wed, 2018-02-07 at 16:49 -0600, Benjamin Marzinski wrote: > configure() is always called with start_waiters=1, so there is no > point > in having the parameter. Remove it. > > Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com> > --- > multipathd/main.c | 12 +++++------- > 1 file changed, 5 insertions(+), 7 deletions(-) Reviewed-by: Martin Wilck <mwilck@suse.com>
diff --git a/multipathd/main.c b/multipathd/main.c index cf76241..233d09f 100644 --- a/multipathd/main.c +++ b/multipathd/main.c @@ -1950,7 +1950,7 @@ checkerloop (void *ap) } int -configure (struct vectors * vecs, int start_waiters) +configure (struct vectors * vecs) { struct multipath * mpp; struct path * pp; @@ -2049,11 +2049,9 @@ configure (struct vectors * vecs, int start_waiters) i--; continue; } - if (start_waiters) { - if (start_waiter_thread(mpp, vecs)) { - remove_map(mpp, vecs, 1); - i--; - } + if (start_waiter_thread(mpp, vecs)) { + remove_map(mpp, vecs, 1); + i--; } } return 0; @@ -2120,7 +2118,7 @@ reconfigure (struct vectors * vecs) rcu_assign_pointer(multipath_conf, conf); call_rcu(&old->rcu, rcu_free_config); - configure(vecs, 1); + configure(vecs); return 0;
configure() is always called with start_waiters=1, so there is no point in having the parameter. Remove it. Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com> --- multipathd/main.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-)