* [PATCH 0/4] Various small multipath-tools improvements @ 2009-04-17 22:07 Mike Snitzer 2009-04-17 22:07 ` [PATCH 1/4] multipath-tools: update scsi_id arguments Mike Snitzer ` (4 more replies) 0 siblings, 5 replies; 10+ messages in thread From: Mike Snitzer @ 2009-04-17 22:07 UTC (permalink / raw) To: dm-devel Just a few cleanup patches. Mike Snitzer (4): multipath-tools: update scsi_id arguments multipath-tools: install libmultipath.so in $syslibdir multipath-tools: update multipathd.init.redhat multipath-tools: change default location for bindings file Makefile.inc | 1 + libmultipath/Makefile | 6 +- libmultipath/defaults.h | 4 +- libmultipath/hwtable.c | 6 +- multipath.conf.annotated | 6 +- multipath.conf.defaults | 78 ++++++++++++++++++------------------ multipath.conf.synthetic | 4 +- multipath/Makefile | 1 + multipath/multipath.conf.5 | 2 +- multipathd/multipathd.init.redhat | 30 +++++++-------- 10 files changed, 69 insertions(+), 69 deletions(-) ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/4] multipath-tools: update scsi_id arguments 2009-04-17 22:07 [PATCH 0/4] Various small multipath-tools improvements Mike Snitzer @ 2009-04-17 22:07 ` Mike Snitzer 2009-04-17 22:07 ` [PATCH 2/4] multipath-tools: install libmultipath.so in $syslibdir Mike Snitzer ` (3 subsequent siblings) 4 siblings, 0 replies; 10+ messages in thread From: Mike Snitzer @ 2009-04-17 22:07 UTC (permalink / raw) To: dm-devel Switch over to using --whitelisted and --device Signed-off-by: Mike Snitzer <snitzer@redhat.com> --- libmultipath/defaults.h | 2 +- libmultipath/hwtable.c | 6 ++-- multipath.conf.annotated | 6 ++-- multipath.conf.defaults | 78 ++++++++++++++++++++++---------------------- multipath.conf.synthetic | 4 +- multipath/multipath.conf.5 | 2 +- 6 files changed, 49 insertions(+), 49 deletions(-) diff --git a/libmultipath/defaults.h b/libmultipath/defaults.h index 87b155e..d7f93cb 100644 --- a/libmultipath/defaults.h +++ b/libmultipath/defaults.h @@ -1,4 +1,4 @@ -#define DEFAULT_GETUID "/lib/udev/scsi_id -g -u -d /dev/%n" +#define DEFAULT_GETUID "/lib/udev/scsi_id --whitelisted --device=/dev/%n" #define DEFAULT_UDEVDIR "/dev" #define DEFAULT_MULTIPATHDIR "/lib/multipath" #define DEFAULT_SELECTOR "round-robin 0" diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c index 1a51b19..52bced9 100644 --- a/libmultipath/hwtable.c +++ b/libmultipath/hwtable.c @@ -193,7 +193,7 @@ static struct hwentry default_hw[] = { /* HP Smart Array */ .vendor = "HP", .product = "LOGICAL VOLUME.*", - .getuid = "/lib/udev/scsi_id -n -g -u -d /dev/%n", + .getuid = DEFAULT_GETUID, .features = DEFAULT_FEATURES, .hwhandler = DEFAULT_HWHANDLER, .selector = DEFAULT_SELECTOR, @@ -235,7 +235,7 @@ static struct hwentry default_hw[] = { { .vendor = "EMC", .product = "SYMMETRIX", - .getuid = "/lib/udev/scsi_id -g -u -ppre-spc3-83 -d /dev/%n", + .getuid = "/lib/udev/scsi_id --page=pre-spc3-83 --whitelisted --device=/dev/%n", .features = DEFAULT_FEATURES, .hwhandler = DEFAULT_HWHANDLER, .selector = DEFAULT_SELECTOR, @@ -794,7 +794,7 @@ static struct hwentry default_hw[] = { { .vendor = "PIVOT3", .product = "RAIGE VOLUME", - .getuid = "/lib/udev/scsi_id -p 0x80 -g -u -d /dev/%n", + .getuid = "/lib/udev/scsi_id --page=0x80 --whitelisted --device=/dev/%n", .features = "1 queue_if_no_path", .hwhandler = DEFAULT_HWHANDLER, .selector = DEFAULT_SELECTOR, diff --git a/multipath.conf.annotated b/multipath.conf.annotated index 10aa5eb..7f5fe0d 100644 --- a/multipath.conf.annotated +++ b/multipath.conf.annotated @@ -55,9 +55,9 @@ # # scope : multipath # # desc : the default program and args to callout to obtain a unique # # path identifier. Absolute path required -# # default : /lib/udev/scsi_id -g -u -s +# # default : /lib/udev/scsi_id --whitelisted --device=/dev/%n # # -# getuid_callout "/lib/udev/scsi_id -g -u -d /dev/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # # # # # name : prio @@ -351,7 +351,7 @@ # # desc : the program and args to callout to obtain a unique # # path identifier. Absolute path required # # -# getuid_callout "/lib/udev/scsi_id -g -u -d /dev/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # # # # # name : prio diff --git a/multipath.conf.defaults b/multipath.conf.defaults index d2dfe2b..3651e40 100644 --- a/multipath.conf.defaults +++ b/multipath.conf.defaults @@ -6,7 +6,7 @@ # polling_interval 5 # selector "round-robin 0" # path_grouping_policy failover -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # prio const # path_checker directio # rr_min_io 1000 @@ -26,7 +26,7 @@ # device { # vendor "APPLE*" # product "Xserve RAID" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -39,7 +39,7 @@ # device { # vendor "3PARdata" # product "VV" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -52,7 +52,7 @@ # device { # vendor "DEC" # product "HSG80" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "1 hp-sw" # path_selector "round-robin 0" @@ -65,7 +65,7 @@ # device { # vendor "HP" # product "A6189A" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -79,7 +79,7 @@ # device { # vendor "(COMPAQ|HP)" # product "(MSA|HSV)1.0.*" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "1 hp-sw" # path_selector "round-robin 0" @@ -93,7 +93,7 @@ # device { # vendor "HP" # product "MSA VOLUME" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -123,7 +123,7 @@ # device { # vendor "(COMPAQ|HP)" # product "HSV1[01]1|HSV2[01]0|HSV300" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -138,7 +138,7 @@ # device { # vendor "HP" # product "MSA2[02]12*" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -153,7 +153,7 @@ # device { # vendor "HP" # product "LOGICAL VOLUME.*" -# getuid_callout "/lib/udev/scsi_id -n -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -168,7 +168,7 @@ # device { # vendor "DDN" # product "SAN DataDirector" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -181,7 +181,7 @@ # device { # vendor "EMC" # product "SYMMETRIX" -# getuid_callout "/lib/udev/scsi_id -g -u -ppre-spc3-83 -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --page=pre-spc3-83 --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -195,7 +195,7 @@ # vendor "DGC" # product ".*" # product_blacklist "LUNZ" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # prio_callout "/sbin/mpath_prio_emc /dev/%n" # features "1 queue_if_no_path" # hardware_handler "1 emc" @@ -211,7 +211,7 @@ # device { # vendor "FSC" # product "CentricStor" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -224,7 +224,7 @@ # device { # vendor "(HITACHI|HP)" # product "OPEN-.*" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -237,7 +237,7 @@ # device { # vendor "HITACHI" # product "DF.*" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -251,7 +251,7 @@ # device { # vendor "IBM" # product "ProFibre 4000R" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -264,7 +264,7 @@ # device { # vendor "IBM" # product "1722-600" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "1 rdac" # path_selector "round-robin 0" @@ -279,7 +279,7 @@ # device { # vendor "IBM" # product "1742" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "1 rdac" # path_selector "round-robin 0" @@ -294,7 +294,7 @@ # device { # vendor "IBM" # product "1814" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "1 rdac" # path_selector "round-robin 0" @@ -309,7 +309,7 @@ # device { # vendor "IBM" # product "1815" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "1 rdac" # path_selector "round-robin 0" @@ -324,7 +324,7 @@ # device { # vendor "IBM" # product "3526" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "1 rdac" # path_selector "round-robin 0" @@ -339,7 +339,7 @@ # device { # vendor "IBM" # product "3542" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -352,7 +352,7 @@ # device { # vendor "IBM" # product "2105(800|F20)" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -365,7 +365,7 @@ # device { # vendor "IBM" # product "1750500" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -379,7 +379,7 @@ # device { # vendor "IBM" # product "2107900" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -392,7 +392,7 @@ # device { # vendor "IBM" # product "2145" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -407,7 +407,7 @@ # vendor "IBM" # product "S/390 DASD ECKD" # product_blacklist "S/390.*" -# getuid_callout "/sbin/dasdinfo -u -b %n" +# getuid_callout "/sbin/dasd_id /dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -420,7 +420,7 @@ # device { # vendor "NETAPP" # product "LUN.*" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -434,7 +434,7 @@ # device { # vendor "IBM" # product "Nseries.*" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -448,7 +448,7 @@ # device { # vendor "Pillar" # product "Axiom.*" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -461,7 +461,7 @@ # device { # vendor "SGI" # product "TP9[13]00" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -474,7 +474,7 @@ # device { # vendor "SGI" # product "TP9[45]00" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "1 rdac" # path_selector "round-robin 0" @@ -489,7 +489,7 @@ # device { # vendor "SGI" # product "IS.*" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "1 rdac" # path_selector "round-robin 0" @@ -504,7 +504,7 @@ # device { # vendor "STK" # product "OPENstorage D280" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -518,7 +518,7 @@ # device { # vendor "SUN" # product "(StorEdge 3510|T4)" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "0" # path_selector "round-robin 0" @@ -531,7 +531,7 @@ # device { # vendor "PIVOT3" # product "RAIGE VOLUME" -# getuid_callout "/sbin/scsi_id -p 0x80 -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --page=0x80 --device=/dev/%n" # features "1 queue_if_no_path" # hardware_handler "0" # path_selector "round-robin 0" @@ -544,7 +544,7 @@ # device { # vendor "SUN" # product "CSM200_R" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "1 rdac" # path_selector "round-robin 0" @@ -559,7 +559,7 @@ # device { # vendor "SUN" # product "LCSM100_F" -# getuid_callout "/lib/udev/scsi_id -g -u -s /block/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # features "0" # hardware_handler "1 rdac" # path_selector "round-robin 0" diff --git a/multipath.conf.synthetic b/multipath.conf.synthetic index bf94c04..efa84f0 100644 --- a/multipath.conf.synthetic +++ b/multipath.conf.synthetic @@ -7,7 +7,7 @@ # polling_interval 10 # selector "round-robin 0" # path_grouping_policy multibus -# getuid_callout "/lib/udev/scsi_id -g -u -d /dev/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # prio const # path_checker directio # rr_min_io 100 @@ -52,7 +52,7 @@ # vendor "COMPAQ " # product "HSV110 (C)COMPAQ" # path_grouping_policy multibus -# getuid_callout "/lib/udev/scsi_id -g -u -d /dev/%n" +# getuid_callout "/lib/udev/scsi_id --whitelisted --device=/dev/%n" # path_checker directio # path_selector "round-robin 0" # hardware_handler "0" diff --git a/multipath/multipath.conf.5 b/multipath/multipath.conf.5 index c02245c..057dd83 100644 --- a/multipath/multipath.conf.5 +++ b/multipath/multipath.conf.5 @@ -109,7 +109,7 @@ Default value is \fImultibus\fR. The default program and args to callout to obtain a unique path identifier. Should be specified with an absolute path. Default value is -.I /lib/udev/scsi_id -g -u -d /dev/%n +.I /lib/udev/scsi_id --whitelisted --device=/dev/%n .TP .B prio_callout The default program and args to callout to obtain a path priority -- 1.6.0.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/4] multipath-tools: install libmultipath.so in $syslibdir 2009-04-17 22:07 [PATCH 0/4] Various small multipath-tools improvements Mike Snitzer 2009-04-17 22:07 ` [PATCH 1/4] multipath-tools: update scsi_id arguments Mike Snitzer @ 2009-04-17 22:07 ` Mike Snitzer 2009-04-19 22:10 ` Christophe Varoqui 2009-04-17 22:07 ` [PATCH 3/4] multipath-tools: update multipathd.init.redhat Mike Snitzer ` (2 subsequent siblings) 4 siblings, 1 reply; 10+ messages in thread From: Mike Snitzer @ 2009-04-17 22:07 UTC (permalink / raw) To: dm-devel Allows libmultipath.so to be installed in the proper lib dir (e.g. /lib64). $libdir is already used for /lib/multipath Signed-off-by: Mike Snitzer <snitzer@redhat.com> --- Makefile.inc | 1 + libmultipath/Makefile | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/Makefile.inc b/Makefile.inc index 3e5bca0..1fe9782 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -21,6 +21,7 @@ multipathdir = $(TOPDIR)/libmultipath mandir = $(prefix)/usr/share/man/man8 man5dir = $(prefix)/usr/share/man/man5 rcdir = $(prefix)/etc/init.d +syslibdir = $(prefix)/lib libdir = $(prefix)/lib/multipath GZIP = /bin/gzip -9 -c diff --git a/libmultipath/Makefile b/libmultipath/Makefile index 35cb911..503223d 100644 --- a/libmultipath/Makefile +++ b/libmultipath/Makefile @@ -26,12 +26,12 @@ $(LIBS): $(OBJS) $(CC) $(SHARED_FLAGS) -Wl,-soname,$@ $(CFLAGS) -o $@ $(OBJS) install: - $(INSTALL_PROGRAM) -d $(DESTDIR)$(prefix)/lib - $(INSTALL_PROGRAM) -m 755 $(LIBS) $(DESTDIR)$(prefix)/lib/$(LIBS) + $(INSTALL_PROGRAM) -d $(DESTDIR)$(syslibdir) + $(INSTALL_PROGRAM) -m 755 $(LIBS) $(DESTDIR)$(syslibdir)/$(LIBS) $(INSTALL_PROGRAM) -m 755 -d $(DESTDIR)$(libdir) uninstall: - rm -f $(DESTDIR)$(prefix)/lib/$(LIBS) + rm -f $(DESTDIR)$(syslibdir)/$(LIBS) clean: rm -f core *.a *.o *.gz *.so -- 1.6.0.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 2/4] multipath-tools: install libmultipath.so in $syslibdir 2009-04-17 22:07 ` [PATCH 2/4] multipath-tools: install libmultipath.so in $syslibdir Mike Snitzer @ 2009-04-19 22:10 ` Christophe Varoqui 2009-04-19 22:36 ` Mike Snitzer 0 siblings, 1 reply; 10+ messages in thread From: Christophe Varoqui @ 2009-04-19 22:10 UTC (permalink / raw) To: device-mapper development Le vendredi 17 avril 2009 à 18:07 -0400, Mike Snitzer a écrit : > Allows libmultipath.so to be installed in the proper lib dir > (e.g. /lib64). $libdir is already used for /lib/multipath > Mike, am I missing something ... I don't the lib/lib64 switching code. > Signed-off-by: Mike Snitzer <snitzer@redhat.com> > --- > Makefile.inc | 1 + > libmultipath/Makefile | 6 +++--- > 2 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/Makefile.inc b/Makefile.inc > index 3e5bca0..1fe9782 100644 > --- a/Makefile.inc > +++ b/Makefile.inc > @@ -21,6 +21,7 @@ multipathdir = $(TOPDIR)/libmultipath > mandir = $(prefix)/usr/share/man/man8 > man5dir = $(prefix)/usr/share/man/man5 > rcdir = $(prefix)/etc/init.d > +syslibdir = $(prefix)/lib > libdir = $(prefix)/lib/multipath > > GZIP = /bin/gzip -9 -c > diff --git a/libmultipath/Makefile b/libmultipath/Makefile > index 35cb911..503223d 100644 > --- a/libmultipath/Makefile > +++ b/libmultipath/Makefile > @@ -26,12 +26,12 @@ $(LIBS): $(OBJS) > $(CC) $(SHARED_FLAGS) -Wl,-soname,$@ $(CFLAGS) -o $@ $(OBJS) > > install: > - $(INSTALL_PROGRAM) -d $(DESTDIR)$(prefix)/lib > - $(INSTALL_PROGRAM) -m 755 $(LIBS) $(DESTDIR)$(prefix)/lib/$(LIBS) > + $(INSTALL_PROGRAM) -d $(DESTDIR)$(syslibdir) > + $(INSTALL_PROGRAM) -m 755 $(LIBS) $(DESTDIR)$(syslibdir)/$(LIBS) > $(INSTALL_PROGRAM) -m 755 -d $(DESTDIR)$(libdir) > > uninstall: > - rm -f $(DESTDIR)$(prefix)/lib/$(LIBS) > + rm -f $(DESTDIR)$(syslibdir)/$(LIBS) > > clean: > rm -f core *.a *.o *.gz *.so ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/4] multipath-tools: install libmultipath.so in $syslibdir 2009-04-19 22:10 ` Christophe Varoqui @ 2009-04-19 22:36 ` Mike Snitzer 2009-04-22 20:44 ` Christophe Varoqui 0 siblings, 1 reply; 10+ messages in thread From: Mike Snitzer @ 2009-04-19 22:36 UTC (permalink / raw) To: device-mapper development On Sun, Apr 19 2009 at 6:10pm -0400, Christophe Varoqui <christophe.varoqui@free.fr> wrote: > Le vendredi 17 avril 2009 à 18:07 -0400, Mike Snitzer a écrit : > > Allows libmultipath.so to be installed in the proper lib dir > > (e.g. /lib64). $libdir is already used for /lib/multipath > > > > Mike, > am I missing something ... I don't the lib/lib64 switching code. The standard install would still just place libmultipath.so in /lib but this patch exposes the ability to override the default e.g.: make install syslibdir=/lib64 libdir=/lib64/multipath The path checker and prioritizer modules would get installed in /lib64/multipath and libmultipath.so in /lib64 > > --- > > Makefile.inc | 1 + > > libmultipath/Makefile | 6 +++--- > > 2 files changed, 4 insertions(+), 3 deletions(-) > > > > diff --git a/Makefile.inc b/Makefile.inc > > index 3e5bca0..1fe9782 100644 > > --- a/Makefile.inc > > +++ b/Makefile.inc > > @@ -21,6 +21,7 @@ multipathdir = $(TOPDIR)/libmultipath > > mandir = $(prefix)/usr/share/man/man8 > > man5dir = $(prefix)/usr/share/man/man5 > > rcdir = $(prefix)/etc/init.d > > +syslibdir = $(prefix)/lib > > libdir = $(prefix)/lib/multipath > > > > GZIP = /bin/gzip -9 -c > > diff --git a/libmultipath/Makefile b/libmultipath/Makefile > > index 35cb911..503223d 100644 > > --- a/libmultipath/Makefile > > +++ b/libmultipath/Makefile > > @@ -26,12 +26,12 @@ $(LIBS): $(OBJS) > > $(CC) $(SHARED_FLAGS) -Wl,-soname,$@ $(CFLAGS) -o $@ $(OBJS) > > > > install: > > - $(INSTALL_PROGRAM) -d $(DESTDIR)$(prefix)/lib > > - $(INSTALL_PROGRAM) -m 755 $(LIBS) $(DESTDIR)$(prefix)/lib/$(LIBS) > > + $(INSTALL_PROGRAM) -d $(DESTDIR)$(syslibdir) > > + $(INSTALL_PROGRAM) -m 755 $(LIBS) $(DESTDIR)$(syslibdir)/$(LIBS) > > $(INSTALL_PROGRAM) -m 755 -d $(DESTDIR)$(libdir) > > > > uninstall: > > - rm -f $(DESTDIR)$(prefix)/lib/$(LIBS) > > + rm -f $(DESTDIR)$(syslibdir)/$(LIBS) > > > > clean: > > rm -f core *.a *.o *.gz *.so > > -- > dm-devel mailing list > dm-devel@redhat.com > https://www.redhat.com/mailman/listinfo/dm-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Re: [PATCH 2/4] multipath-tools: install libmultipath.so in $syslibdir 2009-04-19 22:36 ` Mike Snitzer @ 2009-04-22 20:44 ` Christophe Varoqui 0 siblings, 0 replies; 10+ messages in thread From: Christophe Varoqui @ 2009-04-22 20:44 UTC (permalink / raw) To: device-mapper development > > > > Mike, > > am I missing something ... I don't the lib/lib64 switching code. > > The standard install would still just place libmultipath.so in /lib but > this patch exposes the ability to override the default e.g.: > make install syslibdir=/lib64 libdir=/lib64/multipath > > The path checker and prioritizer modules would get installed in > /lib64/multipath and libmultipath.so in /lib64 > Oh ok, I was expecting some kind of detection code to set /lib64. I understand that this approach is sufficient for your packagers needs. Thanks for explaining. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 3/4] multipath-tools: update multipathd.init.redhat 2009-04-17 22:07 [PATCH 0/4] Various small multipath-tools improvements Mike Snitzer 2009-04-17 22:07 ` [PATCH 1/4] multipath-tools: update scsi_id arguments Mike Snitzer 2009-04-17 22:07 ` [PATCH 2/4] multipath-tools: install libmultipath.so in $syslibdir Mike Snitzer @ 2009-04-17 22:07 ` Mike Snitzer 2009-04-17 22:08 ` [PATCH 4/4] multipath-tools: change default location for bindings file Mike Snitzer 2009-04-18 17:41 ` [PATCH 0/4] Various small multipath-tools improvements christophe.varoqui 4 siblings, 0 replies; 10+ messages in thread From: Mike Snitzer @ 2009-04-17 22:07 UTC (permalink / raw) To: dm-devel Various small improvements to Red Hat's multipathd initscript. Signed-off-by: Mike Snitzer <snitzer@redhat.com> --- multipathd/multipathd.init.redhat | 30 ++++++++++++++---------------- 1 files changed, 14 insertions(+), 16 deletions(-) diff --git a/multipathd/multipathd.init.redhat b/multipathd/multipathd.init.redhat index 4664212..ee776f0 100644 --- a/multipathd/multipathd.init.redhat +++ b/multipathd/multipathd.init.redhat @@ -1,13 +1,16 @@ #!/bin/bash - -# -# /etc/rc.d/init.d/multipathd # -# Starts the multipath daemon +# multipathd Starts the multipath daemon # # chkconfig: - 13 87 -# description: Manage device-mapper multipath devices -# processname: multipathd +# description: Manages device-mapper multipath devices + +### BEGIN INIT INFO +# Provides: multipathd +# Short-Description: Control multipathd +# Description: This service monitors and manages +# device-mapper multipath devices +### END INIT INFO DAEMON=/sbin/multipathd prog=`basename $DAEMON` @@ -16,14 +19,8 @@ lockdir=/var/lock/subsys sysconfig=/etc/sysconfig -system=redhat - -if [ $system = redhat ]; then - # Source function library. - . $initdir/functions -fi +. $initdir/functions -test -x $DAEMON || exit 0 test -r $sysconfig/$prog && . $sysconfig/$prog RETVAL=0 @@ -33,6 +30,7 @@ RETVAL=0 # start() { + test -x $DAEMON || exit 5 echo -n $"Starting $prog daemon: " daemon $DAEMON RETVAL=$? @@ -68,13 +66,13 @@ start) stop) stop ;; -reload) +force-reload|reload) reload ;; restart) restart ;; -condrestart) +condrestart|try-restart) if [ -f $lockdir/$prog ]; then restart fi @@ -85,7 +83,7 @@ status) ;; *) echo $"Usage: $0 {start|stop|status|restart|condrestart|reload}" - RETVAL=1 + RETVAL=2 esac exit $RETVAL -- 1.6.0.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 4/4] multipath-tools: change default location for bindings file 2009-04-17 22:07 [PATCH 0/4] Various small multipath-tools improvements Mike Snitzer ` (2 preceding siblings ...) 2009-04-17 22:07 ` [PATCH 3/4] multipath-tools: update multipathd.init.redhat Mike Snitzer @ 2009-04-17 22:08 ` Mike Snitzer 2009-04-18 17:41 ` [PATCH 0/4] Various small multipath-tools improvements christophe.varoqui 4 siblings, 0 replies; 10+ messages in thread From: Mike Snitzer @ 2009-04-17 22:08 UTC (permalink / raw) To: dm-devel Change from /var/lib/multipath/bindings to /etc/multipath/bindings. On bootup, if /var is not on the same device as /, multipath may start up before /var is mounted. /etc will always be available. Use of /etc/multipath/ is to make selinux happy. Signed-off-by: Mike Snitzer <snitzer@redhat.com> --- libmultipath/defaults.h | 2 +- multipath/Makefile | 1 + 2 files changed, 2 insertions(+), 1 deletions(-) diff --git a/libmultipath/defaults.h b/libmultipath/defaults.h index d7f93cb..4bd0a8b 100644 --- a/libmultipath/defaults.h +++ b/libmultipath/defaults.h @@ -18,6 +18,6 @@ #define DEFAULT_PIDFILE "/var/run/multipathd.pid" #define DEFAULT_SOCKET "/var/run/multipathd.sock" #define DEFAULT_CONFIGFILE "/etc/multipath.conf" -#define DEFAULT_BINDINGS_FILE "/var/lib/multipath/bindings" +#define DEFAULT_BINDINGS_FILE "/etc/multipath/bindings" char * set_default (char * str); diff --git a/multipath/Makefile b/multipath/Makefile index e0031a2..1f48a8f 100644 --- a/multipath/Makefile +++ b/multipath/Makefile @@ -27,6 +27,7 @@ install: $(INSTALL_PROGRAM) -m 644 $(EXEC).8.gz $(DESTDIR)$(mandir) $(INSTALL_PROGRAM) -d $(DESTDIR)$(man5dir) $(INSTALL_PROGRAM) -m 644 $(EXEC).conf.5.gz $(DESTDIR)$(man5dir) + $(INSTALL_PROGRAM) -d $(DESTDIR)/etc/multipath uninstall: rm $(DESTDIR)/etc/udev/rules.d/multipath.rules -- 1.6.0.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 0/4] Various small multipath-tools improvements 2009-04-17 22:07 [PATCH 0/4] Various small multipath-tools improvements Mike Snitzer ` (3 preceding siblings ...) 2009-04-17 22:08 ` [PATCH 4/4] multipath-tools: change default location for bindings file Mike Snitzer @ 2009-04-18 17:41 ` christophe.varoqui 2009-04-18 17:53 ` Mike Snitzer 4 siblings, 1 reply; 10+ messages in thread From: christophe.varoqui @ 2009-04-18 17:41 UTC (permalink / raw) To: device-mapper development Hi Mike, thank you for these patches. Do you happen to have a fetchable git clone to ease integration ? ----- Mail Original ----- De: "Mike Snitzer" <snitzer@redhat.com> À: dm-devel@redhat.com Envoyé: Samedi 18 Avril 2009 00h07:56 GMT +01:00 Amsterdam / Berlin / Berne / Rome / Stockholm / Vienne Objet: [dm-devel] [PATCH 0/4] Various small multipath-tools improvements Just a few cleanup patches. Mike Snitzer (4): multipath-tools: update scsi_id arguments multipath-tools: install libmultipath.so in $syslibdir multipath-tools: update multipathd.init.redhat multipath-tools: change default location for bindings file Makefile.inc | 1 + libmultipath/Makefile | 6 +- libmultipath/defaults.h | 4 +- libmultipath/hwtable.c | 6 +- multipath.conf.annotated | 6 +- multipath.conf.defaults | 78 ++++++++++++++++++------------------ multipath.conf.synthetic | 4 +- multipath/Makefile | 1 + multipath/multipath.conf.5 | 2 +- multipathd/multipathd.init.redhat | 30 +++++++-------- 10 files changed, 69 insertions(+), 69 deletions(-) -- dm-devel mailing list dm-devel@redhat.com https://www.redhat.com/mailman/listinfo/dm-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 0/4] Various small multipath-tools improvements 2009-04-18 17:41 ` [PATCH 0/4] Various small multipath-tools improvements christophe.varoqui @ 2009-04-18 17:53 ` Mike Snitzer 0 siblings, 0 replies; 10+ messages in thread From: Mike Snitzer @ 2009-04-18 17:53 UTC (permalink / raw) To: device-mapper development Hi Christophe, I don't have a public git tree at the moment but I can look to get one setup for future patches. The patches that I posted should apply cleanly to the current HEAD (49944a3d5). BTW, agk pointed out that moving the bindings file may require that some logic be added to migrate the old bindings to the new location. I haven't looked into it yet but will do so shortly. But for now it is probably safest to not apply the change to the bindings file location (patch 4). Mike On Sat, Apr 18 2009 at 1:41pm -0400, christophe.varoqui@free.fr <christophe.varoqui@free.fr> wrote: > Hi Mike, > thank you for these patches. > Do you happen to have a fetchable git clone to ease integration ? > > ----- Mail Original ----- > De: "Mike Snitzer" <snitzer@redhat.com> > À: dm-devel@redhat.com > Envoyé: Samedi 18 Avril 2009 00h07:56 GMT +01:00 Amsterdam / Berlin / Berne / Rome / Stockholm / Vienne > Objet: [dm-devel] [PATCH 0/4] Various small multipath-tools improvements > > Just a few cleanup patches. > > Mike Snitzer (4): > multipath-tools: update scsi_id arguments > multipath-tools: install libmultipath.so in $syslibdir > multipath-tools: update multipathd.init.redhat > multipath-tools: change default location for bindings file > > Makefile.inc | 1 + > libmultipath/Makefile | 6 +- > libmultipath/defaults.h | 4 +- > libmultipath/hwtable.c | 6 +- > multipath.conf.annotated | 6 +- > multipath.conf.defaults | 78 ++++++++++++++++++------------------ > multipath.conf.synthetic | 4 +- > multipath/Makefile | 1 + > multipath/multipath.conf.5 | 2 +- > multipathd/multipathd.init.redhat | 30 +++++++-------- > 10 files changed, 69 insertions(+), 69 deletions(-) > > -- > dm-devel mailing list > dm-devel@redhat.com > https://www.redhat.com/mailman/listinfo/dm-devel > > -- > dm-devel mailing list > dm-devel@redhat.com > https://www.redhat.com/mailman/listinfo/dm-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2009-04-22 20:44 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2009-04-17 22:07 [PATCH 0/4] Various small multipath-tools improvements Mike Snitzer 2009-04-17 22:07 ` [PATCH 1/4] multipath-tools: update scsi_id arguments Mike Snitzer 2009-04-17 22:07 ` [PATCH 2/4] multipath-tools: install libmultipath.so in $syslibdir Mike Snitzer 2009-04-19 22:10 ` Christophe Varoqui 2009-04-19 22:36 ` Mike Snitzer 2009-04-22 20:44 ` Christophe Varoqui 2009-04-17 22:07 ` [PATCH 3/4] multipath-tools: update multipathd.init.redhat Mike Snitzer 2009-04-17 22:08 ` [PATCH 4/4] multipath-tools: change default location for bindings file Mike Snitzer 2009-04-18 17:41 ` [PATCH 0/4] Various small multipath-tools improvements christophe.varoqui 2009-04-18 17:53 ` Mike Snitzer
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.