Improve logging for orphan_path()
authorHannes Reinecke <hare@suse.de>
Fri, 15 Nov 2013 10:29:32 +0000 (11:29 +0100)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Sun, 17 Nov 2013 17:26:31 +0000 (18:26 +0100)
orphan_path() is called from various sections, so add a
description to the call to aid debugging.

Signed-off-by: Hannes Reinecke <hare@suse.de>
libmultipath/configure.c
libmultipath/structs_vec.c
libmultipath/structs_vec.h
multipathd/main.c

index 7f5e065..aad3c9f 100644 (file)
@@ -518,7 +518,7 @@ coalesce_paths (struct vectors * vecs, vector newmp, char * refwwid, int force_r
                /* 1. if path has no unique id or wwid blacklisted */
                if (memcmp(empty_buff, pp1->wwid, WWID_SIZE) == 0 ||
                    filter_path(conf, pp1) > 0) {
-                       orphan_path(pp1);
+                       orphan_path(pp1, "wwid blacklisted");
                        continue;
                }
 
@@ -528,7 +528,7 @@ coalesce_paths (struct vectors * vecs, vector newmp, char * refwwid, int force_r
 
                /* 3. if path has disappeared */
                if (!pp1->size) {
-                       orphan_path(pp1);
+                       orphan_path(pp1, "invalid size");
                        continue;
                }
 
index 993d3e0..14ea1c7 100644 (file)
@@ -77,8 +77,9 @@ adopt_paths (vector pathvec, struct multipath * mpp, int get_info)
 }
 
 extern void
-orphan_path (struct path * pp)
+orphan_path (struct path * pp, const char *reason)
 {
+       condlog(3, "%s: orphan path, %s", pp->dev, reason);
        pp->mpp = NULL;
        pp->dmstate = PSTATE_UNDEF;
        pp->uid_attribute = NULL;
@@ -98,8 +99,7 @@ orphan_paths (vector pathvec, struct multipath * mpp)
 
        vector_foreach_slot (pathvec, pp, i) {
                if (pp->mpp == mpp) {
-                       condlog(4, "%s: orphaned", pp->dev);
-                       orphan_path(pp);
+                       orphan_path(pp, "map flushed");
                }
        }
 }
index a907e85..c6278ac 100644 (file)
@@ -17,7 +17,7 @@ void set_no_path_retry(struct multipath *mpp);
 
 int adopt_paths (vector pathvec, struct multipath * mpp, int get_info);
 void orphan_paths (vector pathvec, struct multipath * mpp);
-void orphan_path (struct path * pp);
+void orphan_path (struct path * pp, const char *reason);
 
 int verify_paths(struct multipath * mpp, struct vectors * vecs, vector rpvec);
 int update_mpp_paths(struct multipath * mpp, vector pathvec);
index 8681aa2..91d7bfc 100644 (file)
@@ -139,6 +139,7 @@ coalesce_maps(struct vectors *vecs, vector nmpv)
        int j;
 
        vector_foreach_slot (ompv, ompp, i) {
+               condlog(3, "%s: coalesce map", ompp->alias);
                if (!find_mp_by_wwid(nmpv, ompp->wwid)) {
                        /*
                         * remove all current maps not allowed by the
@@ -229,7 +230,7 @@ flush_map(struct multipath * mpp, struct vectors * vecs)
        }
        else {
                dm_lib_release();
-               condlog(2, "%s: devmap removed", mpp->alias);
+               condlog(2, "%s: map flushed", mpp->alias);
        }
 
        orphan_paths(vecs->pathvec, mpp);
@@ -561,7 +562,7 @@ rescan:
 fail_map:
        remove_map(mpp, vecs, 1);
 fail:
-       orphan_path(pp);
+       orphan_path(pp, "failed to add path");
        return 1;
 }