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

[libmultipath] rename conf->default_checker_index to conf->checker_index

Misleading name. The real default is DEFAULT_CHECKER_ID.

[libmultipath] rename conf->default_pgpolicy to conf->pgpolicy

Misleading name. The real default is DEFAULT_PGPOLICY.

[libmultipath] remove vector.h include from config.h

[libmultipath] deprecate the "devnode_blacklist" keyword

Please use the "blacklist" keyword instead.

The blacklist section implements the "devnode", "wwid" and "device"
keywords, so it's misleading to keep the old section name.

[libmultipath] add device blacklisting

Syntax is :

devnode_blacklist {
devnode sr*
wwid 6005*
device {
vendor DEC
product *
}
}

[libmultipath] renaming a map triggers its partitions renames

[multipathd] "show topo" as a "show maps topo" alias

[multipathd] add "show config" CLI command

This commands dumps a multipath.conf config file.

The "defaults" section is factorized : settings in sync with the internal
defaults are not printed.

The "devices" section lists the internal hwtable, plus mods and adds from
the actual config file. The layout is factorized : settings in sync with
the "defaults" section are not printed.

The "devnode_blacklist" section is ommited for now, as this requires a bit
more infrastructure.

The "multipaths" section is not factorized as we can not reliably guess the
hardware a multipath is parented to. Multipath entries are just clones
of their actual config file equivalent.

[libmultipath] add internal some defaults handling to propsel.c

o getprio
o checker_index
o getuid

Merge branch 'master' of /linux/storage/multipath-tools/

[libmultipath] fix the internal defaults loading

hwtable.c segfaults : the increment of the hwe pointer is wrong

Mark Haverkamp, OSDL

[libmultipath] STK D280 to use the tpc prioritizer

Align the defaults to SGI TP4500 ones.

Suggested by Chris Penney

[libmultipath] add a buffer len param to get_pgpolicy_name()

[libmultipath] do proper string equality checks in dict.c

[libmultipath] move CHECKINT definition to defaults.h

And transform MAX_CHECKINT into a macro.

Use that in dict.c to set conf->max_checkint with the same formula we
use when we set the in-core default.