Blame SOURCES/0197-RHBZ-1394059-max-sectors-kb.patch

f20720
---
f20720
 libmultipath/config.c      |    3 +
f20720
 libmultipath/config.h      |    3 +
f20720
 libmultipath/configure.c   |    1 
f20720
 libmultipath/defaults.h    |    1 
f20720
 libmultipath/devmapper.c   |    4 +-
f20720
 libmultipath/dict.c        |   87 +++++++++++++++++++++++++++++++++++++++++++++
f20720
 libmultipath/discovery.c   |   60 +++++++++++++++++++++++++++++++
f20720
 libmultipath/discovery.h   |    1 
f20720
 libmultipath/propsel.c     |   25 ++++++++++++
f20720
 libmultipath/propsel.h     |    1 
f20720
 libmultipath/structs.h     |    7 +++
f20720
 multipath/multipath.conf.5 |    8 ++++
f20720
 12 files changed, 200 insertions(+), 1 deletion(-)
f20720
f20720
Index: multipath-tools-130222/libmultipath/config.c
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/config.c
f20720
+++ multipath-tools-130222/libmultipath/config.c
f20720
@@ -344,6 +344,7 @@ merge_hwe (struct hwentry * dst, struct
f20720
 	merge_num(delay_watch_checks);
f20720
 	merge_num(delay_wait_checks);
f20720
 	merge_num(skip_kpartx);
f20720
+	merge_num(max_sectors_kb);
f20720
 
f20720
 	/*
f20720
 	 * Make sure features is consistent with
f20720
@@ -405,6 +406,7 @@ overwrite_hwe (struct hwentry * dst, str
f20720
 	overwrite_num(delay_watch_checks);
f20720
 	overwrite_num(delay_wait_checks);
f20720
 	overwrite_num(skip_kpartx);
f20720
+	overwrite_num(max_sectors_kb);
f20720
 
f20720
 	/*
f20720
 	 * Make sure features is consistent with
f20720
@@ -682,6 +684,7 @@ load_config (char * file, struct udev *u
f20720
 	conf->skip_kpartx = DEFAULT_SKIP_KPARTX;
f20720
 	conf->remove_retries = 0;
f20720
 	conf->disable_changed_wwids = 0;
f20720
+	conf->max_sectors_kb = DEFAULT_MAX_SECTORS_KB;
f20720
 
f20720
 	/*
f20720
 	 * preload default hwtable
f20720
Index: multipath-tools-130222/libmultipath/config.h
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/config.h
f20720
+++ multipath-tools-130222/libmultipath/config.h
f20720
@@ -65,6 +65,7 @@ struct hwentry {
f20720
 	int delay_watch_checks;
f20720
 	int delay_wait_checks;
f20720
 	int skip_kpartx;
f20720
+	int max_sectors_kb;
f20720
 	char * bl_product;
f20720
 };
f20720
 
f20720
@@ -92,6 +93,7 @@ struct mpentry {
f20720
 	int delay_watch_checks;
f20720
 	int delay_wait_checks;
f20720
 	int skip_kpartx;
f20720
+	int max_sectors_kb;
f20720
 	uid_t uid;
f20720
 	gid_t gid;
f20720
 	mode_t mode;
f20720
@@ -148,6 +150,7 @@ struct config {
f20720
 	int skip_kpartx;
f20720
 	int remove_retries;
f20720
 	int disable_changed_wwids;
f20720
+	int max_sectors_kb;
f20720
 	unsigned int version[3];
f20720
 
f20720
 	char * dev;
f20720
Index: multipath-tools-130222/libmultipath/configure.c
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/configure.c
f20720
+++ multipath-tools-130222/libmultipath/configure.c
f20720
@@ -295,6 +295,7 @@ setup_map (struct multipath * mpp, char
f20720
 	select_delay_watch_checks(mpp);
f20720
 	select_delay_wait_checks(mpp);
f20720
 	select_skip_kpartx(mpp);
f20720
+	select_max_sectors_kb(mpp);
f20720
 
f20720
 	sysfs_set_scsi_tmo(mpp);
f20720
 	/*
f20720
Index: multipath-tools-130222/libmultipath/defaults.h
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/defaults.h
f20720
+++ multipath-tools-130222/libmultipath/defaults.h
f20720
@@ -25,6 +25,7 @@
f20720
 #define DEFAULT_RETRIGGER_TRIES 3
f20720
 #define DEFAULT_UEV_WAIT_TIMEOUT 30
f20720
 #define DEFAULT_SKIP_KPARTX SKIP_KPARTX_OFF
f20720
+#define DEFAULT_MAX_SECTORS_KB	MAX_SECTORS_KB_UNDEF
f20720
 
f20720
 #define DEFAULT_CHECKINT	5
f20720
 #define MAX_CHECKINT(a)		(a << 2)
f20720
Index: multipath-tools-130222/libmultipath/dict.c
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/dict.c
f20720
+++ multipath-tools-130222/libmultipath/dict.c
f20720
@@ -976,6 +976,22 @@ def_disable_changed_wwids_handler(vector
f20720
 	return 0;
f20720
 }
f20720
 
f20720
+static int
f20720
+def_max_sectors_kb_handler(vector strvec)
f20720
+{
f20720
+	char * buff;
f20720
+
f20720
+	buff = set_value(strvec);
f20720
+	if (!buff)
f20720
+		return 1;
f20720
+
f20720
+	if ((conf->max_sectors_kb = atoi(buff)) < MAX_SECTORS_KB_MIN)
f20720
+		conf->max_sectors_kb = MAX_SECTORS_KB_UNDEF;
f20720
+
f20720
+	FREE(buff);
f20720
+	return 0;
f20720
+}
f20720
+
f20720
 /*
f20720
  * blacklist block handlers
f20720
  */
f20720
@@ -1765,6 +1781,26 @@ hw_delay_wait_checks_handler(vector strv
f20720
 	return 0;
f20720
 }
f20720
 
f20720
+static int
f20720
+hw_max_sectors_kb_handler(vector strvec)
f20720
+{
f20720
+	struct hwentry *hwe = VECTOR_LAST_SLOT(conf->hwtable);
f20720
+	char * buff;
f20720
+
f20720
+	if (!hwe)
f20720
+		return 1;
f20720
+
f20720
+	buff = set_value(strvec);
f20720
+	if (!buff)
f20720
+		return 1;
f20720
+
f20720
+	if ((hwe->max_sectors_kb = atoi(buff)) < MAX_SECTORS_KB_MIN)
f20720
+		hwe->max_sectors_kb = MAX_SECTORS_KB_UNDEF;
f20720
+
f20720
+	FREE(buff);
f20720
+	return 0;
f20720
+}
f20720
+
f20720
 /*
f20720
  * multipaths block handlers
f20720
  */
f20720
@@ -2316,6 +2352,26 @@ mp_delay_wait_checks_handler(vector strv
f20720
 	return 0;
f20720
 }
f20720
 
f20720
+static int
f20720
+mp_max_sectors_kb_handler(vector strvec)
f20720
+{
f20720
+	struct mpentry *mpe = VECTOR_LAST_SLOT(conf->mptable);
f20720
+	char * buff;
f20720
+
f20720
+	if (!mpe)
f20720
+		return 1;
f20720
+
f20720
+	buff = set_value(strvec);
f20720
+	if (!buff)
f20720
+		return 1;
f20720
+
f20720
+	if ((mpe->max_sectors_kb = atoi(buff)) < MAX_SECTORS_KB_MIN)
f20720
+		mpe->max_sectors_kb = MAX_SECTORS_KB_UNDEF;
f20720
+
f20720
+	FREE(buff);
f20720
+	return 0;
f20720
+}
f20720
+
f20720
 /*
f20720
  * config file keywords printing
f20720
  */
f20720
@@ -2615,6 +2671,16 @@ snprint_mp_delay_wait_checks(char * buff
f20720
 }
f20720
 
f20720
 static int
f20720
+snprint_mp_max_sectors_kb(char * buff, int len, void * data)
f20720
+{
f20720
+	struct mpentry * mpe = (struct mpentry *)data;
f20720
+
f20720
+	if (mpe->max_sectors_kb == MAX_SECTORS_KB_UNDEF)
f20720
+		return 0;
f20720
+	return snprintf(buff, len, "%d", mpe->max_sectors_kb);
f20720
+}
f20720
+
f20720
+static int
f20720
 snprint_hw_fast_io_fail(char * buff, int len, void * data)
f20720
 {
f20720
 	struct hwentry * hwe = (struct hwentry *)data;
f20720
@@ -2993,6 +3059,16 @@ snprint_detect_prio(char * buff, int len
f20720
 }
f20720
 
f20720
 static int
f20720
+snprint_hw_max_sectors_kb(char * buff, int len, void * data)
f20720
+{
f20720
+	struct hwentry * hwe = (struct hwentry *)data;
f20720
+
f20720
+	if (hwe->max_sectors_kb == MAX_SECTORS_KB_UNDEF)
f20720
+		return 0;
f20720
+	return snprintf(buff, len, "%d", hwe->max_sectors_kb);
f20720
+}
f20720
+
f20720
+static int
f20720
 snprint_def_polling_interval (char * buff, int len, void * data)
f20720
 {
f20720
 	return snprintf(buff, len, "%i", conf->checkint);
f20720
@@ -3461,6 +3537,14 @@ snprint_def_disable_changed_wwids(char *
f20720
 }
f20720
 
f20720
 static int
f20720
+snprint_def_max_sectors_kb(char * buff, int len, void * data)
f20720
+{
f20720
+	if (conf->max_sectors_kb == MAX_SECTORS_KB_UNDEF)
f20720
+		return 0;
f20720
+	return snprintf(buff, len, "%d", conf->max_sectors_kb);
f20720
+}
f20720
+
f20720
+static int
f20720
 snprint_ble_simple (char * buff, int len, void * data)
f20720
 {
f20720
 	struct blentry * ble = (struct blentry *)data;
f20720
@@ -3541,6 +3625,7 @@ init_keywords(void)
f20720
 	install_keyword("new_bindings_in_boot", &def_new_bindings_in_boot_handler, &snprint_def_new_bindings_in_boot);
f20720
 	install_keyword("remove_retries", &def_remove_retries_handler, &snprint_def_remove_retries);
f20720
 	install_keyword("disable_changed_wwids", &def_disable_changed_wwids_handler, &snprint_def_disable_changed_wwids);
f20720
+	install_keyword("max_sectors_kb", &def_max_sectors_kb_handler, &snprint_def_max_sectors_kb);
f20720
 	__deprecated install_keyword("default_selector", &def_selector_handler, NULL);
f20720
 	__deprecated install_keyword("default_path_grouping_policy", &def_pgpolicy_handler, NULL);
f20720
 	__deprecated install_keyword("default_uid_attribute", &def_uid_attribute_handler, NULL);
f20720
@@ -3609,6 +3694,7 @@ init_keywords(void)
f20720
 	install_keyword("delay_watch_checks", &hw_delay_watch_checks_handler, &snprint_hw_delay_watch_checks);
f20720
 	install_keyword("delay_wait_checks", &hw_delay_wait_checks_handler, &snprint_hw_delay_wait_checks);
f20720
 	install_keyword("skip_kpartx", &hw_skip_kpartx_handler, &snprint_hw_skip_kpartx);
f20720
+	install_keyword("max_sectors_kb", &hw_max_sectors_kb_handler, &snprint_hw_max_sectors_kb);
f20720
 	install_sublevel_end();
f20720
 
f20720
 	install_keyword_root("multipaths", &multipaths_handler);
f20720
@@ -3637,5 +3723,6 @@ init_keywords(void)
f20720
 	install_keyword("delay_watch_checks", &mp_delay_watch_checks_handler, &snprint_mp_delay_watch_checks);
f20720
 	install_keyword("delay_wait_checks", &mp_delay_wait_checks_handler, &snprint_mp_delay_wait_checks);
f20720
 	install_keyword("skip_kpartx", &mp_skip_kpartx_handler, &snprint_mp_skip_kpartx);
f20720
+	install_keyword("max_sectors_kb", &mp_max_sectors_kb_handler, &snprint_mp_max_sectors_kb);
f20720
 	install_sublevel_end();
f20720
 }
f20720
Index: multipath-tools-130222/libmultipath/discovery.c
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/discovery.c
f20720
+++ multipath-tools-130222/libmultipath/discovery.c
f20720
@@ -12,6 +12,7 @@
f20720
 #include <errno.h>
f20720
 #include <libgen.h>
f20720
 #include <libudev.h>
f20720
+#include <libdevmapper.h>
f20720
 
f20720
 #include "checkers.h"
f20720
 #include "vector.h"
f20720
@@ -27,6 +28,7 @@
f20720
 #include "discovery.h"
f20720
 #include "prio.h"
f20720
 #include "defaults.h"
f20720
+#include "devmapper.h"
f20720
 
f20720
 int
f20720
 store_pathinfo (vector pathvec, vector hwtable, struct udev_device *udevice,
f20720
@@ -166,6 +168,64 @@ declare_sysfs_get_str(rev);
f20720
 declare_sysfs_get_str(dev);
f20720
 
f20720
 int
f20720
+sysfs_set_max_sectors_kb(struct multipath *mpp, int is_reload)
f20720
+{
f20720
+	struct pathgroup * pgp;
f20720
+	struct path *pp;
f20720
+	char buff[11];
f20720
+	struct udev_device *udevice = NULL;
f20720
+	int i, j, len, ret;
f20720
+	int max_sectors_kb;
f20720
+
f20720
+	if (mpp->max_sectors_kb == MAX_SECTORS_KB_UNDEF)
f20720
+		return 0;
f20720
+	max_sectors_kb = mpp->max_sectors_kb;
f20720
+	if (is_reload) {
f20720
+		if (!mpp->dmi && dm_get_info(mpp->alias, &mpp->dmi) != 0) {
f20720
+			condlog(0, "failed to get dm info on %s to set max_sectors_kb", mpp->alias);
f20720
+			return 1;
f20720
+		}
f20720
+		udevice = udev_device_new_from_devnum(conf->udev, 'b',
f20720
+						      makedev(mpp->dmi->major,
f20720
+							      mpp->dmi->minor));
f20720
+		if (!udevice) {
f20720
+			condlog(0, "failed to get udev device to set max_sectors_kb for %s", mpp->alias);
f20720
+			return 1;
f20720
+		}
f20720
+		if (sysfs_attr_get_value(udevice, "queue/max_sectors_kb",
f20720
+					 buff, sizeof(buff)) <= 0) {
f20720
+			condlog(0, "failed to get current max_sectors_kb from %s", mpp->alias);
f20720
+			goto fail_reload;
f20720
+		}
f20720
+		if (sscanf(buff, "%u\n", &max_sectors_kb) != 1) {
f20720
+			condlog(0, "can't parse current max_sectors_kb from %s",
f20720
+				mpp->alias);
f20720
+			goto fail_reload;
f20720
+		}
f20720
+		udev_device_unref(udevice);
f20720
+	}
f20720
+	snprintf(buff, 11, "%d", max_sectors_kb);
f20720
+	len = strlen(buff);
f20720
+
f20720
+	vector_foreach_slot (mpp->pg, pgp, i) {
f20720
+		vector_foreach_slot (pgp->paths, pp, j) {
f20720
+			ret = sysfs_attr_set_value(pp->udev,
f20720
+						   "queue/max_sectors_kb",
f20720
+					  	   buff, len);
f20720
+			if (ret < 0) {
f20720
+				condlog(0, "failed setting max_sectors_kb on %s : %s", pp->dev, strerror(-ret));
f20720
+				return 1;
f20720
+			}
f20720
+		}
f20720
+	}
f20720
+	return 0;
f20720
+
f20720
+fail_reload:
f20720
+	udev_device_unref(udevice);
f20720
+	return 1;
f20720
+}
f20720
+
f20720
+int
f20720
 sysfs_get_timeout(struct path *pp, unsigned int *timeout)
f20720
 {
f20720
 	const char *attr = NULL;
f20720
Index: multipath-tools-130222/libmultipath/discovery.h
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/discovery.h
f20720
+++ multipath-tools-130222/libmultipath/discovery.h
f20720
@@ -41,6 +41,7 @@ int store_pathinfo (vector pathvec, vect
f20720
 		    struct udev_device *udevice, int flag,
f20720
 		    struct path **pp_ptr);
f20720
 int sysfs_set_scsi_tmo (struct multipath *mpp);
f20720
+int sysfs_set_max_sectors_kb(struct multipath *mpp, int is_reload);
f20720
 int sysfs_get_timeout(struct path *pp, unsigned int *timeout);
f20720
 int sysfs_get_host_pci_name(struct path *pp, char *pci_name);
f20720
 int sysfs_get_iscsi_ip_address(struct path *pp, char *ip_address);
f20720
Index: multipath-tools-130222/libmultipath/propsel.c
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/propsel.c
f20720
+++ multipath-tools-130222/libmultipath/propsel.c
f20720
@@ -880,3 +880,28 @@ select_skip_kpartx (struct multipath * m
f20720
 	condlog(3, "skip_kpartx = DISABLED (internal default)");
f20720
 	return 0;
f20720
 }
f20720
+
f20720
+extern int
f20720
+select_max_sectors_kb (struct multipath * mp)
f20720
+{
f20720
+	if (mp->mpe && mp->mpe->max_sectors_kb != MAX_SECTORS_KB_UNDEF) {
f20720
+		mp->max_sectors_kb = mp->mpe->max_sectors_kb;
f20720
+		condlog(3, "max_sectors_kb = %i (multipath setting)",
f20720
+				mp->max_sectors_kb);
f20720
+		return 0;
f20720
+	}
f20720
+	if (mp->hwe && mp->hwe->max_sectors_kb != MAX_SECTORS_KB_UNDEF) {
f20720
+		mp->max_sectors_kb = mp->hwe->max_sectors_kb;
f20720
+		condlog(3, "max_sectors_kb = %i (controler setting)",
f20720
+				mp->max_sectors_kb);
f20720
+		return 0;
f20720
+	}
f20720
+	if (conf->max_sectors_kb != MAX_SECTORS_KB_UNDEF) {
f20720
+		mp->max_sectors_kb = conf->max_sectors_kb;
f20720
+		condlog(3, "max_sectors_kb = %i (config file default)",
f20720
+				mp->max_sectors_kb);
f20720
+		return 0;
f20720
+	}
f20720
+	mp->max_sectors_kb = MAX_SECTORS_KB_UNDEF;
f20720
+	return 0;
f20720
+}
f20720
Index: multipath-tools-130222/libmultipath/propsel.h
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/propsel.h
f20720
+++ multipath-tools-130222/libmultipath/propsel.h
f20720
@@ -24,3 +24,4 @@ int select_deferred_remove(struct multip
f20720
 int select_delay_watch_checks (struct multipath * mp);
f20720
 int select_delay_wait_checks (struct multipath * mp);
f20720
 int select_skip_kpartx (struct multipath * mp);
f20720
+int select_max_sectors_kb (struct multipath * mp);
f20720
Index: multipath-tools-130222/libmultipath/structs.h
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/structs.h
f20720
+++ multipath-tools-130222/libmultipath/structs.h
f20720
@@ -128,6 +128,12 @@ enum skip_kpartx_states {
f20720
 	SKIP_KPARTX_ON,
f20720
 };
f20720
 
f20720
+
f20720
+enum max_sectors_kb_states {
f20720
+	MAX_SECTORS_KB_UNDEF = 0,
f20720
+	MAX_SECTORS_KB_MIN = 4,  /* can't be smaller than page size */
f20720
+};
f20720
+
f20720
 enum scsi_protocol {
f20720
 	SCSI_PROTOCOL_FCP = 0,	/* Fibre Channel */
f20720
 	SCSI_PROTOCOL_SPI = 1,	/* parallel SCSI */
f20720
@@ -245,6 +251,7 @@ struct multipath {
f20720
 	int delay_wait_checks;
f20720
 	int force_udev_reload;
f20720
 	int skip_kpartx;
f20720
+	int max_sectors_kb;
f20720
 	unsigned int dev_loss;
f20720
 	uid_t uid;
f20720
 	gid_t gid;
f20720
Index: multipath-tools-130222/multipath/multipath.conf.5
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/multipath/multipath.conf.5
f20720
+++ multipath-tools-130222/multipath/multipath.conf.5
526b4e
@@ -567,6 +567,10 @@ disable access to the path until the wwi
f20720
 This sets how may times multipath will retry removing a device that is in-use.
f20720
 Between each attempt, multipath will sleep 1 second. The default is
f20720
 .I 0
f20720
+.TP
f20720
+.B max_sectors_kb
f20720
+Sets the max_sectors_kb device parameter on all path devices and the multipath
f20720
+device to the specified value. Default is device dependent.
f20720
 .
f20720
 .SH "blacklist section"
f20720
 The
526b4e
@@ -678,6 +682,8 @@ section:
f20720
 .B delay_wait_checks
f20720
 .TP
f20720
 .B skip_kpartx
f20720
+.TP
f20720
+.B max_sectors_kb
f20720
 .RE
f20720
 .PD
f20720
 .LP
526b4e
@@ -778,6 +784,8 @@ section:
f20720
 .B delay_wait_checks
f20720
 .TP
f20720
 .B skip_kpartx
f20720
+.TP
f20720
+.B max_sectors_kb
f20720
 .RE
f20720
 .PD
f20720
 .LP
f20720
Index: multipath-tools-130222/libmultipath/devmapper.c
f20720
===================================================================
f20720
--- multipath-tools-130222.orig/libmultipath/devmapper.c
f20720
+++ multipath-tools-130222/libmultipath/devmapper.c
f20720
@@ -21,7 +21,7 @@
f20720
 #include "devmapper.h"
f20720
 #include "config.h"
f20720
 #include "sysfs.h"
f20720
-
f20720
+#include "discovery.h"
f20720
 #include "log_pthread.h"
f20720
 #include <sys/types.h>
f20720
 #include <time.h>
f20720
@@ -330,6 +330,7 @@ extern int
f20720
 dm_addmap_create (struct multipath *mpp, char * params) {
f20720
 	int ro;
f20720
 
f20720
+	sysfs_set_max_sectors_kb(mpp, 0);
f20720
 	for (ro = 0; ro <= 1; ro++) {
f20720
 		int err;
f20720
 
f20720
@@ -356,6 +357,7 @@ dm_addmap_create (struct multipath *mpp,
f20720
 
f20720
 extern int
f20720
 dm_addmap_reload (struct multipath *mpp, char *params) {
f20720
+	sysfs_set_max_sectors_kb(mpp, 1);
f20720
 	if (dm_addmap(DM_DEVICE_RELOAD, TGT_MPATH, mpp, params, 0, ADDMAP_RW, SKIP_KPARTX_OFF))
f20720
 		return 1;
f20720
 	if (errno != EROFS)