multipath-tools/.git
3 years agoAdd libmpathcmd library and use it internally
Benjamin Marzinski [Thu, 7 Apr 2016 23:19:58 +0000 (18:19 -0500)]
Add libmpathcmd library and use it internally

Other programs would like to communicate with multipathd to issue
command or check status.  Instead of having them exec multipathd,
I've pulled the code that sends commands and receives replies from
multipathd into its own library.  I've made the multipath tools use
this library internally as well.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
3 years agoFix issues with user_friendly_names initramfs bindings
Benjamin Marzinski [Thu, 7 Apr 2016 23:19:57 +0000 (18:19 -0500)]
Fix issues with user_friendly_names initramfs bindings

Multipath has an issue with user_friendly_names set in the initramfs.
If the bindings are in the initramfs bindings file, it will create them,
and it may use bindings that are different than the ones in the regular
file system.  Once multipathd starts up in the regular file system, it
will try to register the existing bindings, but that may (and in many
cases, is likely to) fail. If it can't reanme it, will pick a new
binding. Since when multipathd starts discovering the existing devices,
it obviously doesn't know all of the existing devices yet, it may very
well pick a binding that's already in use by a device that it hasn't
discovered yet. In this case multipath will be unable to rename the
device to the new binding. Unfortunately, if it fails the rename, it
never resets the alias of the device stored in the mpvec to current
alias of the actual dm device. So multipath will have devices in the
mpvec where the alias and the wwid don't match the actual dm devices
that exist.  This can cause all sorts of problems.

This patch does two things to deal with this.  First, it makes sure that
if the rename fails, the device will reset its alias to the one that it
currently has. Second, it fixes the -B option to actually work
correctly, so that it can be started that way in the initramfs.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
3 years agoretrigger uevents to try and get the uid through udev
Benjamin Marzinski [Thu, 7 Apr 2016 23:19:56 +0000 (18:19 -0500)]
retrigger uevents to try and get the uid through udev

Ideally, udev will be able to grab the wwid when a path device is
discovered, but sometimes this isn't possible. In these cases, the
best thing that could happen would be for udev to actually get the
information, and add it to its database. This patch makes multipath
retrigger uevents a limited number of times before giving up and
trying to get the information itself.

There are two configurables that control how it does this,
"retrigger_tries" and "retrigger_delay". The first sets the number of
times it will try to retrigger a uevent to get the wwid, the second
sets the amount of time to wait between retriggers.

This patch currently only tries reinitializing the path on change events
after multipathd has triggered a change event, and it only tries once
per triggered change event.  Now, its possible that other change events
could occur on the device without multipathd tirggering them.  As the
patch stands now, it won't try to initialize the device on those.  It will,
however still try in the checkerloop, but only after it has finished
retriggering the uevents. We could be much more aggressive here, and assume
that devices that simply won't have a WWID should already be taken care of
by the blacklists, so it would be always a good idea to recheck devices on
change events. What would be ideal is if udev would let us know when it had
problems or timed out when processing a uevent, so we would know if
retiggering the uevent would be useful.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
3 years agomultipathd: use /run instead of /var/run
Benjamin Marzinski [Thu, 7 Apr 2016 23:19:55 +0000 (18:19 -0500)]
multipathd: use /run instead of /var/run

/var/run is now usually a symlink to /run.  If /var is on a separate
filesytem, when multipathd starts up, it might end up writing to
/var/run before the /var filesytem is mounted and thus not have its
pidfile accessible at /var/run afterwards.  On most distrubutions /run
is now a tmpfs and should always be available before multipathd is
started, so multipath should just write there directly, instead of
through the symlink.

If /var/run is not a symlink, continue using it.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
3 years agomultipath-tools: prevent unnecessary reinstate of stand-by paths with implicit tags...
Shiva Krishna [Tue, 8 Mar 2016 16:26:57 +0000 (17:26 +0100)]
multipath-tools: prevent unnecessary reinstate of stand-by paths with implicit tags mode and no active array paths

multipathd treats "stand-by" path as active(ghost) and reinstate path.This
causes I/O hang issues and lots of "change" udev events in cases where only
stand-by paths are present in multipath map and target supports only
implicit tpgs mode(active/passive arrays)

This can happen during system boot where only stand-by paths are discovered
first and continuous retry of I/O's by dm-multipath and change(failed)
events are hogging multipathd and slowing down the entire boot process with
large number of volumes mapped (~100s).

Selecting path checkers other than tur is not a solution as well, as they
will continuously throw messages saying paths are failed for stand-by
state.

This patch will prevent re-instate of stand-by paths in this situation and
thus prevent unnecessary i/o with failed events during boot or when active
array goes down temporarily.

With this, for targets supporting only alua and implicit tpgs mode,
stand-by
paths are not reinstated by path checker unless there is an active path in
the map.

Detailed steps to hit the I/O hang issue even with no_path_retry:

1. Delete all active paths.

mpathal (2e0176ad6309077166c9ce90033bfa248_1) dm-2 Nimble,Server
size=20G features='1 queue_if_no_path' hwhandler='1 alua' wp=rw
`-+- policy='round-robin 0' prio=1 status=active
  |- 8:0:5:1 sdg 8:96  active ghost running
  `- 8:0:6:1 sdh 8:112 active ghost running

2. Issue I/O on mpath with zero active paths.

dd if=/dev/mapper/mpathal of=/dev/null bs=512 count=1 iflag=direct &

Nov 23 15:10:36 hitdev-rhel67 multipathd: mpathal: sdg - tur checker
reports path is in standby state
Nov 23 15:10:36 hitdev-rhel67 multipathd: 8:96: reinstated
Nov 23 15:10:36 hitdev-rhel67 multipathd: mpathal: queue_if_no_path enabled
Nov 23 15:10:36 hitdev-rhel67 multipathd: mpathal: Recovered to normal mode
Nov 23 15:10:36 hitdev-rhel67 multipathd: mpathal: remaining active paths:1
Nov 23 15:10:36 hitdev-rhel67 kernel: sd 8:0:5:1: alua: port group 02
state S non-preferred supports tolusna
Nov 23 15:10:36 hitdev-rhel67 kernel: device-mapper: multipath: Failing
path 8:96.
Nov 23 15:10:36 hitdev-rhel67 multipathd: 8:96: mark as failed
Nov 23 15:10:36 hitdev-rhel67 multipathd: mpathal: Entering recovery mode:
max_retries=20
Nov 23 15:10:36 hitdev-rhel67 multipathd: mpathal: remaining active paths:
0

3. Monitor udev events every 5 seconds.

# udevadm monitor

KERNEL[1448320131.061837] change   /devices/virtual/block/dm-2 (block)
KERNEL[1448320131.064802] change   /devices/virtual/block/dm-2 (block)
UDEV  [1448320131.082838] change   /devices/virtual/block/dm-2 (block)
UDEV  [1448320131.102134] change   /devices/virtual/block/dm-2 (block)
KERNEL[1448320135.551737] change   /devices/virtual/block/dm-2 (block)
KERNEL[1448320135.552701] change   /devices/virtual/block/dm-2 (block)

Signed-off-by: ShivaKrishna Merla <shiva.krishna@nimblestorage.com>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Reviewed-by: Benjamin Marzinski <bmarzins@redhat.com>
3 years agoFix loopback file with kpartx -av
Mathieu Trudel-Lapierre [Wed, 3 Feb 2016 08:24:08 +0000 (09:24 +0100)]
Fix loopback file with kpartx -av

Commit b407050aff25678c8d60cdbb9ac7d3ff44d5a155 apparently breaks the use
of loopback-mounted files if you try to use them directly with kpartx -av.

For instance:
qemu-img create foo.img 20M
sgdisk -n 1 foo.img
kpartx -av foo.img

Fails with the following error:
device-mapper: reload ioctl on loop0p1 failed: No such device
create/reload failed on loop0p1
add map loop0p1 (0:0): 0 4192256 linear 0:0 2048

This can be fixed by stat()'ing the loop device again after picking it

3 years agokpartx: support device names with spaces
Mauricio Faria de Oliveira [Wed, 20 Jan 2016 13:52:45 +0000 (11:52 -0200)]
kpartx: support device names with spaces

Device names with spaces fail to be added to device-mapper tables
because spaces are field separators.  In order to support it, use
device major:minor numbers, instead.  While still there, make the
verbose output of container- and contained-slices consistent.

This is useful on some scenarios with arbitrary loop device names,
and non-user_friendly_names multipath WWIDs with spaces (eg, QEMU
emulated hard drives, and some IBM devices) on some distros.

Changelog:
- V2: Use major()/minor() instead of MAJOR()/MINOR() (build errors).
      Update test-case verification output.
      Thanks: Christophe Varoqui

Test-case:

# img="/tmp/my.img"
# spacename="/dev/my space name"

# truncate --size=1G "$img"
# lodev=$(sudo losetup --find --show "$img")
# ln -sf "$lodev" "$spacename"

# ls -l $lodev "$spacename"
brw-rw----. 1 root disk 7, 0 Jan 20 11:15 /dev/loop0
lrwxrwxrwx. 1 root root   10 Jan 20 11:16 /dev/my space name -> /dev/loop0

# fdisk $lodev
<...>

# fdisk -l $lodev
<...>
      Device Boot      Start         End      Blocks   Id  System
/dev/loop0p1            2048      264191      131072   83  Linux
/dev/loop0p2          264192      788479      262144   83  Linux
/dev/loop0p3          788480     2097151      654336    5  Extended
/dev/loop0p5          790528      856063       32768   83  Linux
/dev/loop0p6          858112      989183       65536   83  Linux
/dev/loop0p7          991232     1515519      262144   83  Linux

Without patch:

# ./kpartx/kpartx -av "$spacename"
device-mapper: reload ioctl on my\x20space\x20name1 failed: Invalid argument
create/reload failed on my space name1
add map my space name1 (0:0): 0 262144 linear /dev/my space name 2048

device-mapper: reload ioctl on my\x20space\x20name2 failed: Invalid argument
create/reload failed on my space name2
add map my space name2 (0:0): 0 524288 linear /dev/my space name 264192

device-mapper: reload ioctl on my\x20space\x20name3 failed: Invalid argument
create/reload failed on my space name3
add map my space name3 (0:0): 0 2 linear /dev/my space name 788480

device-mapper: reload ioctl on my\x20space\x20name5 failed: Invalid argument
add map my space name5 : 0 65536 linear /dev/my space name 790528

device-mapper: reload ioctl on my\x20space\x20name6 failed: Invalid argument
add map my space name6 : 0 131072 linear /dev/my space name 858112

device-mapper: reload ioctl on my\x20space\x20name7 failed: Invalid argument
add map my space name7 : 0 524288 linear /dev/my space name 991232

(note: line breaks added for clarity)

With patch:

# ./kpartx/kpartx -av "$spacename"
add map my space name1 (253:3): 0 262144 linear 7:0 2048
add map my space name2 (253:4): 0 524288 linear 7:0 264192
add map my space name3 (253:5): 0 2 linear 7:0 788480
add map my space name5 (253:6): 0 65536 linear 7:0 790528
add map my space name6 (253:7): 0 131072 linear 7:0 858112
add map my space name7 (253:8): 0 524288 linear 7:0 991232

# dmsetup table | sort
my space name1: 0 262144 linear 7:0 2048
my space name2: 0 524288 linear 7:0 264192
my space name3: 0 2 linear 7:0 788480
my space name5: 0 65536 linear 7:0 790528
my space name6: 0 131072 linear 7:0 858112
my space name7: 0 524288 linear 7:0 991232

# ls -l /dev/mapper/
total 0
crw-------. 1 root root 10, 236 Jan 11 16:29 control
lrwxrwxrwx. 1 root root       7 Jan 20 11:19 my\x20space\x20name1 -> ../dm-3
lrwxrwxrwx. 1 root root       7 Jan 20 11:19 my\x20space\x20name2 -> ../dm-4
lrwxrwxrwx. 1 root root       7 Jan 20 11:19 my\x20space\x20name3 -> ../dm-5
lrwxrwxrwx. 1 root root       7 Jan 20 11:19 my\x20space\x20name5 -> ../dm-6
lrwxrwxrwx. 1 root root       7 Jan 20 11:19 my\x20space\x20name6 -> ../dm-7
lrwxrwxrwx. 1 root root       7 Jan 20 11:19 my\x20space\x20name7 -> ../dm-8

Thanks: Scott Moser (pointers to the error and test-case).

Signed-off-by: Mauricio Faria de Oliveira <mauricfo@linux.vnet.ibm.com>
3 years agoAdd hardware defaults for the PURE FlashArray devices
Brian Bunker [Wed, 20 Jan 2016 06:57:19 +0000 (07:57 +0100)]
Add hardware defaults for the PURE FlashArray devices

4 years agofix unmatched params when mpath prout rollback
Xieyingtai [Thu, 15 Oct 2015 12:06:19 +0000 (14:06 +0200)]
fix unmatched params when mpath prout rollback

Mpath prout register may rollback, but the params are not matched in
creating thread for each path.

4 years agolibmpathpersist: Set open fds limit in mpath_lib_init
Xieyingtai [Wed, 14 Oct 2015 06:06:46 +0000 (08:06 +0200)]
libmpathpersist: Set open fds limit in mpath_lib_init

Mpath persistent reserve out may fail due to file open failed when
too many devices were discoveryed but fd numbers not enough.

Set open fds limit just like multipath in that scene.

4 years agoFix sun partition numbering
Benjamin Marzinski [Thu, 8 Oct 2015 19:44:50 +0000 (14:44 -0500)]
Fix sun partition numbering

On Sun labeled devices, the kpartx partition numbers weren't agreeing
with the underlying device partition numbers.  This is because kpartx
doesn't count slices with no sectors, when determining the numbering.
This patch counts these empty slices, so the numbering comes out the
same.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agoadd raw format multipathd commands
Benjamin Marzinski [Thu, 8 Oct 2015 19:44:45 +0000 (14:44 -0500)]
add raw format multipathd commands

This patch adds the following multipathd interactive commands

show paths raw format $fmt
show maps raw format $fmt

These commands work just like the regular "show ... format"
commands, except that they don't print a header, and don't add
any extra padding that isn't in the format string.  This should
make it easier for programs wanting to parse the output from
these commands.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agoMake multipath ignore devices without mpath prefix
Benjamin Marzinski [Thu, 8 Oct 2015 19:44:51 +0000 (14:44 -0500)]
Make multipath ignore devices without mpath prefix

Multpathd currently assumes that it controls all devices of "multipath"
type. In order to allow people to create multipath type devices outside
of the multipath-tools, this patch makes multipathd only modify devices
if they have a uuid that starts with "mpath-". This was a suggestion
from Hannes a couple of months ago.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agoIncrease host buffer size
Benjamin Marzinski [Thu, 8 Oct 2015 19:44:49 +0000 (14:44 -0500)]
Increase host buffer size

Currently the host buffer only has space for 7 characters, this means
on systems with many scsi hosts (1000 or more), multipath will overflow
this buffer. This can happen pretty easily if there are a large number
of iscsi devices.  This patch increases the host buffer to hold 15
characters, which can deal with 100000000000000 scsi hosts.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agolibmpathpersist: uninstall man page correctly
Benjamin Marzinski [Thu, 8 Oct 2015 19:44:48 +0000 (14:44 -0500)]
libmpathpersist: uninstall man page correctly

the libmpathpersist man pages were being installed to
/usr/share/man/man3, but uninstalled from /usr/share/man/man8.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agoRevert "change order of multipath.rules"
Christophe Varoqui [Tue, 13 Oct 2015 07:30:27 +0000 (09:30 +0200)]
Revert "change order of multipath.rules"

This reverts commit 4340aca696857c1077b00ff6bdc0454a07929aad.

NACK by Hannes Reinecke:

56-multipath.rules has been placed bevor persistent storage so that
it can override settings in 60-persistent-storage.rules.

If you need some additional settings to fixup things after that
please add a new rules file.

4 years agoRemove duplicate hwtable entry for XtremIO
Hannes Reinecke [Mon, 30 Mar 2015 06:23:22 +0000 (08:23 +0200)]
Remove duplicate hwtable entry for XtremIO

Commit b47e607408e1edce3a03744b4aefa292723c7fc6 introduced a new
hwtable entry for EMC XtremIO instead of updating the existing
one.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoresize reply buffer for mutipathd help message
Benjamin Marzinski [Fri, 15 May 2015 23:14:13 +0000 (18:14 -0500)]
resize reply buffer for mutipathd help message

Unlike the other reply messages from multipathd, the generic help
message code couldn't resize the buffer, and the reply had grown too big
for the initial buffer size. This was causing memory corruption and
crashing multipathd instead of printing a help message. This patch
increases the size of the initial reply buffer, and makes the help
message code able to resize the buffer so this doesn't happen in the
future.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agoMake multipath deactivate devices before iscsi shutdown
Benjamin Marzinski [Fri, 15 May 2015 23:14:12 +0000 (18:14 -0500)]
Make multipath deactivate devices before iscsi shutdown

The multipath devices need to be deactivated before iscsi is shutdown,
otherwise the multipath devices will generate a lot of error messages
when it loses its iscsi path devices.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agomake kpartx -d remove all partitions
Benjamin Marzinski [Fri, 15 May 2015 23:14:10 +0000 (18:14 -0500)]
make kpartx -d remove all partitions

Currently kpartx -d only removes the partitions that have entries in the
partition table.  If you remove a partition from the device, and then
run kpartx -d, it will not delete that partition (kpartx -u will).  I don't
see any value in leaving partition devices for partitions that don't even
exist anymore, so this patch makes -d delete all partitions.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agochange order of multipath.rules
Benjamin Marzinski [Fri, 15 May 2015 23:14:09 +0000 (18:14 -0500)]
change order of multipath.rules

At least for RedHat, the rule that calls scsi_id is
60-persistent-storage.rules, so the multipath rule needs to come
after this.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agoupdate multipath rules to deal with partition devices
Benjamin Marzinski [Fri, 15 May 2015 23:14:08 +0000 (18:14 -0500)]
update multipath rules to deal with partition devices

Partition devices should inherit the DM_MULTIPATH_DEVICE_PATH
state of their parents, and if they are part of multipath path devices,
they shouldn't have their own ID_FS_TYPE

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agoRemove trailing whitespace
Christophe Varoqui [Tue, 6 Oct 2015 07:22:53 +0000 (09:22 +0200)]
Remove trailing whitespace

4 years agofix memory leaks on realloc failures
Benjamin Marzinski [Fri, 15 May 2015 23:14:05 +0000 (18:14 -0500)]
fix memory leaks on realloc failures

When realloc fails, it doesn't free the existing memory.  In many places,
multipath was just forgetting about that that memory. It needs to free
up the existing memory, or to reset back to using it.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
4 years agoUse image file device/inode to find the correct loop device in kpartx
Risto Kankkunen [Mon, 29 Jun 2015 15:39:48 +0000 (18:39 +0300)]
Use image file device/inode to find the correct loop device in kpartx

Previously kpartx used the "lo_name" field of struct loop_info to store
and match the image file name. That field is not intended to store path
names (it's not big enough). Therefore kpartx was unable to delete
mappings to file paths longer than 63 characters. It also didn't
properly handle relative file paths: two different files with identical
relative path names could not be mapped. Instead kpartx would modify
the existing mapping when seeing a new file with the same relative path.

The "loopinfo" structure contains the image file device and inode
numbers. Use those to uniquely identify the correct loop device.

4 years agolibmultipath: Fix information not shown when first path is down.
Gabriel Krisman Bertazi [Mon, 5 Oct 2015 20:19:17 +0000 (17:19 -0300)]
libmultipath: Fix information not shown when first path is down.

vendor_id and product_id information are not shown when displaying
multipath topology if the first path is down.

0QEMU_QEMU_HARDDISK_HELLOWORLD dm-0 ,      <--- Missing vendor and product
size=10G features='0' hwhandler='0' wp=rw
|-+- policy='service-time 0' prio=0 status=active
| `- #:#:#:# -   #:#  active undef running
`-+- policy='service-time 0' prio=0 status=enabled
  `- 2:0:1:0 sdd 8:48 active undef running

This happens because we only look at the first_path when printing the
topology.  This patch looks at every slot in the path vector until we
find one with the correct value.

Signed-off-by: Gabriel Krisman Bertazi <krisman@linux.vnet.ibm.com>
4 years agomultipathd: Fixing add map functionality
Tejaswini Poluri [Tue, 28 Jul 2015 09:56:56 +0000 (15:26 +0530)]
multipathd: Fixing add map functionality

Add map feature of multipathd tools fails as the device
is also been completely removed along with removing the map
in the remove map code(ev_remove_map()).
So when add map is called we are creating the device again
in cli_add_map() using the coloasce_paths code.

Signed-off-by:Tejaswini Poluri <tpoluri@mvista.com>

4 years agolibmultipath: make vpd page 0x80 optional
Hannes Reinecke [Mon, 16 Feb 2015 10:10:47 +0000 (11:10 +0100)]
libmultipath: make vpd page 0x80 optional

VPD page 0x80 is optional, so we shouldn't return an error if
the page is not present.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoDo not automatically fall back to vpd uid generation
Hannes Reinecke [Tue, 16 Dec 2014 10:51:12 +0000 (11:51 +0100)]
Do not automatically fall back to vpd uid generation

When a wwid via uid_attribute cannot be generated we should
not fall back to using sysfs vpd attributes automatically.
The uid_attribute can be set to virtually anything and an
automatic fallback will then generated a different wwid,
leading to an incorrect setup.
A fallback is only valid if the uid_attribute value
is the default setting of 'ID_SERIAL', for which the
format is fixed.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: use SG_IO as fallback to generate uid
Hannes Reinecke [Mon, 15 Dec 2014 10:11:15 +0000 (11:11 +0100)]
multipathd: use SG_IO as fallback to generate uid

If both udev attribute and sysfs vpd fails to generate a valid
WWID fallback to use SG_IO directly.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoSeparate out vpd parsing functions
Hannes Reinecke [Mon, 15 Dec 2014 09:42:13 +0000 (10:42 +0100)]
Separate out vpd parsing functions

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agolibmultipath: unset 'uid_attribute' on failure
Hannes Reinecke [Thu, 11 Dec 2014 15:15:41 +0000 (16:15 +0100)]
libmultipath: unset 'uid_attribute' on failure

Due to a race condition within udev the 'uid_attribute'
might not always be set. So we should be zeroing the
'uid_attribute' when retrieving the uid by other means,
otherwise the discovery process will blacklist the device.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agolibmultipath: Fall back to SG_IO if no UID could be assigned
Hannes Reinecke [Wed, 10 Dec 2014 08:35:20 +0000 (09:35 +0100)]
libmultipath: Fall back to SG_IO if no UID could be assigned

Relying on udev attributes or sysfs vpd pages might fail,
in which case we're unable to assign an UID and multipath
will fail to start up.
Implement a fallback to read VPD page 0x83 directly in
these cases.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoUpdate multipath.conf.5 to clarify wwid generation
Hannes Reinecke [Tue, 22 Jul 2014 10:07:26 +0000 (12:07 +0200)]
Update multipath.conf.5 to clarify wwid generation

Add a new section in multipath.conf.5 to clarify wwid generation
and update wrong defaults for the property whitelist.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoUse sysfs attribute vpd_pg80 to read serial number
Hannes Reinecke [Wed, 3 Sep 2014 12:14:45 +0000 (14:14 +0200)]
Use sysfs attribute vpd_pg80 to read serial number

There is no need to issue an ioctl to read the serial number
as it's already present in sysfs. And we should not print
out the serial number unconditionally but rather check
the discovery mask.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoAssign local priority for NAA VPD descriptor
Hannes Reinecke [Wed, 10 Dec 2014 08:13:10 +0000 (09:13 +0100)]
Assign local priority for NAA VPD descriptor

Some devices assign several NAA VPD descriptors. So implement
an internal priority to select the 'best' NAA descriptor.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoRead wwid from sysfs vpg_pg83 attribute
Hannes Reinecke [Tue, 22 Jul 2014 09:39:15 +0000 (11:39 +0200)]
Read wwid from sysfs vpg_pg83 attribute

Using 'uid_attribute' per default has the problem that udev
might not be able to retrieve the device ID in time as the
device might be (temporarily) blocked.
It also has the problem that the 'ID_SERIAL' attribute is
not well defined and might have been overridden by other udev
rules.

As recent kernels have a 'vpd_pg83' sysfs attribute multipath
should be reading this one directly and extract the uid from there.
With that multipath does not need to do any I/O to generate the
device wwid, eliminating one common error cause during failover.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoSkip USB devices during discovery
Hannes Reinecke [Wed, 3 Sep 2014 12:31:14 +0000 (14:31 +0200)]
Skip USB devices during discovery

Some USB devices even present VPD pages, but they still remain
USB devices, and we shouldn't run multipath on those.
So detect USB devices properly and skip them during discovery.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoIgnore devices when sysfs_get_tgt_nodename fails
Hannes Reinecke [Wed, 3 Sep 2014 12:16:43 +0000 (14:16 +0200)]
Ignore devices when sysfs_get_tgt_nodename fails

If sysfs_get_tgt_nodename fails we should not try to use this
device.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: use local variable for watchdog configuration
Hannes Reinecke [Wed, 3 Sep 2014 06:21:15 +0000 (08:21 +0200)]
multipathd: use local variable for watchdog configuration

The 'watchdog' setting in struct config needs to be a local
variable in multipathd/main.c, as the config will be freed
during reconfiguration and the checkerloop might then
access an invalid structure.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoRevert to ACT_RELOAD in domap if the map exists
Hannes Reinecke [Thu, 21 Aug 2014 09:57:55 +0000 (11:57 +0200)]
Revert to ACT_RELOAD in domap if the map exists

ACT_CREATE checks for an existing map already, so we might as
well revert to ACT_RELOAD in these cases and avoid an error
during configuration.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoRemove dm_udev_XXX wrapper functions
Hannes Reinecke [Wed, 20 Aug 2014 12:56:09 +0000 (14:56 +0200)]
Remove dm_udev_XXX wrapper functions

As the definitions are encapsulated with the correct definitions
anyway there is no need to have private wrappers here.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipath: enable sync support
Hannes Reinecke [Fri, 16 May 2014 08:20:33 +0000 (10:20 +0200)]
multipath: enable sync support

multipath didn't enable 'sync support' for device-mapper,
causing device-mapper to create the device nodes internally
rather than relying on udev here.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: lock cli client list
Hannes Reinecke [Wed, 20 Aug 2014 08:57:22 +0000 (10:57 +0200)]
multipathd: lock cli client list

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agouxlsnr: use typedef for trigger function
Hannes Reinecke [Wed, 20 Aug 2014 09:19:05 +0000 (11:19 +0200)]
uxlsnr: use typedef for trigger function

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: Use standard lists for CLI handling
Hannes Reinecke [Tue, 19 Aug 2014 11:46:48 +0000 (13:46 +0200)]
multipathd: Use standard lists for CLI handling

We already have standard list handling macros in list.h, so
we should be using them for CLI, too.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoUse strlen() when checking for valid wwid
Hannes Reinecke [Tue, 19 Aug 2014 10:31:10 +0000 (12:31 +0200)]
Use strlen() when checking for valid wwid

The wwid is required to be a string, so we should be using
strlen() when checking for a valid WWID.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: Issue warning on CLI command timeout
Hannes Reinecke [Mon, 18 Aug 2014 10:22:22 +0000 (12:22 +0200)]
multipathd: Issue warning on CLI command timeout

Some CLI commands like 'reconfigure' could take longer than the
socket timeout, causing the CLI to report 'timeout receiving packet'.
In these cases the multipath daemon should issue a warning with
the actual time spent in the CLI call to give the administator
some hints on the best socket timeout.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoRework uev_add_path()
Hannes Reinecke [Mon, 9 Mar 2015 11:06:35 +0000 (07:06 -0400)]
Rework uev_add_path()

Rework uev_add_path() to handle failed and blacklisted
paths properly.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoAllow zero-sized devices during configuration
Hannes Reinecke [Mon, 18 Aug 2014 11:22:47 +0000 (13:22 +0200)]
Allow zero-sized devices during configuration

A size of '0' doesn't indicate an invalid device; other paths might
end up with a correct size.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: do not remove paths without uevents
Hannes Reinecke [Wed, 13 Aug 2014 17:56:52 +0000 (19:56 +0200)]
multipathd: do not remove paths without uevents

multipathd should not remove any paths without the corresponding
uevent. Otherwise the daemon will never be able to reinstate that
path as it wouldn't get any uevents.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: reload map if reinstate failed
Hannes Reinecke [Tue, 12 Aug 2014 09:56:21 +0000 (11:56 +0200)]
multipathd: reload map if reinstate failed

The kernel might fail the 'reinstate' device-mapper message
if the path is disabled. In these cases we need to reload the
map to give device-mapper a chance to add the correct devices
to the table.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoDo not print empty multipaths section
Hannes Reinecke [Tue, 12 Aug 2014 08:09:59 +0000 (10:09 +0200)]
Do not print empty multipaths section

The CLI command 'show config' or the 'multipath -t' command
should not print an empty 'multipaths' section.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoImplement 'uxsock_timeout' keyword
Hannes Reinecke [Mon, 11 Aug 2014 10:49:56 +0000 (12:49 +0200)]
Implement 'uxsock_timeout' keyword

On larger systems the default CLI receive timeout of 1 second
might be too small, and the CLI commands might fail before
they can be processed. For these cases I've implemented a new
keywork 'uxsock_timeout' which can be increased to avoid these
issues.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoReturn error when receiving CLI packet
Hannes Reinecke [Mon, 11 Aug 2014 10:34:39 +0000 (12:34 +0200)]
Return error when receiving CLI packet

When receiving a packet from the CLI we should be returning
an error code.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipath: do not print state 'orphan' for option '-l'
Hannes Reinecke [Mon, 11 Aug 2014 10:05:19 +0000 (12:05 +0200)]
multipath: do not print state 'orphan' for option '-l'

When called with '-l' multipath would mistakenly print a state
of 'orphan' instead off 'running' of 'offline'.
This is a regression introduced by commit 7023d320.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoFixup device-mapper 'cookie' handling
Hannes Reinecke [Tue, 29 Jul 2014 13:44:46 +0000 (15:44 +0200)]
Fixup device-mapper 'cookie' handling

device-mapper has a 'cookie', which is inserted with the ioctl
for modifying device-mapper devices.
It is used as a synchronization point between udev and any other
applications to notify the latter when udev has finished
processing the event.
Originally multipath would only use a single cookie for every
transaction, and wait for that cookie at the end of the program.
Which works well if you only have one transaction, but for several
(like calling 'multipath') it will actually overwrite the cookie
and fail to wait for earlier events.
This causes libdevmapper to create the device nodes on its own,
and the device nodes not being handled by udev.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoRemove last argument from verify_paths()
Hannes Reinecke [Tue, 29 Jul 2014 10:41:06 +0000 (12:41 +0200)]
Remove last argument from verify_paths()

Argument is always NULL, so remove it.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoAdd paths with a size of '0' as 'ghost' paths
Hannes Reinecke [Tue, 29 Jul 2014 09:51:38 +0000 (11:51 +0200)]
Add paths with a size of '0' as 'ghost' paths

SPC-3 does not require that a port in 'standby' has to support
the 'READ CAPACITY' command. As such we should not reject those
paths, but rather add them as 'ghost' paths.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoRemove sysfs_get_dev
Hannes Reinecke [Tue, 29 Jul 2014 09:29:41 +0000 (11:29 +0200)]
Remove sysfs_get_dev

Pointless, and can be replaced by udev_device_get_devnum().

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipath: Use standard 'major' macro
Hannes Reinecke [Tue, 29 Jul 2014 09:16:33 +0000 (11:16 +0200)]
multipath: Use standard 'major' macro

Instead of carrying our own hand-crafted macros we should be using
the system-provided 'major' macro.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agokpartx: use standard 'major' and 'minor' macros
Hannes Reinecke [Tue, 29 Jul 2014 09:14:39 +0000 (11:14 +0200)]
kpartx: use standard 'major' and 'minor' macros

Instead of hand-coding our own versions we should be using the
standard 'major' and 'minor' macros.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agompath_persist: cleanup
Hannes Reinecke [Tue, 29 Jul 2014 09:10:02 +0000 (11:10 +0200)]
mpath_persist: cleanup

Cleanup whitespace issues and use the standard 'major' and 'minor'
macros.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoUse poll() when receiving uevents
Hannes Reinecke [Thu, 24 Jul 2014 06:51:23 +0000 (08:51 +0200)]
Use poll() when receiving uevents

Currently uevent_listen() will be adding each event individually
to the internal queue. This leads to a lock contention on high
load, as uevent_listen() has to grab the shared lock before
doing so.
This patch batches the uevent reception so that uevents will
only ever be added to the internal queue if there are no more
events pending.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoSeparate out uevent parsing functions
Hannes Reinecke [Thu, 24 Jul 2014 06:22:52 +0000 (08:22 +0200)]
Separate out uevent parsing functions

Move the uevent parsing into separate functions.
No functional change.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoCheck for valid DM_DEVICE_INFO before proceeding
Hannes Reinecke [Wed, 23 Jul 2014 13:13:08 +0000 (15:13 +0200)]
Check for valid DM_DEVICE_INFO before proceeding

Calling 'DM_DEVICE_INFO' might succeed but the returned context
might refer to an invalid device. So one needs to check the 'exists'
field to avoid this.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoDo not treat 'transport-offline' paths as 'offline'
Hannes Reinecke [Wed, 23 Jul 2014 08:54:13 +0000 (10:54 +0200)]
Do not treat 'transport-offline' paths as 'offline'

'transport-offline' status is entered whenever the transport has
detected an offline device, but the SCSI midlayer still has
references to it.
Once the last reference drops the SCSI midlayer will properly
deregister the device and multipath will be updating the status.
But until then we should treat this device as simply 'path down',
and not marking it offline.
After all, some transports like FC will be entering the
'transport-offline' state after fast_io_fail triggered, and
might revert back to 'running' after reconfiguration.

References: bnc#888378

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoRevert 'return PATH_DOWN for quiesced paths'
Hannes Reinecke [Wed, 23 Jul 2014 08:51:35 +0000 (10:51 +0200)]
Revert 'return PATH_DOWN for quiesced paths'

Has been superseded by commit 00a2549a42707e5e1ef51ef6f638dc5ceaa15740

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipath: check for running daemon when called with '-u'
Hannes Reinecke [Wed, 9 Jul 2014 09:43:28 +0000 (11:43 +0200)]
multipath: check for running daemon when called with '-u'

When multipath is called to check for a valid device it really
should check if the daemon is running. Otherwise it'll blindly
assume the device to be eligible for multipathing even though
the daemon isn't running.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoUpdate hwtable for EMC XtremIO
Hannes Reinecke [Wed, 2 Jul 2014 07:10:05 +0000 (09:10 +0200)]
Update hwtable for EMC XtremIO

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: cleanup foreground operation
Hannes Reinecke [Tue, 24 Jun 2014 06:28:39 +0000 (08:28 +0200)]
multipathd: cleanup foreground operation

When specifying '-d' it actually does two things, namely do not
daemonize the program and prints logging messages to stdout
instead of syslog.
This partially clashes with '-s' which suppresses the timestamp
printing. So separate both use cases and ensure that '-d' doesn't
overwrite '-s' if specified later.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: handle DOMAP_RETRY
Hannes Reinecke [Tue, 24 Jun 2014 06:01:27 +0000 (08:01 +0200)]
multipathd: handle DOMAP_RETRY

Calling domap() will first check if all paths are available by
using 'flock' on each path. This might fail if other processes
are holding the lock already. In these cases we should retry
domap() as the processes should terminate eventually.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agokpartx.rules: Skip kpartx for multipath events
Hannes Reinecke [Fri, 13 Jun 2014 08:43:08 +0000 (10:43 +0200)]
kpartx.rules: Skip kpartx for multipath events

Whenever multipath signals a PATH_FAILED or PATH_REINSTATED
event we don't have to call 'kpartx'; the contents on the disk
haven't changed.
And we should never call kpartx if all paths are down, irrespective
of the uevent.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years ago11-dm-mpath.rules: Import blkid values if all paths are down
Hannes Reinecke [Thu, 12 Jun 2014 15:35:55 +0000 (17:35 +0200)]
11-dm-mpath.rules: Import blkid values if all paths are down

When all paths are failed we should not try to run any programs
requiring disk access. However, we still need to create the
symlinks so as not to confuse systemd.
So import the blkid values from the database in these cases.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipath.rules: fixup race condition with systemd
Hannes Reinecke [Tue, 20 May 2014 10:34:05 +0000 (12:34 +0200)]
multipath.rules: fixup race condition with systemd

systemd will call 'fsck' on any device, even if no device
type is found. This might cause a race between the mount call
and any fsck call on the device-mapper device.
So for multipath we should arbitrary set the device type to
'none', as then systemd with detect a missing 'fsck.none'
command and skip fsck here.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipath.rules: Whitelist devices
Hannes Reinecke [Fri, 16 May 2014 08:29:02 +0000 (10:29 +0200)]
multipath.rules: Whitelist devices

Currently multipath runs only on SCSI and DASD
devices (and cciss, but they are no longer supported).
So we can as well whitelist them and avoid the curious
warning about 'multipath -u /dev/loop failed'.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoInstall multipath rule under '56-multipath.rules'
Hannes Reinecke [Fri, 11 Apr 2014 09:59:43 +0000 (11:59 +0200)]
Install multipath rule under '56-multipath.rules'

The multipath rule needs to be inserted between the
sg3_utils rules for it to be able to intercept
symlink generation.

References: bnc#873151

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipath: implement option '-u' for uevents
Hannes Reinecke [Fri, 11 Apr 2014 09:23:00 +0000 (11:23 +0200)]
multipath: implement option '-u' for uevents

When calling 'multipath' from an uevent udev will not have
all information for that device, as it's being written into
the database _after_ the event has been processed.
This patch implements an option '-u' which uses the information
from the program environment when checking the device.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoRemove trailing linefeed from sysfs attributes
Hannes Reinecke [Fri, 11 Apr 2014 06:12:33 +0000 (08:12 +0200)]
Remove trailing linefeed from sysfs attributes

When reading from sysfs attributes might have a trailing linefeed,
which will then show up in the log messages as an additional
linefeed.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoMake checker_put() and prio_put() idempotent
Hannes Reinecke [Fri, 11 Apr 2014 06:10:36 +0000 (08:10 +0200)]
Make checker_put() and prio_put() idempotent

checker_put() and prio_put() should check whether an instance
was selected prior to dropping a reference to it.
Otherwise they'll be dropping a reference which they haven't
increased in the first place.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: implement 'list path <path>' cli command
Hannes Reinecke [Thu, 10 Apr 2014 11:02:35 +0000 (13:02 +0200)]
multipathd: implement 'list path <path>' cli command

Implement CLI command for printing the status of a single path.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoDisable reassign maps per default
Hannes Reinecke [Fri, 11 Apr 2014 06:03:08 +0000 (08:03 +0200)]
Disable reassign maps per default

lvm has undergone various updates, and should now check for
correct device assignment. So disable the 'reassign_maps'
feature per default.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoAllow for empty path argument when printing information
Hannes Reinecke [Fri, 11 Apr 2014 06:01:38 +0000 (08:01 +0200)]
Allow for empty path argument when printing information

When calling the various print functions we should be
allowing for empty path argument.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoFixup wwid blacklist printing
Hannes Reinecke [Thu, 10 Apr 2014 11:31:14 +0000 (13:31 +0200)]
Fixup wwid blacklist printing

When a wwid is blacklisted the log message looks like
'(null): wwid XXX blacklisted'.
Fix it up by printing either the corresponding device or
avoid printing the '(null)' entry if no corresponding
device is given.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoUse 'SCSI_IDENT_.*' as the default property whitelist
Hannes Reinecke [Thu, 10 Apr 2014 10:21:52 +0000 (12:21 +0200)]
Use 'SCSI_IDENT_.*' as the default property whitelist

59-scsi-sg_utils.rules export the VPD pages as
SCSI_IDENT_<association>_<type>.
So whenever we have a SCSI_IDENT_* property we know it
has come from VPD pages and we have a legit device.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agokpartx.rules: do not call blkid
Hannes Reinecke [Thu, 10 Apr 2014 08:39:01 +0000 (10:39 +0200)]
kpartx.rules: do not call blkid

blkid is already called by the device-mapper rules, no need
to do it ourselves.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoDo not print empty device strings during discovery
Hannes Reinecke [Wed, 9 Apr 2014 10:03:27 +0000 (12:03 +0200)]
Do not print empty device strings during discovery

If the path provides no information we should not print out
empty strings, but rather an instructive message telling us so.

References: bnc#872712

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agomultipathd: set correct PID when running in debug mode
Hannes Reinecke [Wed, 9 Apr 2014 07:18:17 +0000 (09:18 +0200)]
multipathd: set correct PID when running in debug mode

When running in debug mode the PID is not set, causing
the PID in the pidfile to always be set to '0'.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoFixup dependencies in multipathd.service
Hannes Reinecke [Wed, 9 Apr 2014 14:39:02 +0000 (16:39 +0200)]
Fixup dependencies in multipathd.service

The multipathd.service should be started after the .socket,
and it requires local-fs-pre to be reached.
Otherwise it won't be able to access /sys etc.

References: bnc#872712

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoFixup multipathd.socket to resolve ordering dependeny
Hannes Reinecke [Fri, 21 Mar 2014 11:26:06 +0000 (12:26 +0100)]
Fixup multipathd.socket to resolve ordering dependeny

Per default any socket file is started after /run is available,
as the socket might be placed there.
This results in a dependency cycle as multipathd requires the
socket, but the daemon should be started as early as possible,
and certainly before /run is mounted.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoAdd multipath rules for systemd support
Hannes Reinecke [Wed, 26 Feb 2014 13:36:43 +0000 (14:36 +0100)]
Add multipath rules for systemd support

Systemd and dracut rely on the DM_MULTIPATH_DEVICE_PATH setting
to mark the devices as used by multipathing. So add a ruleset
providing this setting.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoMake systemd installation path configurable
Hannes Reinecke [Tue, 21 Jan 2014 12:46:16 +0000 (13:46 +0100)]
Make systemd installation path configurable

Older systems might have systemd installed in /lib, so make
the path for systemd and udev configurable.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agolibmultipath: Skip paths with empty wwid
Hannes Reinecke [Fri, 14 Feb 2014 09:14:09 +0000 (10:14 +0100)]
libmultipath: Skip paths with empty wwid

If the WWID is empty this is pretty much a terminal error, as
we don't have anything to identify the multipath device.
So skip those devices during discovery.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agolibmultipath: Prefer deprecated 'getuid' callout
Hannes Reinecke [Fri, 14 Feb 2014 09:11:24 +0000 (10:11 +0100)]
libmultipath: Prefer deprecated 'getuid' callout

We need first to check if the deprecated 'getuid' callout
should be used, otherwise it might never be called.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agodiscovery: do not fail discovery on individual devices
Hannes Reinecke [Thu, 30 Jan 2014 10:41:30 +0000 (11:41 +0100)]
discovery: do not fail discovery on individual devices

Any errors returned from individual devices during discovery do
not invalidate the entire discovery. So instead of returning
the pointless accumulated error count we should be returning
the number of devices which which could not be discovered or
the negative error number if the discovery itself failed.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoDouble uevent stacksize yet again
Hannes Reinecke [Mon, 16 Dec 2013 14:15:10 +0000 (15:15 +0100)]
Double uevent stacksize yet again

Another report indicates that the uevent stacksize might still be
too small.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agolibmultipath: filter for missing property in get_refwwid()
Hannes Reinecke [Thu, 13 Feb 2014 15:33:27 +0000 (16:33 +0100)]
libmultipath: filter for missing property in get_refwwid()

get_refwwid() needs to filter for missing udev properties,
otherwise it'll accept just about any SCSI device.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agompath_persist: Do not call exit() from a shared library
Hannes Reinecke [Tue, 21 Jan 2014 15:13:05 +0000 (16:13 +0100)]
mpath_persist: Do not call exit() from a shared library

mpath_persist is a shared library, and shouldn't call exit().
Just return an error code here.

And cleanup formatting while we're at it.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agolibmultipath: remove compilation warning in devmapper.c
Hannes Reinecke [Mon, 16 Mar 2015 10:42:47 +0000 (11:42 +0100)]
libmultipath: remove compilation warning in devmapper.c

When LIBDM_API_DEFERRED is not defined the compiler would warn
about 'dm_cancel_remove_partmaps' being defined but never used.

Signed-off-by: Hannes Reinecke <hare@suse.de>
4 years agoAdd find_multipaths to multipath.conf man page
Benjamin Marzinski [Fri, 13 Mar 2015 18:27:27 +0000 (13:27 -0500)]
Add find_multipaths to multipath.conf man page

find_multipaths wasn't listed in the multipath.conf man page. Add it.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>