From: Martin Wilck <mwilck@xxxxxxxx>
The "hidden" and "claimed by foreign" properties take precedence
over blacklisting/whitelisting in pathinfo(). We should take
this into account when printing the device list.
Signed-off-by: Martin Wilck <mwilck@xxxxxxxx>
---
libmultipath/print.c | 25 ++++++++++++++++++-------
1 file changed, 18 insertions(+), 7 deletions(-)
diff --git a/libmultipath/print.c b/libmultipath/print.c
index fb94f86..ea95d69 100644
--- a/libmultipath/print.c
+++ b/libmultipath/print.c
@@ -30,6 +30,7 @@
#include "debug.h"
#include "discovery.h"
#include "util.h"
+#include "foreign.h"
#define MAX(x,y) (((x) > (y)) ? (x) : (y))
#define MIN(x,y) (((x) > (y)) ? (y) : (x))
@@ -2061,13 +2062,23 @@ int snprint_devices(struct config *conf, char *buff, size_t len,
pp = find_path_by_dev(vecs->pathvec, devname);
if (!pp) {
- r = filter_devnode(conf->blist_devnode,
- conf->elist_devnode,
- devname);
- if (r > 0)
- status = "devnode blacklisted, unmonitored";
- else
- status = "devnode whitelisted, unmonitored";
+ const char *hidden;
+
+ hidden = udev_device_get_sysattr_value(u_dev,
+ "hidden");
+ if (hidden && !strcmp(hidden, "1"))
+ status = "hidden, unmonitored";
+ else if (is_claimed_by_foreign(u_dev))
+ status = "foreign, monitored";
+ else {
+ r = filter_devnode(conf->blist_devnode,
+ conf->elist_devnode,
+ devname);
+ if (r > 0)
+ status = "devnode blacklisted, unmonitored";
+ else
+ status = "devnode whitelisted, unmonitored";
+ }
} else
status = " devnode whitelisted, monitored";
--
2.26.2
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel
