We map PATH_TIMEOUT to PATH_DOWN in pathinfo(), but not in get_new_state().
Do it there, too, to treat the states consistently.
This avoids logging "checker timed out" twice in update_path_state(), even
if log_checker_err is set to "once".

Signed-off-by: Martin Wilck <[email protected]>
---
 multipathd/main.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/multipathd/main.c b/multipathd/main.c
index d87c7b1..9bce4d6 100644
--- a/multipathd/main.c
+++ b/multipathd/main.c
@@ -2503,8 +2503,10 @@ get_new_state(struct path *pp)
         * Wait for uevent for removed paths;
         * some LLDDs like zfcp keep paths unavailable
         * without sending uevents.
+        * Also, map PATH_TIMEOUT to PATH_DOWN here, like we do in
+        * pathinfo().
         */
-       if (newstate == PATH_REMOVED)
+       if (newstate == PATH_REMOVED || newstate == PATH_TIMEOUT)
                newstate = PATH_DOWN;
 
        /*
-- 
2.53.0


Reply via email to