| Message ID | 1487045213-15776-5-git-send-email-bmarzins@redhat.com (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Delegated to: | christophe varoqui |
| Headers | show |
diff --git a/multipathd/main.c b/multipathd/main.c index adc3258..5480e96 100644 --- a/multipathd/main.c +++ b/multipathd/main.c @@ -1457,7 +1457,8 @@ int update_prio(struct path *pp, int refresh_all) } oldpriority = pp->priority; conf = get_multipath_config(); - pathinfo(pp, conf, DI_PRIO); + if (pp->state != PATH_DOWN) + pathinfo(pp, conf, DI_PRIO); put_multipath_config(conf); if (pp->priority == oldpriority)
Multipathd shouldn't be updating the priority of failed paths in the checkerloop. The current avoids this in almost all cases, but not all. Close the loophole. Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com> --- multipathd/main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)