On Tue, 2023-01-31 at 13:34 -0600, Benjamin Marzinski wrote: > Since we have a function to set the action to reload, use it. > > Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx> Reviewed-by: Martin Wilck <mwilck@xxxxxxxx> > --- > libmultipath/configure.c | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git a/libmultipath/configure.c b/libmultipath/configure.c > index e689f8a7..050b984a 100644 > --- a/libmultipath/configure.c > +++ b/libmultipath/configure.c > @@ -729,9 +729,7 @@ void select_action (struct multipath *mpp, const > struct _vector *curmp, > > if (force_reload) { > mpp->force_udev_reload = 1; > - mpp->action = ACT_RELOAD; > - condlog(3, "%s: set ACT_RELOAD (forced by user)", > - mpp->alias); > + select_reload_action(mpp, "forced by user"); > return; > } > if (cmpp->size != mpp->size) { > @@ -744,9 +742,7 @@ void select_action (struct multipath *mpp, const > struct _vector *curmp, > > if (!is_udev_ready(cmpp) && count_active_paths(mpp) > 0) { > mpp->force_udev_reload = 1; > - mpp->action = ACT_RELOAD; > - condlog(3, "%s: set ACT_RELOAD (udev incomplete)", > - mpp->alias); > + select_reload_action(mpp, "udev incomplete"); > return; > } > -- dm-devel mailing list dm-devel@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/dm-devel