We will change the data structure initialization to a lazy
approach, where pp->udev isn't necessarily initialized
when get_mpvec() is called. Deal with it.

Signed-off-by: Martin Wilck <[email protected]>
---
 libmpathpersist/mpath_persist.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/libmpathpersist/mpath_persist.c b/libmpathpersist/mpath_persist.c
index 6505774f..fee1db72 100644
--- a/libmpathpersist/mpath_persist.c
+++ b/libmpathpersist/mpath_persist.c
@@ -16,6 +16,7 @@
 #include "config.h"
 #include "switchgroup.h"
 #include "discovery.h"
+#include "configure.h"
 #include "dmparser.h"
 #include <ctype.h>
 #include "propsel.h"
@@ -96,6 +97,17 @@ updatepaths (struct multipath * mpp)
                                continue;
                        }
                        pp->mpp = mpp;
+                       if (pp->udev == NULL) {
+                               pp->udev = get_udev_device(pp->dev_t, DEV_DEVT);
+                               if (pp->udev == NULL) {
+                                       pp->state = PATH_DOWN;
+                                       continue;
+                               }
+                               conf = get_multipath_config();
+                               pathinfo(pp, conf, DI_SYSFS|DI_CHECKER);
+                               put_multipath_config(conf);
+                               continue;
+                       }
                        if (pp->state == PATH_UNCHECKED ||
                                        pp->state == PATH_WILD) {
                                conf = get_multipath_config();
-- 
2.21.0

--
dm-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to