[multipath] add '-h' option

Calling 'multipath -h' always added the line 'Unknown switch: (null)' in
front of the usage text since '-h' was not defined.
This patch just adds the '-h' option.

Volker Sameske, IBM

[prio] ALUA prioritizer broken for paths with high minor

Here's a patch which I think will allow mpath_prio_alua to
work better with device files with minor numbers >= 256.

Edward Goggin, EMC

[libmultipath] remove some more absolute path in hwtable

Hannes sent a patch to remove one for IBM 3526.
Go a bit further, removing all absolute paths.

[multipath] disabling lock_fs call on DM_DEVICE_RESUME libdevmapper ioctl

This patch to dm_simplecmd of multipath-tools/libmultipath/devmapper.c will
disable the use of lock_fs during I/O suspension while changing
dm-multipath
device mapper maps via the DM_DEVICE_RESUME ioctl of libdevmapper.

Disabling lock_fs will result in not flushing I/O to the dm-multipath block
device before changing the maps for the mapped device.  Flushing I/O at
this
time appears not only unnecessary for dm-multipath devices but causes some
troublesome delays due to SCSI HBA driver target-side timeouts when these
maps are changed as a result of paths coming and going while upgrading
software (a non-disruptive ucode upgrade) on a storage system.

As is the case for any change which is dependent on similar changes in
libdevmapper and the kernel, this patch will introduce a dependency on
libdevmapper and kernel.  Not sure which versions first had this support
but
at least libdevmapper 1.02.02 and linux kernel version 2.6.18-rc2 have the
required support.

Edward Goggin, EMC

[multipath] Fixup return code for 'multipath -l'

'multipath -l' should really set the return code to '0' if no error
occurred.

Hannes Reinecke, Suse

[multipath] remove obsolete '-D' option

our documentation folks complained that the manpage documents an
obsolete / unused '-D' option for multipath.

This patch removes this description from the manpage.

Hannes Reinecke, Suse

[libmultipath] Fixup hwtable regexp

appearently no-one noticed that we've switched to regular expressions
instead of shell-style fnmatch for the hwtable.
Quite surprisingly, really, that no-one complained; the change has been
done in October 2005.

Anyway, this patch cleans it up. And also splits off the IBM DS6000 and
IBM SVC, as they are separate machines.

Hannes Reinecke, Suse

[priority] disable debug output for pp_tpc

this patch removes the debug output for pp_tpc and also decreases the
priority ratings so that the passive path will have a priority of 0.

Hannes Reinecke, Suse

[libmultipath] add support for IBM 3526

we've found another IBM storage server in our lab; this patch adds the
necessary hwtable entry for it.

However, this is an RDAC machine, so it'll work only in some cases. One
day we really need an RDAC hw-handler.

Hannes Reinecke, Suse

[checkers] EMC checker to report LUNZ as failing (2)

Fix braino while applying patch.

[checkers] EMC checker to report LUNZ as failing

This patch simply reports a PATH_DOWN status for paths to an EMC CLARiiON
LUNZ (or an unbound) block device since these paths cannot possibly be used
to issue I/O to such a device.  In fact, allowing block I/O to a LUNZ
dm-multipath device will cause the process to hang until the no_path_retry
timeout is incurred.  This is not desirable.

Ed Goggin, EMC

[libmultipath] make sure to terminate strftime buff properly

Bernhard Fischer

[libmultipath] allow for non-default DESTDIR in hwtable

Bernhard Fischer

[libmultipath] make sure that the callout program picks up the PATH from
the invoking shell

Bernhard Fischer

[multipathd] sweets

- vector_foreach_slot takes an unsigned i
- don't use old-style initializer for setscheduler's sched_param

Bernhard Fischer

[typo] s/controler/controller/g

Bernhard Fischer

[libmultipath] move newline to dlog format parameter

Bernhard Fischer

[man] update date of manpages

Bernhard Fischer

[libmultipath] rename the Hitachi prioritizer in defaults table

Matthias Rudolph, HDS

[libmultipath] fix 'multipath -d' behaviour

Volker Sameske noted 'multipath -d' was displaying the same info multiple
times, and listed paths which should be listed.

Fix that.

[multipathd] get the layout right with "sho topo" CLI commands

[libmultipath] snprint_size tweaks

Prefer 1.8T to 1824G, like "GNU df".
Make units knowledge easily expandable.
Push the logic to 'P', for petabyte, in this patch.

[kpartx][libmultipath] make multipath and kpartx prefix the uuid

In order for other programs to figure out who owns the dm-devices that
are created by multipath and kpartx, this patch adds a prefix to the uuid
for all multipath and kpartx created devices.  When multipath gets the
uuid,
the prefix is stripped back off. However other programs can use the device
mapper library or dmsetup to easily check which devices were created by
multipath or kpartx.

the multipath prefix is mpath-
the kpartx prefix is part%d- where %d is the partition number.

Ben Marzinski, Redhat

Finish up the work on SG_IO timeout

all over the place.

[libcheckers] workaround for rather profound path checking issue

Overloaded storage response time can be very long.
SG_IO timouts after DEF_TIMEOUT milliseconds, and checkers interprets this
as a path failure. multipathd then proactively evicts the path from the DM
multipath table in this case.

This generaly snow balls and ends up in full eviction and IO errors for end
users. Bad. This may also cause SCSI bus resets, causing disruption for all
local and external storage hardware users.

Provision a long timeout. Longer than any real-world application would cope
with.

If someone can suggest a real fix, like an evidence the ioctl timed out,
voice up. The ioctl being blocking, I'd rather lower the timeout and retry
than apply this workaround.

[libmultipath] print dm-* sysfs device name in "show topo"

Useful info for admins.

Also fix a small bug in print_multipath() not reseting a buffer upon
iteration.

[multipath] getopt cleanup

'q' and 'i' are never used, so don't accept them as valid options

Benjamin Marzinski, Redhat

[libmultipath] fix hwtable brainos

Matthias Rudolph remembered me the most simple regex rules. Shame.

[libmultipath] fix segv in load_config()

segv when a vendor or product string was not set in the
configuration file.

Note that now

device {
vendor DCG
}

means :

device {
vendor DCG
product *
}

[libmultipath] print the multipath strings in "show topo"

Helps usability in complex SAN setups.

[libmultipath] '\0' terminate strings from sysfs after the last char

Makes for fancier vendor/product/rev strings for example.
And generally makes sense.

[libmultipath] hwtable maintenance

Mostly tweaks to the Hitachi products, from Matthias Rudolph.

[libmultipath] fix select_alias weirdness

In select_alias, the code that tries to pull the alias from dm_get_name()
sets mp->alias. So even if it fails,  mp->alias will never get set to
mp->wwid.

Benjamin Marzinski, Redhat

[libmultipath] multipath size output in line with "df -h"

[size=XX GB] replaced with [size=XXG] for example.

[libmultipath] pick up existing alias (if any) from current dm config

This small patch should allow multipath(8) and multipathd(8) to acquire its
internal alias name from the device-mapper's device-name for the device if a
multipath specific alias (user friendly one or not) has __not__ been set.
Otherwise, assuming that a multipath device has been renamed beforehand by
someone invoking "dm_setup rename", simply invoking "multipath" changes the
device-mapper's device-name for the device back to the multipath assigned
UID name.

Edward Goggin, EMC

[build] provision for custom versioning

apply custom build/rpm versioning for multipath-tools

Brian Wong, Pillardata

[libmultipath] add Apple Xserve defaults

I tested Apple's Xserve RAID with the multipath tools.

Here is the entry I had to make in the config file, if
one wants to add it to the dafault table:

Shyam Sundar.

[priority] pp_alua correctness fix

Tom Kalus noted path_priority/pp_aula/spc.h misrepresented to spec
regarding the product string lentgh. Credits for the fix too.

Fix acked by Stefan Bader.

[priority] add new HDS Modular prioritizer

Written by Matthias Rudolph, HDS.

[libmultipath] push WWID_SIZE to 128

64 was too small for "scsi_id -p 0x80" in most cases.

As a consequence, the tools refuse to get the uid at all and the path was
discarded ... which is still the good behaviour.

The problem tracking was done by Tarik Sedlarevic, Visionbytes.

[libmultipath] fix segfault when no config file

Brian Wong noticed the segfault and tracked it down to free_keywords()
trying to walk a NULL keywords vector.  Credits.

[libmultipath] get_serial overflow fix

Arsene Gschwind tracked down this bug. Credits.

The patch also normalize the return codes, even if they are not
used yet.

[libmultipath] remove harmless duplicated code

[libmultipath] update netapp and IBM nseries (netapp) hwtable.c

David Wysochanski, Netapp

[multipath] cosmetical fix

I'd like to ask some cosmetical fix in libmultipath.
In Redhat /etc/rc.sysinit calls /sbin/multipath -v 0 after modprobe
dm-multipath to catch the disks already attached.

Now it prints a small garbage on screen for every map, I think.
Actually every multipath -F; multipath/multipath -v 0 sequence does
that.

The reason is neither print_multipath_topology nor
snprint_multipath_topology initialize buff if verbosity is 0.

Maxim Kozover

[multipathd] some more output coherency

[libmultipath] plug the keywords vector leak upon "reconfigure"

[multipathd] plug a small leak on "reconfigure" CLI command

[multipathd] don't set the SIGKILL signal

Valgrind told me it was useless.

[libmultipath] compilation fix

waiter.h should be empty when libmultipath is built for multipath(8)
so preprocess its content.

[multipathd] ignore SIGPIPE

Maxim Kozover reported daemon segfault when breaking the recieving side
of the unix socket. Credits.

[libmultipath] NetApp maintainer change

[libmultipath] a bit of code reorganisation

o locking primitives moved to libmultipath/lock.[ch]
o waiter threads control primitives moved to libmultipath/waiter.[ch]
o update_multipath() and queue_mode_{add,del}_path() moved to
libmultipath/structs_vec.c

[libmultipath] move the tools' version string to the lib

o Remove multipath/main.h
o Print the version string in the CLI syntax reference

[build] minor compilation issues

SuSE checker found some minor issues:

- refwwid in libmultipath/configure.c:get_refwwid() might be used
uninitialized. As dev_type is a simple 'int' there is nothing which
prevents the unsuspecting user to call it with an unhandled value.
We shoud rather use an enum and set refwwid to NULL to be on the safe
side.

- attr in libmultipath/discovery.c:devt2devname() might be used
uninitialized. Quite unlikely, but might happen in principle.
So better have it initialized.

- r in multipath/main.c:main() might be used uninitialized.
True. So have it initialized.

[build] libmultipath Makefile fix

libmultipath/Makefile resets CFLAGS, so we can't pass anything from the
top-level makefile.

Hannes Reinecke, SuSE

[libmultipath] IBM defaults tweaks

There is a new hardware ID for the IBM F20; and I've also switched to
'dasd_id' from udev instead of dasdview from s390-tools.

Hannes Reinecke, SuSE

[multipath] release preparation

[libmultipath] add pathcountgr() helper function

Teach pathcount() to use it.

Edward Goggin, EMC

[multipathd] add more checkers message printing upon path failures

Edward Goggin, EMC

[checkers] emc_clariion checker init() fix

Edward Goggin, EMC

[checkers] checker_put() to exec the free fn only if one is registered

Edward Goggin, EMC

[multipathd] ev_add_map should return ok for spurious events

Benjamin Marzinski, Redhat

[libmultipath] "show config" segfault due to checkers framework work

[libmultipath] per-hwentry product string blacklist

Associate a default blacklist product string with each hwentry.
This is useful for the CLARiiON in order to blacklist LUNZ entries
by default.

Edward Goggin, EMC

[libmultipath] pathinfo() should not try get_prio() if path is down

David Wysochanski, Netapp
Edward Goggin, EMC

[libmultipath] cycling remove/add a map leaks paths info

When you delete a multipath map, orphaning the path clears out some members
of the path structure.  Adding that map again doesn't bring them back.
Specifically, you loose your checker file descriptor.

The attached patch restores most of the members. It doesn't reset getuid.

Benjamin Marzinski, Redhat

[multipathd] syncing path states

Occasionally, the path states for multipathd and device mapper are out of
sync.
This happens either when multipathd starts, or after a multipath map is
created or reloaded (what a path is added or removed, or a map is added).
The attached patch syncs the paths' dmstate with it's state after one of
these actions happens.

Benjamin Marzinski, Redhat

[multipathd] nr_active inc/dec losts or overzealous

There is some weirdness in the nr_active handling in multipathd, For
instance, whenever you change state to a PATH_DOWN or PATH_SHAKY state,
you decrement the number of active paths, even if you change from
PATH_DOWN to PATH_SHAKY.

This patch attempts to fix the problem.  It slightly changes how some
of the states are handled.

[multipathd] due failback not happening

If the failback period is set to more that four times the polling interval,
paths will never fail back. This is because the failback_tick keeps getting
reset. The attached patch fixes this.

Benjamin Marzinski, Redhat

[multipathd] blacklisting and blacklist changes ignored on spots

multipathd doesn't correctly blacklist devices by wwid, because the
blacklist check happens before the device information is known.

This patch fixes that.

Benjamin Marzinski, Redhat

[build] fix install target glitch in alua prioritizer

Triggered by "make rpm" on fresh systems.
Spotted and fixed by Brian Wong, Pillardata

[checkers] fix segfault in the checker lookup function

Multipath was segfaulting on startup because of some pointer arithmatic
craziness with the checkers. The attached patch fixes it.

Benjamin Marzinski, Redhat

[checkers] API rewrite

The context handling and the layering were poor, so rewrite.

The users now need to :

- alloc a "struct checker"
- select a checker by name : checker_lookup()
- init own checker instance : checker_get()
- set the path file descriptor to check : checker_set_fd()
- use : checker_check()
- release : checker_put()

Checkers now are asked to provide 3 functions :

- foo_init : alloc and initialize the checker context
- foo_free : free the context
- foo      : the checking function proper

These 3 are registered in a static checkers array, along with the checker
name. Lookups are based on checker name.

The users are all updated :

- checker_get is folded into path (re)discovery
- checker_put is folded into path free/orphan

Additional gains :

- directio is updated to use a context to avoid computations upon each
  check
- checkers code is leaner

Please test and report.

Merge ... 137.250.31.225/rsync/git/linux/storage/multipath-tools.git

[kpartx] Add support for mac partition table

Signed-off-by: Bastian Blank <waldi@debian.org>

[build] make rpm fix

Adaptation to new prioritizer and path changes

Brian Wong, Pillardata

[libmultipath] discovery broken for DASD

Since I moved the configlet setting right after vendor/product string
fetching in sysfs_scsi_pathinfo() and forgot about its *ccw* counterpart.

Stefan Weinhuber and Stefan Bader spotted and fixed that.

[libmultipath] set_default_blacklist() : fix the fix

[libmultipath] set_default_blacklist() should not register .rodata strings

... because free_blacklist will try to free those strings, and some glibc
are set to be verbose about that by default.

[libmultipath] snprint_hw_prio_callout() segv

... if prio_callout is not defined in the default config section.

[libmultipath] correct keyword lookup in snprint_hwentry()

[libmultipath] fix blacklist printing segv when "show config"

[libmultipath] fix uninitialized var in discovery.c:get_serial()

[libmultipath] plug leaks in blacklist free_* functions

[libmultipath] plug leak in discover.c:syfs_get_*()

[config] synthetic template mispelled the "user_friendly_names" keyword

Missed the ending 's'.

[libmultipath] "show conf" CLI command to output the blacklist too

All multipath.conf keywords and sections are now covered by "show conf".

[libmultipath] fix new "blacklist" keyword incorrectly spelled to "devnode"

[libmultipath] yet more output prefixing

[libmultipath] move apply_format() from discovery.c to callout.c

[libmultipath] don't set the hwe configlet pointer if !(mask & DI_SYSFS)

Vendor and product strings are fetched if (mask & DI_SYSFS).

We shouldn't even try to find_hwe(vendor, product) if there is a chance
the params are not set.

[libmultipath] more path: and multipath: output prefixing

[libmultipath] add "map:" prefix to propsel functions output

[libmultipath] path discovery reorganisation

o split logic in smaller functions
o prefix output with "%s:...", pp->dev
o rename "curpath" to "pp"

[config] add the "device" blacklist keyword to config templates

[libmultipath] adapt discover.c to the new libsysfs2 API

In fact, I tried to use only the functions common to libsysfs v1 and v2.

[libmultipath] rename conf->default_hwhandler to conf->hwhandler

Misleading name. The real default is DEFAULT_HWHANDLER.

[libmultipath] rename conf->default_getprio to conf->getprio

Misleading name. The real default is DEFAULT_GETPRIO.

[libmultipath] rename conf->default_getuid to conf->getuid

Misleading name. The real default is DEFAULT_GETUID.