[dm-devel] configlet specifications being ignored in multipath-tools-0.4.3

[lan <transter@xxxxxxxxx>]

For multipath-tools-0.4.3, when running 'multipath' to setup multipath
devices, it appears that some configlet specifications are being
ignored because the select_getXXX calls (e.g. select_getprio,
select_checkfn, select_getuid) made in devinfo assume that the path's
hwe pointer is already initialized, but this does not happen until
after devinfo is called.  Thus, the path ends up being intialized with
the default values instead of configlet-specificed values.  The
following change appears to fix this problem.  Please excuse me if
this problem has already been resolved.

Thanks!
Lan

diff -Naur multipath-tools-0.4.3/multipath/devinfo.c
multipath-tools-0.4.3-patched/multipath/devinfo.c
--- multipath-tools-0.4.3/multipath/devinfo.c   2005-03-04
16:32:41.000000000 -0800
+++ multipath-tools-0.4.3-patched/multipath/devinfo.c   2005-03-23
03:49:13.000000000 -0800
@@ -422,6 +422,9 @@
        pp->claimed = get_claimed(pp->dev_t);
        dbg("claimed = %i", pp->claimed);

+       /* get and store hwe pointer */
+        pp->hwe = find_hwe(conf->hwtable, pp->vendor_id, pp->product_id);
+
        /*
         * get path state, no message collection, no context
         */
diff -Naur multipath-tools-0.4.3/multipath/main.c
multipath-tools-0.4.3-patched/multipath/main.c
--- multipath-tools-0.4.3/multipath/main.c      2005-03-19
05:44:11.000000000 -0800
+++ multipath-tools-0.4.3-patched/multipath/main.c      2005-03-23
00:16:31.000000000 -0800
@@ -157,8 +157,6 @@
                        free(curpath);
                        continue;
                }
-               curpath->hwe = find_hwe(conf->hwtable, curpath->vendor_id,
-                                                     curpath->product_id);
                vector_alloc_slot(pathvec);
                vector_set_slot(pathvec, curpath);
        }