multipath-tools: remove pointless ".*" from devnode blacklists
authorXose Vazquez Perez <xose.vazquez@gmail.com>
Mon, 3 Oct 2016 15:53:20 +0000 (17:53 +0200)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Mon, 3 Oct 2016 16:05:07 +0000 (18:05 +0200)
".*" is useless at the end of a regex, and just "*" is a bug.
And document them.

Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: device-mapper development <dm-devel@redhat.com>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
libmultipath/blacklist.c
multipath/multipath.conf.5

index 9687399..aa8d975 100644 (file)
@@ -172,7 +172,7 @@ setup_default_blist (struct config * conf)
        char * str;
        int i;
 
-       str = STRDUP("^(ram|raw|loop|fd|md|dm-|sr|scd|st)[0-9]*");
+       str = STRDUP("^(ram|raw|loop|fd|md|dm-|sr|scd|st)[0-9]");
        if (!str)
                return 1;
        if (store_ble(conf->blist_devnode, str, ORIGIN_DEFAULT))
@@ -184,19 +184,19 @@ setup_default_blist (struct config * conf)
        if (store_ble(conf->blist_devnode, str, ORIGIN_DEFAULT))
                return 1;
 
-       str = STRDUP("^dcssblk[0-9]*");
+       str = STRDUP("^dcssblk[0-9]");
        if (!str)
                return 1;
        if (store_ble(conf->blist_devnode, str, ORIGIN_DEFAULT))
                return 1;
 
-       str = STRDUP("^nvme.*");
+       str = STRDUP("^nvme");
        if (!str)
                return 1;
        if (store_ble(conf->blist_devnode, str, ORIGIN_DEFAULT))
                return 1;
 
-       str = STRDUP("(SCSI_IDENT_.*|ID_WWN)");
+       str = STRDUP("(SCSI_IDENT_|ID_WWN)");
        if (!str)
                return 1;
        if (store_ble(conf->elist_property, str, ORIGIN_DEFAULT))
index 7dd997a..313e21b 100644 (file)
@@ -849,6 +849,10 @@ The following keywords are recognized:
 .TP 17
 .B devnode
 Regular expression of the device nodes to be excluded.
+.RS
+.TP
+Default value is: \fB^(ram|raw|loop|fd|md|dm-|sr|scd|st)[0-9]\fR, \fB^(td|hd|vd)[a-z]\fR, \fB^dcssblk[0-9]\fR and \fB^nvme\fR
+.RE
 .TP
 .B wwid
 The \fIWorld Wide Identification\fR of a device.
@@ -888,7 +892,7 @@ The \fIWorld Wide Identification\fR of a device.
 Regular expression of the udev property to be whitelisted.
 .RS
 .TP
-Default value is: \fB(ID_WWN|SCSI_IDENT_.*)\fR
+Default value is: \fB(SCSI_IDENT_|ID_WWN)\fR
 .RE
 .TP
 .B device