Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package multipath-tools for openSUSE:Factory
checked in at 2022-02-17 00:30:17
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/multipath-tools (Old)
and /work/SRC/openSUSE:Factory/.multipath-tools.new.1956 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "multipath-tools"
Thu Feb 17 00:30:17 2022 rev:140 rq:955369 version:0.8.8+64+suse.f265f7e0
Changes:
--------
--- /work/SRC/openSUSE:Factory/multipath-tools/multipath-tools.changes
2022-02-10 23:12:16.392238262 +0100
+++
/work/SRC/openSUSE:Factory/.multipath-tools.new.1956/multipath-tools.changes
2022-02-17 00:31:03.737427801 +0100
@@ -1,0 +2,16 @@
+Wed Feb 16 13:24:47 UTC 2022 - Martin Wilck <mwi...@suse.com>
+
+- Update to version 0.8.8+64+suse.f265f7e0:
+ * libmultipath: fix printing native nvme multipath topology
+ (bsc#1196011)
+ * libmultipath: add %L path wildcard for 64-bit hex LUN
+ * libmultipath: support host adapter name lookup for s390x ccw bus
+
+-------------------------------------------------------------------
+Tue Feb 15 16:35:10 UTC 2022 - Martin Wilck <mwi...@suse.com>
+
+- Update to version 0.8.8+60+suse.4c5922cb:
+ * multipathd: add suppport for FC Fabric Performance Impact
+ Notifications (FPIN) (bsc#1195506)
+
+-------------------------------------------------------------------
Old:
----
multipath-tools-0.8.8+57+suse.dfb672fe.obscpio
New:
----
multipath-tools-0.8.8+64+suse.f265f7e0.obscpio
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ multipath-tools.spec ++++++
--- /var/tmp/diff_new_pack.ATKA3H/_old 2022-02-17 00:31:04.661427642 +0100
+++ /var/tmp/diff_new_pack.ATKA3H/_new 2022-02-17 00:31:04.669427640 +0100
@@ -29,7 +29,7 @@
%define libdmmp_version %(echo %{_libdmmp_version} | tr . _)
Name: multipath-tools
-Version: 0.8.8+57+suse.dfb672fe
+Version: 0.8.8+64+suse.f265f7e0
Release: 0
Summary: Tools to Manage Multipathed Devices with the device-mapper
License: GPL-2.0-only
++++++ _servicedata ++++++
--- /var/tmp/diff_new_pack.ATKA3H/_old 2022-02-17 00:31:04.741427628 +0100
+++ /var/tmp/diff_new_pack.ATKA3H/_new 2022-02-17 00:31:04.745427627 +0100
@@ -1,6 +1,6 @@
<servicedata>
<service name="tar_scm">
<param
name="url">https://github.com/openSUSE/multipath-tools.git</param>
- <param
name="changesrevision">dfb672fefaf30f5eb30ef5bfb717d5eefc425e9e</param></service></servicedata>
+ <param
name="changesrevision">f265f7e0e522325065d521b770663730dec5995d</param></service></servicedata>
(No newline at EOF)
++++++ multipath-tools-0.8.8+57+suse.dfb672fe.obscpio ->
multipath-tools-0.8.8+64+suse.f265f7e0.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/multipath-tools-0.8.8+57+suse.dfb672fe/Makefile.inc
new/multipath-tools-0.8.8+64+suse.f265f7e0/Makefile.inc
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/Makefile.inc 2022-02-09
10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/Makefile.inc 2022-02-16
10:32:37.000000000 +0100
@@ -159,6 +159,19 @@
echo "$$found" \
)
+# Check whether a file contains a variable with name $1 in header file $2
+check_var = $(shell \
+ if grep -Eq "(^|[[:blank:]])$1([[:blank:]]|=|$$)" "$2"; then \
+ found=1; \
+ status="yes"; \
+ else \
+ found=0; \
+ status="no"; \
+ fi; \
+ echo 1>&2 "Checking for .. $1 in $2 ... $$status"; \
+ echo "$$found" \
+ )
+
%.o: %.c
@echo building $@ because of $?
$(CC) $(CFLAGS) $(CPPFLAGS) -c -o $@ $<
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/Makefile
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/Makefile
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/Makefile
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/Makefile
2022-02-16 10:32:37.000000000 +0100
@@ -45,6 +45,11 @@
CFLAGS += -DLIBDM_API_HOLD_CONTROL
endif
+ifneq ($(call
check_var,ELS_DTAG_LNK_INTEGRITY,$(LINUX_HEADERS_INCDIR)/scsi/fc/fc_els.h),0)
+ CFLAGS += -DFPIN_EVENT_HANDLER
+endif
+
+
OBJS = parser.o vector.o devmapper.o callout.o \
hwtable.o blacklist.o util.o dmparser.o config.o \
structs.o discovery.o propsel.o dict.o \
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/dict.c
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/dict.c
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/dict.c
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/dict.c
2022-02-16 10:32:37.000000000 +0100
@@ -512,6 +512,59 @@
find_multipaths_optvals[conf->find_multipaths]);
}
+static const char * const marginal_pathgroups_optvals[] = {
+ [MARGINAL_PATHGROUP_OFF] = "off",
+ [MARGINAL_PATHGROUP_ON] = "on",
+#ifdef FPIN_EVENT_HANDLER
+ [MARGINAL_PATHGROUP_FPIN] = "fpin",
+#endif
+};
+
+static int
+def_marginal_pathgroups_handler(struct config *conf, vector strvec,
+ const char *file, int line_nr)
+{
+ char *buff;
+ unsigned int i;
+
+ buff = set_value(strvec);
+ if (!buff)
+ return 1;
+ for (i = MARGINAL_PATHGROUP_OFF;
+ i < ARRAY_SIZE(marginal_pathgroups_optvals); i++) {
+ if (marginal_pathgroups_optvals[i] != NULL &&
+ !strcmp(buff, marginal_pathgroups_optvals[i])) {
+ conf->marginal_pathgroups = i;
+ break;
+ }
+ }
+
+ if (i >= ARRAY_SIZE(marginal_pathgroups_optvals)) {
+ if (strcmp(buff, "no") == 0 || strcmp(buff, "0") == 0)
+ conf->marginal_pathgroups = MARGINAL_PATHGROUP_OFF;
+ else if (strcmp(buff, "yes") == 0 || strcmp(buff, "1") == 0)
+ conf->marginal_pathgroups = MARGINAL_PATHGROUP_ON;
+ /* This can only be true if FPIN_EVENT_HANDLER isn't defined,
+ * otherwise this check will have already happened above */
+ else if (strcmp(buff, "fpin") == 0)
+ condlog(1, "%s line %d, support for \"fpin\" is not
compiled in for marginal_pathgroups", file, line_nr);
+ else
+ condlog(1, "%s line %d, invalid value for
marginal_pathgroups: \"%s\"",
+ file, line_nr, buff);
+ }
+ free(buff);
+ return 0;
+}
+
+static int
+snprint_def_marginal_pathgroups(struct config *conf, struct strbuf *buff,
+ const void *data)
+{
+ return append_strbuf_quoted(buff,
+
marginal_pathgroups_optvals[conf->marginal_pathgroups]);
+}
+
+
declare_def_handler(selector, set_str)
declare_def_snprint_defstr(selector, print_str, DEFAULT_SELECTOR)
declare_hw_handler(selector, set_str)
@@ -1526,9 +1579,6 @@
declare_hw_handler(all_tg_pt, set_yes_no_undef)
declare_hw_snprint(all_tg_pt, print_yes_no_undef)
-declare_def_handler(marginal_pathgroups, set_yes_no)
-declare_def_snprint(marginal_pathgroups, print_yes_no)
-
declare_def_handler(recheck_wwid, set_yes_no_undef)
declare_def_snprint_defint(recheck_wwid, print_yes_no_undef,
DEFAULT_RECHECK_WWID)
declare_ovr_handler(recheck_wwid, set_yes_no_undef)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/discovery.c
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/discovery.c
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/discovery.c
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/discovery.c
2022-02-16 10:32:37.000000000 +0100
@@ -475,60 +475,33 @@
return 0;
}
-int sysfs_get_host_adapter_name(const struct path *pp, char *adapter_name)
-{
- int proto_id;
-
- if (!pp || !adapter_name)
- return 1;
-
- proto_id = pp->sg_id.proto_id;
-
- if (proto_id != SCSI_PROTOCOL_FCP &&
- proto_id != SCSI_PROTOCOL_SAS &&
- proto_id != SCSI_PROTOCOL_ISCSI &&
- proto_id != SCSI_PROTOCOL_SRP) {
- return 1;
- }
- /* iscsi doesn't have adapter info in sysfs
- * get ip_address for grouping paths
- */
- if (pp->sg_id.proto_id == SCSI_PROTOCOL_ISCSI)
- return sysfs_get_iscsi_ip_address(pp, adapter_name);
-
- /* fetch adapter pci name for other protocols
- */
- return sysfs_get_host_pci_name(pp, adapter_name);
-}
-
-int sysfs_get_host_pci_name(const struct path *pp, char *pci_name)
+static int sysfs_get_host_bus_id(const struct path *pp, char *bus_id)
{
struct udev_device *hostdev, *parent;
char host_name[HOST_NAME_LEN];
- const char *driver_name, *value;
+ const char *driver_name, *subsystem_name, *value;
- if (!pp || !pci_name)
+ if (!pp || !bus_id)
return 1;
- sprintf(host_name, "host%d", pp->sg_id.host_no);
+ snprintf(host_name, sizeof(host_name), "host%d", pp->sg_id.host_no);
hostdev = udev_device_new_from_subsystem_sysname(udev,
"scsi_host", host_name);
if (!hostdev)
return 1;
- parent = udev_device_get_parent(hostdev);
- while (parent) {
+ for (parent = udev_device_get_parent(hostdev);
+ parent;
+ parent = udev_device_get_parent(parent)) {
driver_name = udev_device_get_driver(parent);
- if (!driver_name) {
- parent = udev_device_get_parent(parent);
- continue;
- }
- if (!strcmp(driver_name, "pcieport"))
+ subsystem_name = udev_device_get_subsystem(parent);
+ if (driver_name && !strcmp(driver_name, "pcieport"))
+ break;
+ if (subsystem_name && !strcmp(subsystem_name, "ccw"))
break;
- parent = udev_device_get_parent(parent);
}
if (parent) {
- /* pci_device found
+ /* pci_device or ccw fcp device found
*/
value = udev_device_get_sysname(parent);
@@ -537,7 +510,7 @@
return 1;
}
- strncpy(pci_name, value, SLOT_NAME_SIZE);
+ strlcpy(bus_id, value, SLOT_NAME_SIZE);
udev_device_unref(hostdev);
return 0;
}
@@ -545,6 +518,32 @@
return 1;
}
+int sysfs_get_host_adapter_name(const struct path *pp, char *adapter_name)
+{
+ int proto_id;
+
+ if (!pp || !adapter_name)
+ return 1;
+
+ proto_id = pp->sg_id.proto_id;
+
+ if (proto_id != SCSI_PROTOCOL_FCP &&
+ proto_id != SCSI_PROTOCOL_SAS &&
+ proto_id != SCSI_PROTOCOL_ISCSI &&
+ proto_id != SCSI_PROTOCOL_SRP) {
+ return 1;
+ }
+ /* iscsi doesn't have adapter info in sysfs
+ * get ip_address for grouping paths
+ */
+ if (pp->sg_id.proto_id == SCSI_PROTOCOL_ISCSI)
+ return sysfs_get_iscsi_ip_address(pp, adapter_name);
+
+ /* fetch adapter bus-ID for other protocols
+ */
+ return sysfs_get_host_bus_id(pp, adapter_name);
+}
+
int sysfs_get_iscsi_ip_address(const struct path *pp, char *ip_address)
{
struct udev_device *hostdev;
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/discovery.h
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/discovery.h
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/discovery.h
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/discovery.h
2022-02-16 10:32:37.000000000 +0100
@@ -44,7 +44,6 @@
struct path **pp_ptr);
int sysfs_set_scsi_tmo (struct multipath *mpp, unsigned int checkint);
int sysfs_get_timeout(const struct path *pp, unsigned int *timeout);
-int sysfs_get_host_pci_name(const struct path *pp, char *pci_name);
int sysfs_get_iscsi_ip_address(const struct path *pp, char *ip_address);
int sysfs_get_host_adapter_name(const struct path *pp,
char *adapter_name);
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/foreign/nvme.c
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/foreign/nvme.c
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/foreign/nvme.c
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/foreign/nvme.c
2022-02-16 10:32:37.000000000 +0100
@@ -337,7 +337,7 @@
static int nvme_style(__attribute__((unused)) const struct gen_multipath* gm,
struct strbuf *buf, __attribute__((unused)) int verbosity)
{
- return append_strbuf_str(buf, "%%w [%%G]:%%d %%s");
+ return append_strbuf_str(buf, "%w [%G]:%d %s");
}
static const struct gen_multipath_ops nvme_map_ops = {
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/libmultipath.version
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/libmultipath.version
---
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/libmultipath.version
2022-02-09 10:49:48.000000000 +0100
+++
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/libmultipath.version
2022-02-16 10:32:37.000000000 +0100
@@ -56,6 +56,7 @@
check_foreign;
cleanup_charp;
cleanup_lock;
+ cleanup_mutex;
cleanup_ucharp;
close_fd;
coalesce_paths;
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/print.c
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/print.c
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/print.c
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/print.c
2022-02-16 10:32:37.000000000 +0100
@@ -95,6 +95,7 @@
" \"host_wwpn\" : \"%R\",\n" \
" \"target_wwpn\" : \"%r\",\n" \
" \"host_adapter\" : \"%a\",\n" \
+ " \"lun_hex\" : \"%L\",\n" \
" \"marginal_st\" : \"%M\""
#define PROGRESS_LEN 10
@@ -452,6 +453,24 @@
pp->sg_id.lun);
}
+
+static int
+snprint_path_lunhex (struct strbuf *buff, const struct path * pp)
+{
+ uint64_t lunhex = SCSI_INVALID_LUN, scsilun;
+
+ if (!pp || pp->sg_id.host_no < 0)
+ return print_strbuf(buff, "0x%016" PRIx64, lunhex);
+
+ scsilun = pp->sg_id.lun;
+ /* cf. Linux kernel function int_to_scsilun() */
+ lunhex = ((scsilun & 0x000000000000ffffULL) << 48) |
+ ((scsilun & 0x00000000ffff0000ULL) << 16) |
+ ((scsilun & 0x0000ffff00000000ULL) >> 16) |
+ ((scsilun & 0xffff000000000000ULL) >> 48);
+ return print_strbuf(buff, "0x%016" PRIx64, lunhex);
+}
+
static int
snprint_dev (struct strbuf *buff, const struct path * pp)
{
@@ -843,6 +862,7 @@
{'0', "failures", snprint_path_failures},
{'P', "protocol", snprint_path_protocol},
{'I', "init_st", snprint_initialized},
+ {'L', "LUN hex", snprint_path_lunhex},
};
static const struct pathgroup_data pgd[] = {
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/propsel.c
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/propsel.c
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/propsel.c
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/propsel.c
2022-02-16 10:32:37.000000000 +0100
@@ -83,6 +83,8 @@
"(setting: multipath command line [-p] flag)";
static const char autodetect_origin[] =
"(setting: storage device autodetected)";
+static const char fpin_marginal_path_origin[] =
+ "(setting: overridden by marginal_path_fpin)";
static const char marginal_path_origin[] =
"(setting: implied by marginal_path check)";
static const char delay_watch_origin[] =
@@ -1050,9 +1052,12 @@
const char *origin;
STRBUF_ON_STACK(buff);
- if (marginal_path_check_enabled(mp)) {
+ if (marginal_path_check_enabled(mp) || (conf->marginal_pathgroups ==
MARGINAL_PATHGROUP_FPIN)) {
mp->san_path_err_threshold = NU_NO;
- origin = marginal_path_origin;
+ if (conf->marginal_pathgroups == MARGINAL_PATHGROUP_FPIN)
+ origin = fpin_marginal_path_origin;
+ else
+ origin = marginal_path_origin;
goto out;
}
mp_set_mpe(san_path_err_threshold);
@@ -1073,9 +1078,12 @@
const char *origin;
STRBUF_ON_STACK(buff);
- if (marginal_path_check_enabled(mp)) {
+ if (marginal_path_check_enabled(mp) || (conf->marginal_pathgroups ==
MARGINAL_PATHGROUP_FPIN)) {
mp->san_path_err_forget_rate = NU_NO;
- origin = marginal_path_origin;
+ if (conf->marginal_pathgroups == MARGINAL_PATHGROUP_FPIN)
+ origin = fpin_marginal_path_origin;
+ else
+ origin = marginal_path_origin;
goto out;
}
mp_set_mpe(san_path_err_forget_rate);
@@ -1097,9 +1105,12 @@
const char *origin;
STRBUF_ON_STACK(buff);
- if (marginal_path_check_enabled(mp)) {
+ if (marginal_path_check_enabled(mp) || (conf->marginal_pathgroups ==
MARGINAL_PATHGROUP_FPIN)) {
mp->san_path_err_recovery_time = NU_NO;
- origin = marginal_path_origin;
+ if (conf->marginal_pathgroups == MARGINAL_PATHGROUP_FPIN)
+ origin = fpin_marginal_path_origin;
+ else
+ origin = marginal_path_origin;
goto out;
}
mp_set_mpe(san_path_err_recovery_time);
@@ -1121,6 +1132,12 @@
const char *origin;
STRBUF_ON_STACK(buff);
+ if (conf->marginal_pathgroups == MARGINAL_PATHGROUP_FPIN) {
+ mp->marginal_path_err_sample_time = NU_NO;
+ origin = fpin_marginal_path_origin;
+ goto out;
+ }
+
mp_set_mpe(marginal_path_err_sample_time);
mp_set_ovr(marginal_path_err_sample_time);
mp_set_hwe(marginal_path_err_sample_time);
@@ -1144,6 +1161,12 @@
const char *origin;
STRBUF_ON_STACK(buff);
+ if (conf->marginal_pathgroups == MARGINAL_PATHGROUP_FPIN) {
+ mp->marginal_path_err_rate_threshold = NU_NO;
+ origin = fpin_marginal_path_origin;
+ goto out;
+ }
+
mp_set_mpe(marginal_path_err_rate_threshold);
mp_set_ovr(marginal_path_err_rate_threshold);
mp_set_hwe(marginal_path_err_rate_threshold);
@@ -1161,6 +1184,12 @@
const char *origin;
STRBUF_ON_STACK(buff);
+ if (conf->marginal_pathgroups == MARGINAL_PATHGROUP_FPIN) {
+ mp->marginal_path_err_recheck_gap_time = NU_NO;
+ origin = fpin_marginal_path_origin;
+ goto out;
+ }
+
mp_set_mpe(marginal_path_err_recheck_gap_time);
mp_set_ovr(marginal_path_err_recheck_gap_time);
mp_set_hwe(marginal_path_err_recheck_gap_time);
@@ -1179,6 +1208,12 @@
const char *origin;
STRBUF_ON_STACK(buff);
+ if (conf->marginal_pathgroups == MARGINAL_PATHGROUP_FPIN) {
+ mp->marginal_path_double_failed_time = NU_NO;
+ origin = fpin_marginal_path_origin;
+ goto out;
+ }
+
mp_set_mpe(marginal_path_double_failed_time);
mp_set_ovr(marginal_path_double_failed_time);
mp_set_hwe(marginal_path_double_failed_time);
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/structs.h
new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/structs.h
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/libmultipath/structs.h
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/libmultipath/structs.h
2022-02-16 10:32:37.000000000 +0100
@@ -111,6 +111,12 @@
__FIND_MULTIPATHS_LAST,
};
+enum marginal_pathgroups_mode {
+ MARGINAL_PATHGROUP_OFF = YN_NO,
+ MARGINAL_PATHGROUP_ON = YN_YES,
+ MARGINAL_PATHGROUP_FPIN,
+};
+
enum flush_states {
FLUSH_UNDEF = YNU_UNDEF,
FLUSH_DISABLED = YNU_NO,
@@ -419,6 +425,7 @@
unsigned char prflag;
int all_tg_pt;
struct gen_multipath generic_mp;
+ bool fpin_must_reload;
};
static inline int marginal_path_check_enabled(const struct multipath *mpp)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/multipath/multipath.conf.5
new/multipath-tools-0.8.8+64+suse.f265f7e0/multipath/multipath.conf.5
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/multipath/multipath.conf.5
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/multipath/multipath.conf.5
2022-02-16 10:32:37.000000000 +0100
@@ -1095,23 +1095,32 @@
.
.TP
.B marginal_pathgroups
-If set to \fIno\fR, the \fIdelay_*_checks\fR, \fImarginal_path_*\fR, and
+If set to \fIoff\fR, the \fIdelay_*_checks\fR, \fImarginal_path_*\fR, and
\fIsan_path_err_*\fR options will keep marginal, or \(dqshaky\(dq, paths from
being reinstated until they have been monitored for some time. This can cause
situations where all non-marginal paths are down, and no paths are usable
until multipathd detects this and reinstates a marginal path. If the multipath
device is not configured to queue IO in this case, it can cause IO errors to
occur, even though there are marginal paths available. However, if this
-option is set to \fIyes\fR, when one of the marginal path detecting methods
+option is set to \fIon\fR, when one of the marginal path detecting methods
determines that a path is marginal, it will be reinstated and placed in a
separate pathgroup that will only be used after all the non-marginal pathgroups
have been tried first. This prevents the possibility of IO errors occurring
while marginal paths are still usable. After the path has been monitored
for the configured time, and is declared healthy, it will be returned to its
-normal pathgroup. See "Shaky paths detection" below for more information.
+normal pathgroup.
+If this option is set to \fIfpin\fR, multipathd will receive fpin
+notifications, set path states to "marginal" accordingly, and regroup paths
+as described for \fIon\fR. This option can't be used in combination
+with other options for "Shaky path detection" (see below). \fBNote:\fR If this
+is set to \fIfpin\fR, the \fImarginal_path_*\fR and \fIsan_path_err_*\fR
+options are implicitly set to \fIno\fP. Also, this option cannot be switched
+either to or from \fIfpin\fR on a multipathd reconfigure. multipathd must be
+restarted for the change to take effect.
+See "Shaky paths detection" below for more information.
.RS
.TP
-The default is: \fBno\fR
+The default is: \fBoff\fR
.RE
.
.
@@ -1848,6 +1857,13 @@
path checks by multipathd, which is variable and controlled by the
\fIpolling_interval\fR and \fImax_polling_interval\fR parameters.
.
+.TP
+.B \(dqFPIN \(dq failure tracking
+Fibre channel fabrics can notify hosts about fabric-level issues such
+as integrity failures or congestion with so-called Fabric Performance
+Impact Notifications (FPINs).On receiving the fpin notifications through ELS
+multipathd will move the affected path and port states to marginal.
+.
.RS 8
.LP
This method is \fBdeprecated\fR in favor of the \(dqmarginal_path\(dq failure
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/multipathd/Makefile
new/multipath-tools-0.8.8+64+suse.f265f7e0/multipathd/Makefile
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/multipathd/Makefile
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/multipathd/Makefile
2022-02-16 10:32:37.000000000 +0100
@@ -4,6 +4,10 @@
CFLAGS += -DLIBDM_API_GET_ERRNO
endif
+ifneq ($(call
check_var,ELS_DTAG_LNK_INTEGRITY,$(LINUX_HEADERS_INCDIR)/scsi/fc/fc_els.h),0)
+ CFLAGS += -DFPIN_EVENT_HANDLER
+ FPIN_SUPPORT = 1
+endif
#
# debugging stuff
#
@@ -34,6 +38,12 @@
OBJS = main.o pidfile.o uxlsnr.o uxclnt.o cli.o cli_handlers.o waiter.o \
dmevents.o init_unwinder.o
+ifeq ($(FPIN_SUPPORT),1)
+OBJS += fpin_handlers.o
+endif
+
+
+
EXEC = multipathd
all : $(EXEC)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/multipathd/fpin.h
new/multipath-tools-0.8.8+64+suse.f265f7e0/multipathd/fpin.h
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/multipathd/fpin.h
1970-01-01 01:00:00.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/multipathd/fpin.h
2022-02-16 10:32:37.000000000 +0100
@@ -0,0 +1,20 @@
+#ifndef __FPIN_H__
+#define __FPIN_H__
+
+#ifdef FPIN_EVENT_HANDLER
+void *fpin_fabric_notification_receiver(void *unused);
+void *fpin_els_li_consumer(void *data);
+void fpin_clean_marginal_dev_list(__attribute__((unused)) void *arg);
+#else
+static void *fpin_fabric_notification_receiver(__attribute__((unused))void
*unused)
+{
+ return NULL;
+}
+static void *fpin_els_li_consumer(__attribute__((unused))void *data)
+{
+ return NULL;
+}
+/* fpin_clean_marginal_dev_list() is never called */
+#endif
+
+#endif
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/multipathd/fpin_handlers.c
new/multipath-tools-0.8.8+64+suse.f265f7e0/multipathd/fpin_handlers.c
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/multipathd/fpin_handlers.c
1970-01-01 01:00:00.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/multipathd/fpin_handlers.c
2022-02-16 10:32:37.000000000 +0100
@@ -0,0 +1,540 @@
+#include <errno.h>
+#include <unistd.h>
+#include <sys/types.h>
+#include <sys/socket.h>
+#include <libudev.h>
+#include <scsi/scsi_netlink_fc.h>
+#include <scsi/fc/fc_els.h>
+
+#include "parser.h"
+#include "vector.h"
+#include "structs.h"
+#include "structs_vec.h"
+#include "main.h"
+#include "debug.h"
+#include "util.h"
+#include "sysfs.h"
+
+#include "fpin.h"
+#include "devmapper.h"
+
+static pthread_cond_t fpin_li_cond = PTHREAD_COND_INITIALIZER;
+static pthread_mutex_t fpin_li_mutex = PTHREAD_MUTEX_INITIALIZER;
+static pthread_mutex_t fpin_li_marginal_dev_mutex = PTHREAD_MUTEX_INITIALIZER;
+
+static LIST_HEAD(els_marginal_list_head);
+static LIST_HEAD(fpin_li_marginal_dev_list_head);
+
+
+#define DEF_RX_BUF_SIZE 4096
+#define DEV_NAME_LEN 128
+#define FCH_EVT_LINKUP 0x2
+#define FCH_EVT_LINK_FPIN 0x501
+#define FCH_EVT_RSCN 0x5
+
+#define list_first_entry(ptr, type, member) \
+ list_entry((ptr)->next, type, member)
+
+/* max ELS frame Size */
+#define FC_PAYLOAD_MAXLEN 2048
+
+struct els_marginal_list {
+ uint32_t event_code;
+ uint16_t host_num;
+ uint16_t length;
+ char payload[FC_PAYLOAD_MAXLEN];
+ struct list_head node;
+};
+/* Structure to store the marginal devices info */
+struct marginal_dev_list {
+ char dev_t[BLK_DEV_SIZE];
+ uint32_t host_num;
+ struct list_head node;
+};
+
+static void _udev_device_unref(void *p)
+{
+ udev_device_unref(p);
+}
+
+
+/*set/unset the path state to marginal*/
+static int fpin_set_pathstate(struct path *pp, bool set)
+{
+ const char *action = set ? "set" : "unset";
+
+ if (!pp || !pp->mpp || !pp->mpp->alias)
+ return -1;
+
+ condlog(3, "\n%s: %s marginal path %s (fpin)",
+ action, pp->mpp->alias, pp->dev_t);
+ pp->marginal = set;
+ pp->mpp->fpin_must_reload = true;
+ return 0;
+}
+
+/* This will unset marginal state of a device*/
+static void fpin_path_unsetmarginal(char *devname, struct vectors *vecs)
+{
+ struct path *pp;
+
+ pp = find_path_by_dev(vecs->pathvec, devname);
+ if (!pp)
+ pp = find_path_by_devt(vecs->pathvec, devname);
+
+ fpin_set_pathstate(pp, false);
+}
+
+/*This will set the marginal state of a device*/
+static int fpin_path_setmarginal(struct path *pp)
+{
+ return fpin_set_pathstate(pp, true);
+}
+
+/* Unsets all the devices in the list from marginal state */
+static void
+fpin_unset_marginal_dev(uint32_t host_num, struct vectors *vecs)
+{
+ struct marginal_dev_list *tmp_marg = NULL;
+ struct marginal_dev_list *marg = NULL;
+ struct multipath *mpp;
+ int ret = 0;
+ int i;
+
+ pthread_cleanup_push(cleanup_lock, &vecs->lock);
+ lock(&vecs->lock);
+ pthread_testcancel();
+
+ pthread_mutex_lock(&fpin_li_marginal_dev_mutex);
+ pthread_cleanup_push(cleanup_mutex, &fpin_li_marginal_dev_mutex);
+ pthread_testcancel();
+ if (list_empty(&fpin_li_marginal_dev_list_head)) {
+ condlog(4, "Marginal List is empty\n");
+ goto empty;
+ }
+ list_for_each_entry_safe(marg, tmp_marg,
&fpin_li_marginal_dev_list_head, node) {
+ if (marg->host_num != host_num)
+ continue;
+ condlog(4, " unsetting marginal dev: is %s %d\n",
+ tmp_marg->dev_t, tmp_marg->host_num);
+ fpin_path_unsetmarginal(marg->dev_t, vecs);
+ list_del(&marg->node);
+ free(marg);
+ }
+empty:
+ pthread_cleanup_pop(1);
+ /* walk backwards because reload_and_sync_map() can remove mpp */
+ vector_foreach_slot_backwards(vecs->mpvec, mpp, i) {
+ if (mpp->fpin_must_reload) {
+ ret = reload_and_sync_map(mpp, vecs, 0);
+ if (ret == 2)
+ condlog(2, "map removed during reload");
+ else
+ mpp->fpin_must_reload = false;
+ }
+ }
+ pthread_cleanup_pop(1);
+}
+
+/*
+ * On Receiving the frame from HBA driver, insert the frame into link
+ * integrity frame list which will be picked up later by consumer thread for
+ * processing.
+ */
+static int
+fpin_els_add_li_frame(struct fc_nl_event *fc_event)
+{
+ struct els_marginal_list *els_mrg = NULL;
+ int ret = 0;
+
+ if (fc_event->event_datalen > FC_PAYLOAD_MAXLEN)
+ return -EINVAL;
+
+ pthread_mutex_lock(&fpin_li_mutex);
+ pthread_cleanup_push(cleanup_mutex, &fpin_li_mutex);
+ pthread_testcancel();
+ els_mrg = calloc(1, sizeof(struct els_marginal_list));
+ if (els_mrg != NULL) {
+ els_mrg->host_num = fc_event->host_no;
+ els_mrg->event_code = fc_event->event_code;
+ els_mrg->length = fc_event->event_datalen;
+ memcpy(els_mrg->payload, &(fc_event->event_data),
fc_event->event_datalen);
+ list_add_tail(&els_mrg->node, &els_marginal_list_head);
+ pthread_cond_signal(&fpin_li_cond);
+ } else
+ ret = -ENOMEM;
+ pthread_cleanup_pop(1);
+ return ret;
+
+}
+
+/*Sets the rport port_state to marginal*/
+static void fpin_set_rport_marginal(struct udev_device *rport_dev)
+{
+ sysfs_attr_set_value(rport_dev, "port_state",
+ "Marginal", strlen("Marginal"));
+}
+
+/*Add the marginal devices info into the list*/
+static void
+fpin_add_marginal_dev_info(uint32_t host_num, char *devname)
+{
+ struct marginal_dev_list *newdev = NULL;
+
+ newdev = calloc(1, sizeof(struct marginal_dev_list));
+ if (newdev != NULL) {
+ newdev->host_num = host_num;
+ strlcpy(newdev->dev_t, devname, BLK_DEV_SIZE);
+ condlog(4, "\n%s hostno %d devname %s\n", __func__,
+ host_num, newdev->dev_t);
+ pthread_mutex_lock(&fpin_li_marginal_dev_mutex);
+ list_add_tail(&(newdev->node),
+ &fpin_li_marginal_dev_list_head);
+ pthread_mutex_unlock(&fpin_li_marginal_dev_mutex);
+ }
+}
+
+/*
+ * This function goes through the vecs->pathvec, and for
+ * each path, check that the host number,
+ * the target WWPN associated with the path matches
+ * with the els wwpn and sets the path and port state to
+ * Marginal
+ */
+static int fpin_chk_wwn_setpath_marginal(uint16_t host_num, struct vectors
*vecs,
+ uint64_t els_wwpn)
+{
+ struct path *pp;
+ struct multipath *mpp;
+ int i, k;
+ char rport_id[42];
+ const char *value = NULL;
+ struct udev_device *rport_dev = NULL;
+ uint64_t wwpn;
+ int ret = 0;
+
+ pthread_cleanup_push(cleanup_lock, &vecs->lock);
+ lock(&vecs->lock);
+ pthread_testcancel();
+
+ vector_foreach_slot(vecs->pathvec, pp, k) {
+ /* Checks the host number and also for the SCSI FCP */
+ if (pp->sg_id.proto_id != SCSI_PROTOCOL_FCP || host_num !=
pp->sg_id.host_no)
+ continue;
+ sprintf(rport_id, "rport-%d:%d-%d",
+ pp->sg_id.host_no, pp->sg_id.channel,
pp->sg_id.transport_id);
+ rport_dev = udev_device_new_from_subsystem_sysname(udev,
+ "fc_remote_ports", rport_id);
+ if (!rport_dev) {
+ condlog(2, "%s: No fc_remote_port device for '%s'",
pp->dev,
+ rport_id);
+ continue;
+ }
+ pthread_cleanup_push(_udev_device_unref, rport_dev);
+ value = udev_device_get_sysattr_value(rport_dev, "port_name");
+ if (!value)
+ goto unref;
+
+ if (value)
+ wwpn = strtol(value, NULL, 16);
+ /*
+ * If the port wwpn matches sets the path and port state
+ * to marginal
+ */
+ if (wwpn == els_wwpn) {
+ ret = fpin_path_setmarginal(pp);
+ if (ret < 0)
+ goto unref;
+ fpin_set_rport_marginal(rport_dev);
+ fpin_add_marginal_dev_info(host_num, pp->dev);
+ }
+unref:
+ pthread_cleanup_pop(1);
+ }
+ /* walk backwards because reload_and_sync_map() can remove mpp */
+ vector_foreach_slot_backwards(vecs->mpvec, mpp, i) {
+ if (mpp->fpin_must_reload) {
+ ret = reload_and_sync_map(mpp, vecs, 0);
+ if (ret == 2)
+ condlog(2, "map removed during reload");
+ else
+ mpp->fpin_must_reload = false;
+ }
+ }
+ pthread_cleanup_pop(1);
+ return ret;
+}
+
+/*
+ * This function loops around all the impacted wwns received as part of els
+ * frame and sets the associated path and port states to marginal.
+ */
+static int
+fpin_parse_li_els_setpath_marginal(uint16_t host_num, struct fc_tlv_desc *tlv,
+ struct vectors *vecs)
+{
+ uint32_t wwn_count = 0, iter = 0;
+ uint64_t wwpn;
+ struct fc_fn_li_desc *li_desc = (struct fc_fn_li_desc *)tlv;
+ int count = 0;
+ int ret = 0;
+
+ /* Update the wwn to list */
+ wwn_count = be32_to_cpu(li_desc->pname_count);
+ condlog(4, "Got wwn count as %d\n", wwn_count);
+
+ for (iter = 0; iter < wwn_count; iter++) {
+ wwpn = be64_to_cpu(li_desc->pname_list[iter]);
+ ret = fpin_chk_wwn_setpath_marginal(host_num, vecs, wwpn);
+ if (ret < 0)
+ condlog(2, "failed to set the path marginal associated
with wwpn: 0x%" PRIx64 "\n", wwpn);
+
+ count++;
+ }
+ return count;
+}
+
+/*
+ * This function process the ELS frame received from HBA driver,
+ * and sets the path associated with the port wwn to marginal
+ * and also set the port state to marginal.
+ */
+static int
+fpin_process_els_frame(uint16_t host_num, char *fc_payload, struct vectors
*vecs)
+{
+
+ int count = -1;
+ struct fc_els_fpin *fpin = (struct fc_els_fpin *)fc_payload;
+ struct fc_tlv_desc *tlv;
+
+ tlv = (struct fc_tlv_desc *)&fpin->fpin_desc[0];
+
+ /*
+ * Parse the els frame and set the affected paths and port
+ * state to marginal
+ */
+ count = fpin_parse_li_els_setpath_marginal(host_num, tlv, vecs);
+ if (count <= 0)
+ condlog(4, "Could not find any WWNs, ret = %d\n",
+ count);
+ return count;
+}
+
+/*
+ * This function process the FPIN ELS frame received from HBA driver,
+ * and push the frame to appropriate frame list. Currently we have only FPIN
+ * LI frame list.
+ */
+static int
+fpin_handle_els_frame(struct fc_nl_event *fc_event)
+{
+ int ret = -1;
+ uint32_t els_cmd;
+ struct fc_els_fpin *fpin = (struct fc_els_fpin *)&fc_event->event_data;
+ struct fc_tlv_desc *tlv;
+ uint32_t dtag;
+
+ els_cmd = (uint32_t)fc_event->event_data;
+ tlv = (struct fc_tlv_desc *)&fpin->fpin_desc[0];
+ dtag = be32_to_cpu(tlv->desc_tag);
+ condlog(4, "Got CMD in add as 0x%x fpin_cmd 0x%x dtag 0x%x\n",
+ els_cmd, fpin->fpin_cmd, dtag);
+
+ if ((fc_event->event_code == FCH_EVT_LINK_FPIN) ||
+ (fc_event->event_code == FCH_EVT_LINKUP) ||
+ (fc_event->event_code == FCH_EVT_RSCN)) {
+
+ if (els_cmd == ELS_FPIN) {
+ /*
+ * Check the type of fpin by checking the tag info
+ * At present we are supporting only LI events
+ */
+ if (dtag == ELS_DTAG_LNK_INTEGRITY) {
+ /*Push the Payload to FPIN frame queue. */
+ ret = fpin_els_add_li_frame(fc_event);
+ if (ret != 0)
+ condlog(0, "Failed to process LI frame
with error %d\n",
+ ret);
+ } else {
+ condlog(4, "Unsupported FPIN received 0x%x\n",
dtag);
+ return ret;
+ }
+ } else {
+ /*Push the Payload to FPIN frame queue. */
+ ret = fpin_els_add_li_frame(fc_event);
+ if (ret != 0)
+ condlog(0, "Failed to process Linkup/RSCN event
with error %d evnt %d\n",
+ ret, fc_event->event_code);
+ }
+ } else
+ condlog(4, "Invalid command received: 0x%x\n", els_cmd);
+ return ret;
+}
+
+/*cleans the global marginal dev list*/
+void fpin_clean_marginal_dev_list(__attribute__((unused)) void *arg)
+{
+ struct marginal_dev_list *tmp_marg = NULL;
+
+ pthread_mutex_lock(&fpin_li_marginal_dev_mutex);
+ while (!list_empty(&fpin_li_marginal_dev_list_head)) {
+ tmp_marg = list_first_entry(&fpin_li_marginal_dev_list_head,
+ struct marginal_dev_list, node);
+ list_del(&tmp_marg->node);
+ free(tmp_marg);
+ }
+ pthread_mutex_unlock(&fpin_li_marginal_dev_mutex);
+}
+
+/* Cleans the global els marginal list */
+static void fpin_clean_els_marginal_list(void *arg)
+{
+ struct list_head *head = (struct list_head *)arg;
+ struct els_marginal_list *els_marg;
+
+ while (!list_empty(head)) {
+ els_marg = list_first_entry(head, struct els_marginal_list,
+ node);
+ list_del(&els_marg->node);
+ free(els_marg);
+ }
+}
+
+static void rcu_unregister(__attribute__((unused)) void *param)
+{
+ rcu_unregister_thread();
+}
+/*
+ * This is the FPIN ELS consumer thread. The thread sleeps on pthread cond
+ * variable unless notified by fpin_fabric_notification_receiver thread.
+ * This thread is only to process FPIN-LI ELS frames. A new thread and frame
+ * list will be added if any more ELS frames types are to be supported.
+ */
+void *fpin_els_li_consumer(void *data)
+{
+ struct list_head marginal_list_head;
+ int ret = 0;
+ uint16_t host_num;
+ struct els_marginal_list *els_marg;
+ uint32_t event_code;
+ struct vectors *vecs = (struct vectors *)data;
+
+ pthread_cleanup_push(rcu_unregister, NULL);
+ rcu_register_thread();
+ pthread_cleanup_push(fpin_clean_marginal_dev_list, NULL);
+ INIT_LIST_HEAD(&marginal_list_head);
+ pthread_cleanup_push(fpin_clean_els_marginal_list,
+ (void *)&marginal_list_head);
+ for ( ; ; ) {
+ pthread_mutex_lock(&fpin_li_mutex);
+ pthread_cleanup_push(cleanup_mutex, &fpin_li_mutex);
+ pthread_testcancel();
+ while (list_empty(&els_marginal_list_head))
+ pthread_cond_wait(&fpin_li_cond, &fpin_li_mutex);
+
+ if (!list_empty(&els_marginal_list_head)) {
+ condlog(4, "Invoke List splice tail\n");
+ list_splice_tail_init(&els_marginal_list_head,
&marginal_list_head);
+ }
+ pthread_cleanup_pop(1);
+
+ while (!list_empty(&marginal_list_head)) {
+ els_marg = list_first_entry(&marginal_list_head,
+ struct
els_marginal_list, node);
+ host_num = els_marg->host_num;
+ event_code = els_marg->event_code;
+ /* Now finally process FPIN LI ELS Frame */
+ condlog(4, "Got a new Payload buffer, processing it\n");
+ if ((event_code == FCH_EVT_LINKUP) || (event_code ==
FCH_EVT_RSCN))
+ fpin_unset_marginal_dev(host_num, vecs);
+ else {
+ ret = fpin_process_els_frame(host_num,
els_marg->payload, vecs);
+ if (ret <= 0)
+ condlog(0, "ELS frame processing failed
with ret %d\n", ret);
+ }
+ list_del(&els_marg->node);
+ free(els_marg);
+
+ }
+ }
+
+ pthread_cleanup_pop(1);
+ pthread_cleanup_pop(1);
+ pthread_cleanup_pop(1);
+ return NULL;
+}
+
+static void receiver_cleanup_list(__attribute__((unused)) void *arg)
+{
+ pthread_mutex_lock(&fpin_li_mutex);
+ fpin_clean_els_marginal_list(&els_marginal_list_head);
+ pthread_mutex_unlock(&fpin_li_mutex);
+}
+
+/*
+ * Listen for ELS frames from driver. on receiving the frame payload,
+ * push the payload to a list, and notify the fpin_els_li_consumer thread to
+ * process it. Once consumer thread is notified, return to listen for more ELS
+ * frames from driver.
+ */
+void *fpin_fabric_notification_receiver(__attribute__((unused))void *unused)
+{
+ int ret;
+ long fd;
+ uint32_t els_cmd;
+ struct fc_nl_event *fc_event = NULL;
+ struct sockaddr_nl fc_local;
+ unsigned char buf[DEF_RX_BUF_SIZE]
__attribute__((aligned(sizeof(uint64_t))));
+ size_t plen = 0;
+
+ pthread_cleanup_push(rcu_unregister, NULL);
+ rcu_register_thread();
+
+ pthread_cleanup_push(receiver_cleanup_list, NULL);
+ fd = socket(PF_NETLINK, SOCK_DGRAM, NETLINK_SCSITRANSPORT);
+ if (fd < 0) {
+ condlog(0, "fc socket error %ld", fd);
+ return NULL;
+ }
+
+ pthread_cleanup_push(close_fd, (void *)fd);
+ memset(&fc_local, 0, sizeof(fc_local));
+ fc_local.nl_family = AF_NETLINK;
+ fc_local.nl_groups = ~0;
+ fc_local.nl_pid = getpid();
+ ret = bind(fd, (struct sockaddr *)&fc_local, sizeof(fc_local));
+ if (ret == -1) {
+ condlog(0, "fc socket bind error %d\n", ret);
+ goto out;
+ }
+ for ( ; ; ) {
+ condlog(4, "Waiting for ELS...\n");
+ ret = read(fd, buf, DEF_RX_BUF_SIZE);
+ if (ret < 0) {
+ condlog(0, "failed to read the els frame (%d)", ret);
+ continue;
+ }
+ condlog(4, "Got a new request %d\n", ret);
+ if (!NLMSG_OK((struct nlmsghdr *)buf, (unsigned int)ret)) {
+ condlog(0, "bad els frame read (%d)", ret);
+ continue;
+ }
+ /* Push the frame to appropriate frame list */
+ plen = NLMSG_PAYLOAD((struct nlmsghdr *)buf, 0);
+ fc_event = (struct fc_nl_event *)NLMSG_DATA(buf);
+ if (plen < sizeof(*fc_event)) {
+ condlog(0, "too short (%d) to be an FC event", ret);
+ continue;
+ }
+ els_cmd = (uint32_t)fc_event->event_data;
+ condlog(4, "Got host no as %d, event 0x%x, len %d evntnum %d
evntcode %d\n",
+ fc_event->host_no, els_cmd,
fc_event->event_datalen,
+ fc_event->event_num, fc_event->event_code);
+ fpin_handle_els_frame(fc_event);
+ }
+out:
+ pthread_cleanup_pop(1);
+ pthread_cleanup_pop(1);
+ pthread_cleanup_pop(1);
+ return NULL;
+}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/multipath-tools-0.8.8+57+suse.dfb672fe/multipathd/main.c
new/multipath-tools-0.8.8+64+suse.f265f7e0/multipathd/main.c
--- old/multipath-tools-0.8.8+57+suse.dfb672fe/multipathd/main.c
2022-02-09 10:49:48.000000000 +0100
+++ new/multipath-tools-0.8.8+64+suse.f265f7e0/multipathd/main.c
2022-02-16 10:32:37.000000000 +0100
@@ -16,6 +16,7 @@
#include <linux/oom.h>
#include <libudev.h>
#include <urcu.h>
+#include "fpin.h"
#ifdef USE_SYSTEMD
#include <systemd/sd-daemon.h>
#endif
@@ -132,9 +133,11 @@
pid_t daemon_pid;
static pthread_mutex_t config_lock = PTHREAD_MUTEX_INITIALIZER;
static pthread_cond_t config_cond;
-static pthread_t check_thr, uevent_thr, uxlsnr_thr, uevq_thr, dmevent_thr;
+static pthread_t check_thr, uevent_thr, uxlsnr_thr, uevq_thr, dmevent_thr,
+ fpin_thr, fpin_consumer_thr;
static bool check_thr_started, uevent_thr_started, uxlsnr_thr_started,
- uevq_thr_started, dmevent_thr_started;
+ uevq_thr_started, dmevent_thr_started, fpin_thr_started,
+ fpin_consumer_thr_started;
static int pid_fd = -1;
static inline enum daemon_status get_running_state(void)
@@ -2847,6 +2850,7 @@
reconfigure (struct vectors * vecs)
{
struct config * old, *conf;
+ int old_marginal_pathgroups;
conf = load_config(DEFAULT_CONFIGFILE);
if (!conf)
@@ -2876,10 +2880,20 @@
uxsock_timeout = conf->uxsock_timeout;
old = rcu_dereference(multipath_conf);
+ old_marginal_pathgroups = old->marginal_pathgroups;
+ if ((old_marginal_pathgroups == MARGINAL_PATHGROUP_FPIN) !=
+ (conf->marginal_pathgroups == MARGINAL_PATHGROUP_FPIN)) {
+ condlog(1, "multipathd must be restarted to turn %s fpin
marginal paths",
+ (old_marginal_pathgroups == MARGINAL_PATHGROUP_FPIN)?
+ "off" : "on");
+ conf->marginal_pathgroups = old_marginal_pathgroups;
+ }
conf->sequence_nr = old->sequence_nr + 1;
rcu_assign_pointer(multipath_conf, conf);
call_rcu(&old->rcu, rcu_free_config);
-
+#ifdef FPIN_EVENT_HANDLER
+ fpin_clean_marginal_dev_list(NULL);
+#endif
configure(vecs);
@@ -3098,6 +3112,11 @@
pthread_cancel(uevq_thr);
if (dmevent_thr_started)
pthread_cancel(dmevent_thr);
+ if (fpin_thr_started)
+ pthread_cancel(fpin_thr);
+ if (fpin_consumer_thr_started)
+ pthread_cancel(fpin_consumer_thr);
+
if (check_thr_started)
pthread_join(check_thr, NULL);
@@ -3109,6 +3128,11 @@
pthread_join(uevq_thr, NULL);
if (dmevent_thr_started)
pthread_join(dmevent_thr, NULL);
+ if (fpin_thr_started)
+ pthread_join(fpin_thr, NULL);
+ if (fpin_consumer_thr_started)
+ pthread_join(fpin_consumer_thr, NULL);
+
/*
* As all threads are joined now, and we're in DAEMON_SHUTDOWN
@@ -3202,6 +3226,7 @@
char *envp;
enum daemon_status state;
int exit_code = 1;
+ int fpin_marginal_paths = 0;
init_unwinder();
mlockall(MCL_CURRENT | MCL_FUTURE);
@@ -3280,7 +3305,10 @@
setscheduler();
set_oom_adj();
-
+#ifdef FPIN_EVENT_HANDLER
+ if (conf->marginal_pathgroups == MARGINAL_PATHGROUP_FPIN)
+ fpin_marginal_paths = 1;
+#endif
/*
* Startup done, invalidate configuration
*/
@@ -3348,6 +3376,22 @@
goto failed;
} else
uevq_thr_started = true;
+
+ if (fpin_marginal_paths) {
+ if ((rc = pthread_create(&fpin_thr, &misc_attr,
+ fpin_fabric_notification_receiver, NULL))) {
+ condlog(0, "failed to create the fpin receiver thread:
%d", rc);
+ goto failed;
+ } else
+ fpin_thr_started = true;
+
+ if ((rc = pthread_create(&fpin_consumer_thr,
+ &misc_attr, fpin_els_li_consumer, vecs))) {
+ condlog(0, "failed to create the fpin consumer thread
thread: %d", rc);
+ goto failed;
+ } else
+ fpin_consumer_thr_started = true;
+ }
pthread_attr_destroy(&misc_attr);
while (1) {
++++++ multipath-tools.obsinfo ++++++
--- /var/tmp/diff_new_pack.ATKA3H/_old 2022-02-17 00:31:05.133427561 +0100
+++ /var/tmp/diff_new_pack.ATKA3H/_new 2022-02-17 00:31:05.133427561 +0100
@@ -1,5 +1,5 @@
name: multipath-tools
-version: 0.8.8+57+suse.dfb672fe
-mtime: 1644400188
-commit: dfb672fefaf30f5eb30ef5bfb717d5eefc425e9e
+version: 0.8.8+64+suse.f265f7e0
+mtime: 1645003957
+commit: f265f7e0e522325065d521b770663730dec5995d
