From: Hannes Reinecke <hare@suse.de>
Date: Wed, 21 Jan 2009 14:01:38 +0000 (+0100)
Subject: More debugging output when synchronizing path states
X-Git-Tag: upstream/0.5.0~106^2~18
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6df6f4fb158132d90a9a228ac5b47f3571a7f14d;p=platform%2Fupstream%2Fmultipath-tools.git

More debugging output when synchronizing path states

When synchronizing path state we might end up removing a path.
However, if this path is not in state PATH_DOWN there is an
error somewhere. So modify the messages accordingly.

Signed-off-by: Hannes Reinecke <hare@suse.de>
---

diff --git a/libmultipath/structs_vec.c b/libmultipath/structs_vec.c
index fa48e00..e07b5fd 100644
--- a/libmultipath/structs_vec.c
+++ b/libmultipath/structs_vec.c
@@ -471,8 +471,15 @@ verify_paths(struct multipath * mpp, struct vectors * vecs, vector rpvec)
 		 */
 		if (!pp->sysdev || sysfs_get_dev(pp->sysdev,
 						 pp->dev_t, BLK_DEV_SIZE)) {
-			condlog(0, "%s: failed to access path %s", mpp->alias,
-				pp->sysdev ? pp->sysdev->devpath : pp->dev_t);
+			if (pp->state != PATH_DOWN) {
+				condlog(1, "%s: removing valid path %s in state %d",
+					mpp->alias,
+					pp->sysdev?pp->sysdev->devpath:pp->dev_t, pp->state);
+			} else {
+				condlog(3, "%s: failed to access path %s",
+					mpp->alias,
+					pp->sysdev ? pp->sysdev->devpath : pp->dev_t);
+			}
 			count++;
 			vector_del_slot(mpp->paths, i);
 			i--;