[dm-devel] [PATCH 23/42] Path checker should return PATH_DOWN when no path is found

Hannes Reinecke hare at suse.de
Tue Jan 8 13:54:01 UTC 2013


If the path checker fails to lookup the path in sysfs it's
already gone, so we should rather return 'PATH_DOWN' here.
Otherwise the path will never marked failed and no failover
will happen.

Signed-off-by: Hannes Reinecke <hare at suse.de>
---
 libmultipath/discovery.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c
index 6848b70..7ed8556 100644
--- a/libmultipath/discovery.c
+++ b/libmultipath/discovery.c
@@ -659,11 +659,11 @@ path_offline (struct path * pp)
 
 	if (!parent) {
 		condlog(1, "%s: failed to get sysfs information", pp->dev);
-		return PATH_WILD;
+		return PATH_DOWN;
 	}
 
 	if (sysfs_get_state(parent, buff, SCSI_STATE_SIZE))
-		return PATH_WILD;
+		return PATH_DOWN;
 
 	condlog(3, "%s: path state = %s", pp->dev, buff);
 
-- 
1.7.4.2




More information about the dm-devel mailing list