[libmultipath] fix user_friendly_names race

Without this patch I could fairly easily get two multipath mpath0 entries
in /var/lib/multipath/bindings by running 8 concurrent instances of
multipath(8) while with the patch I cannot get this problem to occur.

The posix file byte range locks used to provide atomicity for accessing the
entries in the multipath bindings file get released from whenever __any__
descriptor or FILE structure for that file is closed.  This patch delays
the fclose() for the FILE structures used within lookup_binding() and
rlookup_binding() until there is no more need for the atomicity.

Edward Goggin, EMC

[multipathd] man page update

Add CLI commands description.

Volker Sameske, IBM

[libmultipath] tweak defaults for DDN S2A8500

When using multipath-tools 0.4.7, I would like to reports that DDN S2A8500
disk array reports a wrong status whith 'readsector0' checker. 'directio'
seems to be the right checker for such storage system.

Frederic Temporelli, Bull

[checkers] handling CLARiiON I/O to inactive snapshot logical units

Prevent an I/O hang which can occur with host I/O to an inactive CLARiiON
snapshot logical unit.  While the inactive snapshot LU is presented to a
host, inquiry, TUR, and read capacity commands succeed, but read and write
commands are failed.  Complicating the matter is the fact that read/write
to the active paths fails with a particular sense code/asc/ascq combination
while read/write to the passive paths does not.

Edward Goggin, EMC

[checkers] sg_read() enhancement

Retry SGIO 3 times max if we get a "Unit Attention" in sense data.

Edward Goggin, EMC

[libchecker] move sg_read() to a new libsg.c

This function is bound to be shared among checkers.
So make it easy to do so.

[checkers] multipath-wide shared context for checkers

At least one checker need to shared data between checkers attached to
all paths of a multipath.

This patch addresses this need by adding a "void * mpcontext" to
struct multipath, and its "void **" sibling in struct checker.

checker_init() ensures the multipath->mpcontext is attached to
*checker->mpcontext.

Checkers are in charge of allocating the memory block and attaching it
to *checker->mpcontext. the .init checker function is a natural
place for that.

Freeing the mpcontext in done in free_multipath().

[defaults] Change of IBM DS8000 path grouping policy

Michael Loehr, IBM

[libmultipath] braino, still get path prio for paths with PATH_DOWN

Edward Goggin, EMC

[priority] adjust SCSI RTPG request buffer size in ALUA prioritizer

The following patch adds error and size checking to the REPORT TARGET
PORT GROUPS command issued by rtpg.c.  The old code issued a buffer of
128 bytes, but never checked the return length.  The new code starts
with a buffer of 128 bytes, but reallocates it if the buffer is too
small (SCSI returns the necessary length in the response data).  This is
more robust as it handles devices that consume more than 128 bytes for
RTPG, like the Incipient NSP.

I tried to handle errors, etc. as closely as I could to what is already
there in the code.  Let me know if you'd prefer anything written
differently.

Brian Geisel

[libmultipath] clarify priority values

Define two values :
PRIO_UNDEF: prioritizer not exec yet, or returned an error
PRIO_DEFAULT: no prioritizer set, default value arbitrary set to 1

Changes :
1) 0 is no longer used as a possible priority value.
2) don't exec prioritizer in pathinfo() if path is down and priority is
defined

Edward Goggin, EMC
Dave Wysochanski, Red Hat
& Me

[libcheckers] document checker states

Dave Wysochanski, Red Hat

[libmultipath] scope limiting fixes

I messed up the patch a bit.  Apparently dm_get_uuid() can
return a zero without actually having found a dm name which
matches the name parameter.

Also, I changed dm_get_uuid() so it either returns 1 or a null
string in the uuid return parameter -- in 2 different cases
it was doing neither.

The <dm_device_name> qualifier for the command line
"multipath -ll <dm device name>" can now accept up
to 3 different names for the same dm mapped device,
that is, a user friendly name, an alias in
/etc/multipath.conf, and the current name as
reported by "dmsetup ls".

Edward Goggin, EMC

[libmultipath] configure() scope-limiting corner case

This patch enables "multipath -l <dm device name>"
to work if the current dm_alias has been changed by
"dmsetup rename" or other means.

Edward Goggin, EMC

[libcheckers]

Change to libcheckers/readsector0.c to allow other checkers than
readsector0 to use sg_read().

Also provide the sense info in a buffer supplied by caller.

Edward Goggin, EMC

[libmultipath] pathinfo() to invoke the path priority callout

Changed libmultipath/discovery.c to allow pathinfo() to invoke the path
priority callout function of a path with a path state of PATH_DOWN since
some paths (e.g., paths to inactive snapshot LU on CLARiiON) may have
their state set to PATH_DOWN for reasons other than transport
connectivity failure.

Without this change, it is possible to create a situation whereby there
are paths of different priorities in the same path group even though the
path group policy is priority based grouping.  This is certainly the
case for paths to a multipath mapped device for an inactive snapshot
CLARiiON LUs when the paths change to a healthy state as soon as the
snapshot is activated.

Edward Goggin, EMC

[libmultipath] remove dup_hwe()

Deprecated by recently introduced find_hwe_strmatch().

[libmultipath] RR_WEIGHT_PRIORITIES undefined

Yet. So don't use it in the recently introduced  HSV2.* defaults.

[libmultipath] Update HP HSV200 hardware table

According to HP the behaviour of the HSV200 depends on the software
release; all revisions smaller or equal to '3000' have active/passive
support; all revisions above that have active/active support.
Translated it means that the older firmware uses the HP-style
'START_STOP_UNIT' command to switch paths, while the newer firmware
supports ALUA.
So we should update the hardware tables accordingly.

Signed-off-by: Hannes Reinecke <hare@suse.de>

[libmultipath] Add support for matching against revision field

The standard SCSI inquiry data provides for a field 'revision' which
specifies the software revision of the device. Some vendors changed
the behaviour during software releases so it's desireable that we
can match on that field in the hardware table, too.

Signed-off-by: Hannes Reinecke <hare@suse.de>

[libmultipath] Use strmatch to load config file

Currently we're using regexec to load the initial hardware table.
Unfortunately the hardware table has regex as defaults, so it's quite
possible for two different regular expressions do match.
But that's quite undesired as we just want a clean 1:1 copy here.
So we're better using a string match for comparison.

Signed-off-by: Hannes Reinecke <hare@suse.de>

[libmultipath] multipath devices are not created for the Luns having 8 paths

When setting the rr_weight to 'priorities' multipath fails to
setup a device-mapper table. This is due to a missing initialization
of the minio field, which will be multiplied by the path priority
for each path. This results in an eventual overflow of the minio
field and an invalid device mapper table.

References: 203823
Signed-off-by: Hannes Reinecke <hare@suse.de>

[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.