Bug 1313324 - Individual errors during multipath discovery invalidate the entire discovery.
Individual errors during multipath discovery invalidate the entire discovery.
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: device-mapper-multipath (Show other bugs)
Unspecified Linux
urgent Severity urgent
: rc
: ---
Assigned To: Ben Marzinski
Lin Li
Steven J. Levine
: ZStream
Depends On:
Blocks: 1203710 1309188 1328515 1343747
  Show dependency treegraph
Reported: 2016-03-01 07:08 EST by Gorka Eguileor
Modified: 2016-11-04 04:18 EDT (History)
9 users (show)

See Also:
Fixed In Version: device-mapper-multipath-0.4.9-88.el7
Doc Type: Bug Fix
Doc Text:
Failures on some devices no longer keep multipath from creating other devices Previously, the "multipath" command could fail to set up working devices because of failures on unrelated devices since it would quit early if it failed to get the information on any of the devices it was trying to create. With this fix, multipath no longer quits early if it fails to get information on some of the devices and failures on some devices no longer keep multipath from creating others.
Story Points: ---
Clone Of:
: 1328515 1343747 (view as bug list)
Last Closed: 2016-11-04 04:18:35 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Gorka Eguileor 2016-03-01 07:08:51 EST
Description of problem:

Under heavy load (creating and deleting many devices simultaneously) multipath calls (for example "multipath -l $device") will fail if during discovery phase there's a problem retrieving individual udev information from a device (because it has been deleted after listing the devices).  And the call should not fail.

In the code we get the list of devices [A], then if we fail at [B], we will end up returning a positive number at [C] and then caller will fail operation.

        udev_enumerate_add_match_subsystem(udev_iter, "block");
        udev_enumerate_scan_devices(udev_iter);   <-------------------------------- A
                                udev_enumerate_get_list_entry(udev_iter)) {
                const char *devtype;
                devpath = udev_list_entry_get_name(entry);
                condlog(4, "Discover device %s", devpath);
                udevice = udev_device_new_from_syspath(conf->udev, devpath); <----- B
                if (!udevice) {
                        condlog(4, "%s: no udev information", devpath);
                devtype = udev_device_get_devtype(udevice);
                if(devtype && !strncmp(devtype, "disk", 4))
                        r += path_discover(pathvec, conf, udevice, flag);

        condlog(4, "Discovery status %d", r);
        return r;   <-------------------------------------------------------------- C

I believe upstream commit 646e754853b123a075b4cede7d9ccf540e8c9b0c should fix this.
Comment 4 Ben Marzinski 2016-03-29 23:03:51 EDT
I've backported the upstream fix.
Comment 12 Jon 2016-04-05 08:50:57 EDT
Ben,  Yes the customer can use device-mapper-multipath-0.4.9-88.el7.  I will remove the zstream flag.  Thanks.
Comment 19 Raunak Kumar 2016-06-07 14:30:34 EDT
When is the ETA for this fix for both Redhat 6.8 and 7.2 ?
Comment 20 Ben Marzinski 2016-06-07 15:51:09 EDT
The rhel-7.2 zstream for this bug is Bug 1328515, which has already been released. There is currently no rhel-6 version of this bug. I can create one, but rhel-6.8 has been out for a while, and rhel-6.9 is not yet in the planning phase. You'd need to talk to a support person about getting a 6.8 zstream.
Comment 21 Ben Marzinski 2016-06-07 16:02:30 EDT
The rhel-6 bug is Bug 1343747
Comment 24 errata-xmlrpc 2016-11-04 04:18:35 EDT
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.