multipathd: use 0 for initial pg if there are no priority groups
authorMike Snitzer <snitzer@redhat.com>
Wed, 20 Apr 2011 21:24:00 +0000 (17:24 -0400)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Thu, 21 Apr 2011 04:34:17 +0000 (06:34 +0200)
dm-multipath now accommodates a DM table that doesn't have any priority
groups, see: http://git.kernel.org/linus/a490a07a67b7a37

That kernel change imposes that multipathd must pass 0 for the initial
priority group if there aren't any priority groups (currently passes 1).

There is no need to check the multipath target version to decide whether
to pass an initial_pg of 0 or 1 because until recently dm-multipath
never accommodated a DM table with no priority groups.

Signed-off-by: Mike Snitzer <snitzer@redhat.com>
libmultipath/dmparser.c

index 1ef3aad..0803bec 100644 (file)
@@ -52,6 +52,7 @@ assemble_map (struct multipath * mp)
        int i, j;
        int shift, freechar;
        int minio;
        int i, j;
        int shift, freechar;
        int minio;
+       int nr_priority_groups, initial_pg_nr;
        char * p;
        struct pathgroup * pgp;
        struct path * pp;
        char * p;
        struct pathgroup * pgp;
        struct path * pp;
@@ -60,9 +61,12 @@ assemble_map (struct multipath * mp)
        p = mp->params;
        freechar = sizeof(mp->params);
 
        p = mp->params;
        freechar = sizeof(mp->params);
 
+       nr_priority_groups = VECTOR_SIZE(mp->pg);
+       initial_pg_nr = (nr_priority_groups ? mp->bestpg : 0);
+
        shift = snprintf(p, freechar, "%s %s %i %i",
                         mp->features, mp->hwhandler,
        shift = snprintf(p, freechar, "%s %s %i %i",
                         mp->features, mp->hwhandler,
-                        VECTOR_SIZE(mp->pg), mp->bestpg);
+                        nr_priority_groups, initial_pg_nr);
 
        if (shift >= freechar) {
                fprintf(stderr, "mp->params too small\n");
 
        if (shift >= freechar) {
                fprintf(stderr, "mp->params too small\n");