All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] multipath-tools: delete obsolete information from multipath.conf.5
       [not found] <20260607103831.336833-1-xose.vazquez@gmail.com>
@ 2026-06-07 10:38 ` Xose Vazquez Perez
  2026-06-10 22:46   ` Benjamin Marzinski
  2026-06-07 10:38 ` [PATCH 2/3] multipath-tools: remove explicit width from .TP macros in multipath.conf.5 Xose Vazquez Perez
  2026-06-07 10:38 ` [PATCH 3/3] multipath-tools: remove explicit widths from macros in the remaining man pages Xose Vazquez Perez
  2 siblings, 1 reply; 6+ messages in thread
From: Xose Vazquez Perez @ 2026-06-07 10:38 UTC (permalink / raw)
  Cc: Xose Vazquez Perez, Martin Wilck, Benjamin Marzinski,
	Christophe Varoqui, DM_DEVEL-ML

repeat_count (rr_min_io, rr_min_io_rq, rr_weight) has been unsupported
since kernel 4.6 ( commits 90a4323ccfea and  21136f89d76d ).

And also clean up other deprecated entries.

Cc: Martin Wilck <mwilck@suse.com>
Cc: Benjamin Marzinski <bmarzins@redhat.com>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: DM_DEVEL-ML <dm-devel@lists.linux.dev>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 multipath/multipath.conf.5.in | 69 +++--------------------------------
 1 file changed, 6 insertions(+), 63 deletions(-)

diff --git a/multipath/multipath.conf.5.in b/multipath/multipath.conf.5.in
index 84cd1a0a..e3a9811c 100644
--- a/multipath/multipath.conf.5.in
+++ b/multipath/multipath.conf.5.in
@@ -6,7 +6,7 @@
 .\" Update the date below if you make any significant change.
 .\" ----------------------------------------------------------------------------
 .
-.TH MULTIPATH.CONF 5 2025-09-12 Linux
+.TH MULTIPATH.CONF 5 2026-05-27 Linux
 .
 .
 .\" ----------------------------------------------------------------------------
@@ -198,9 +198,8 @@ kernel multipath target:
 .TP 12
 .I "round-robin 0"
 Choose the path for the next bunch of I/O by looping through every path in the
-path group, sending \fBthe same number of I/O requests\fR to each path. Some
-aspects of behavior can be controlled with the attributes: \fIrr_min_io\fR,
-\fIrr_min_io_rq\fR and \fIrr_weight\fR.
+path group, sending \fBthe same number of I/O requests\fR to each path.
+
 .TP
 .I "queue-length 0"
 (Since 2.6.31 kernel) Choose the path for the next bunch of I/O based on \fBthe lowest
@@ -598,24 +597,12 @@ The default is: \fBmanual\fR
 .
 .TP
 .B rr_min_io
-Number of I/O requests to route to a path before switching to the next in the
-same path group. This is only for \fIBlock I/O\fR(BIO) based multipath and
-only apply to \fIround-robin\fR path_selector.
-.RS
-.TP
-The default is: \fB1000\fR
-.RE
+(Deprecated since kernel 4.6) This option is not supported anymore, and will be ignored.
 .
 .
 .TP
 .B rr_min_io_rq
-Number of I/O requests to route to a path before switching to the next in the
-same path group. This is only for \fIRequest\fR based multipath and
-only apply to \fIround-robin\fR path_selector.
-.RS
-.TP
-The default is: \fB1\fR
-.RE
+(Deprecated since kernel 4.6) This option is not supported anymore, and will be ignored.
 .
 .
 .TP
@@ -634,16 +621,7 @@ The default is: \fBmax\fR
 .
 .TP
 .B rr_weight
-If set to \fIpriorities\fR the multipath configurator will assign path weights
-as "path prio * rr_min_io". Possible values are
-.I priorities
-or
-.I uniform .
-Only apply to \fIround-robin\fR path_selector.
-.RS
-.TP
-The default is: \fBuniform\fR
-.RE
+(Deprecated since kernel 4.6) This option is not supported anymore, and will be ignored.
 .
 .
 .TP
@@ -821,28 +799,16 @@ The default is: \fB<unset>\fR
 .TP
 .B bindings_file
 (Deprecated) This option is not supported anymore, and will be ignored.
-.RS
-.TP
-The compiled-in value is: \fB@STATE_DIR@/bindings\fR
-.RE
 .
 .
 .TP
 .B wwids_file
 (Deprecated) This option is not supported anymore, and will be ignored.
-.RS
-.TP
-The compiled-in value is: \fB@STATE_DIR@/wwids\fR
-.RE
 .
 .
 .TP
 .B prkeys_file
 (Deprecated) This option is not supported anymore, and will be ignored.
-.RS
-.TP
-The compiled-in value is: \fB@STATE_DIR@/prkeys\fR
-.RE
 .
 .
 .TP
@@ -1008,10 +974,6 @@ The default is: \fB<unset>\fR
 .TP
 .B config_dir
 (Deprecated) This option is not supported anymore, and will be ignored.
-.RS
-.TP
-The compiled-in value is: \fB@CONFIGDIR@\fR
-.RE
 .
 .
 .TP
@@ -1324,7 +1286,6 @@ The default is: \fBno\fR
 .TP
 .B disable_changed_wwids
 (Deprecated) This option is not supported anymore, and will be ignored.
-.RE
 .
 .
 .TP
@@ -1580,14 +1541,8 @@ section:
 .TP
 .B failback
 .TP
-.B rr_weight
-.TP
 .B no_path_retry
 .TP
-.B rr_min_io
-.TP
-.B rr_min_io_rq
-.TP
 .B flush_on_last_del
 .TP
 .B features
@@ -1769,14 +1724,8 @@ section:
 .TP
 .B failback
 .TP
-.B rr_weight
-.TP
 .B no_path_retry
 .TP
-.B rr_min_io
-.TP
-.B rr_min_io_rq
-.TP
 .B fast_io_fail_tmo
 .TP
 .B dev_loss_tmo
@@ -1855,14 +1804,8 @@ the values are taken from the \fIdevices\fR or \fIdefaults\fR sections:
 .TP
 .B failback
 .TP
-.B rr_weight
-.TP
 .B no_path_retry
 .TP
-.B rr_min_io
-.TP
-.B rr_min_io_rq
-.TP
 .B flush_on_last_del
 .TP
 .B fast_io_fail_tmo
-- 
2.54.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/3] multipath-tools: remove explicit width from .TP macros in multipath.conf.5
       [not found] <20260607103831.336833-1-xose.vazquez@gmail.com>
  2026-06-07 10:38 ` [PATCH 1/3] multipath-tools: delete obsolete information from multipath.conf.5 Xose Vazquez Perez
@ 2026-06-07 10:38 ` Xose Vazquez Perez
  2026-06-10 23:41   ` Benjamin Marzinski
  2026-06-07 10:38 ` [PATCH 3/3] multipath-tools: remove explicit widths from macros in the remaining man pages Xose Vazquez Perez
  2 siblings, 1 reply; 6+ messages in thread
From: Xose Vazquez Perez @ 2026-06-07 10:38 UTC (permalink / raw)
  Cc: Xose Vazquez Perez, Martin Wilck, Benjamin Marzinski,
	Christophe Varoqui, DM_DEVEL-ML

This allows man/groff to calculate the width automatically, preventing
excessive whitespace and improving readability on various terminal sizes.

Cc: Martin Wilck <mwilck@suse.com>
Cc: Benjamin Marzinski <bmarzins@redhat.com>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: DM_DEVEL-ML <dm-devel@lists.linux.dev>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 multipath/multipath.conf.5.in | 56 +++++++++++++++++------------------
 1 file changed, 28 insertions(+), 28 deletions(-)

diff --git a/multipath/multipath.conf.5.in b/multipath/multipath.conf.5.in
index e3a9811c..ef08ed2b 100644
--- a/multipath/multipath.conf.5.in
+++ b/multipath/multipath.conf.5.in
@@ -103,7 +103,7 @@ matches, standard regular expression syntax using the special characters "^" and
 .LP
 .
 The following \fIsection\fP keywords are recognized:
-.TP 17
+.TP
 .B defaults
 This section defines default values for attributes which are used
 whenever no values are given in the appropriate device or multipath
@@ -142,7 +142,7 @@ device-specific settings for all devices.
 The \fIdefaults\fR section recognizes the following keywords:
 .
 .
-.TP 17
+.TP
 .B verbosity
 Default verbosity. Higher values increase the verbosity level. Valid
 levels are between 0 and 6.
@@ -195,7 +195,7 @@ The default is: \fBno\fR
 The default path selector algorithm to use; they are offered by the
 kernel multipath target:
 .RS
-.TP 12
+.TP
 .I "round-robin 0"
 Choose the path for the next bunch of I/O by looping through every path in the
 path group, sending \fBthe same number of I/O requests\fR to each path.
@@ -223,7 +223,7 @@ The default is: \fBservice-time 0\fR
 (Hardware-dependent) The default path grouping policy to apply to unspecified
 multipaths. Possible values are:
 .RS
-.TP 12
+.TP
 .I failover
 One path per priority group.
 .TP
@@ -333,7 +333,7 @@ of this path. Higher number have a higher priority.
 \fI"none"\fR is a valid value. Currently the following path priority routines
 are implemented:
 .RS
-.TP 12
+.TP
 .I const
 Return a constant priority of \fI1\fR.
 .TP
@@ -408,12 +408,12 @@ NetAPP E/EF Series, where it is \fBalua\fR). If \fBdetect_prio\fR is
 Arguments to pass to to the prio function. This only applies to certain
 prioritizers:
 .RS
-.TP 12
+.TP
 .I weighted
 Needs a value of the form
 \fI"<hbtl|devname|serial|wwn> <regex1> <prio1> <regex2> <prio2> ..."\fR
 .RS
-.TP 8
+.TP
 .I hbtl
 Regex can be of SCSI H:B:T:L format. For example: 1:0:.:. , *:0:0:.
 .TP
@@ -430,11 +430,11 @@ Regex can be of the form \fI"host_wwnn:host_wwpn:target_wwnn:target_wwpn"\fR
 these values can be looked up through sysfs or by running \fImultipathd show paths format
 "%N:%R:%n:%r"\fR. For example: 0x200100e08ba0aea0:0x210100e08ba0aea0:.*:.* , .*:.*:iqn.2009-10.com.redhat.msp.lab.ask-06:.*
 .RE
-.TP 12
+.TP
 .I path_latency
 Needs a value of the form "io_num=\fI<20>\fR base_num=\fI<10>\fR"
 .RS
-.TP 8
+.TP
 .I io_num
 The number of read IOs sent to the current path continuously, used to calculate the average path latency.
 Valid Values: Integer, [20, 200].
@@ -445,7 +445,7 @@ Double-precision floating-point, [1.1, 10]. And Max average latency value is 100
 For example: If base_num=10, the paths will be grouped in priority groups with path latency <=1us, (1us, 10us],
 (10us, 100us], (100us, 1ms], (1ms, 10ms], (10ms, 100ms], (100ms, 1s], (1s, 10s], (10s, 100s], >100s.
 .RE
-.TP 12
+.TP
 .I alua
 If \fIexclusive_pref_bit\fR is set, paths with the \fIpreferred path\fR bit
 set will always be in their own path group.
@@ -456,17 +456,17 @@ set will always be in their own path group.
 .TP
 .I datacore
 .RS
-.TP 8
+.TP
 .I preferredsds
 (Mandatory) The preferred "SDS name".
 .TP
 .I timeout
 (Optional) The timeout for the INQUIRY, in ms.
 .RE
-.TP 12
+.TP
 .I iet
 .RS
-.TP 8
+.TP
 .I preferredip=...
 (Mandatory) Th preferred IP address, in dotted decimal notation, for iSCSI targets.
 .RE
@@ -481,7 +481,7 @@ Specify any device-mapper features to be used. Syntax is \fInum list\fR
 where \fInum\fR is the number, between 0 and 8, of features in \fIlist\fR.
 Possible values for the feature list are:
 .RS
-.TP 12
+.TP
 .I queue_if_no_path
 (Deprecated, superseded by \fIno_path_retry\fR) Queue I/O if no path is active.
 Identical to the \fIno_path_retry\fR with \fIqueue\fR value. If both this
@@ -520,7 +520,7 @@ to respond. The asynchronous checkers (\fItur\fR and \fIdirectio\fR) will not
 pause multipathd. Instead, multipathd will check for a response once per
 second, until \fIchecker_timeout\fR seconds have elapsed. Possible values are:
 .RS
-.TP 12
+.TP
 .I readsector0
 (Deprecated) Read the first sector of the device. This checker is being
 deprecated, please use \fItur\fR or \fIdirectio\fR instead.
@@ -574,7 +574,7 @@ Tell multipathd how to manage path group failback.
 To select \fIimmediate\fR or a \fIvalue\fR, it's mandatory that the device
 has support for a working prioritizer.
 .RS
-.TP 12
+.TP
 .I immediate
 Immediately failback to the highest priority pathgroup that contains
 active paths.
@@ -628,7 +628,7 @@ The default is: \fBmax\fR
 .B no_path_retry
 Specify what to do when all paths are down. Possible values are:
 .RS
-.TP 12
+.TP
 .I value > 0
 Number of retries until disable I/O queueing.
 .TP
@@ -1154,7 +1154,7 @@ blocked from further processing by higher layers - such as LVM - if and only
 if it\'s considered a valid multipath device path), and b) when multipathd
 detects a new device. The following values are possible:
 .RS
-.TP 10
+.TP
 .I strict
 Both multipath and multipathd treat only such devices as multipath devices
 which have been part of a multipath map previously, and which are therefore
@@ -1421,7 +1421,7 @@ by multipath-tools.
 .
 The following keywords are recognized in both sections. The defaults are empty
 unless explicitly stated.
-.TP 17
+.TP
 .B devnode
 Regular expression matching the device nodes to be excluded/included.
 .RS
@@ -1511,7 +1511,7 @@ from later entries take precedence.
 .
 .
 The \fImultipath\fR subsection recognizes the following attributes:
-.TP 17
+.TP
 .B wwid
 (Mandatory) World Wide Identifier. Detected multipath maps are matched against this attribute.
 Note that, unlike the \fIwwid\fR attribute in the \fIblacklist\fR section,
@@ -1530,7 +1530,7 @@ section:
 .sp 1
 .PD .1v
 .RS
-.TP 18
+.TP
 .B path_grouping_policy
 .TP
 .B path_selector
@@ -1587,7 +1587,7 @@ section:
 .SH "devices section"
 .\" ----------------------------------------------------------------------------
 .
-.TP 4
+.TP
 .B Important:
 The built-in hardware device table of
 .I multipath-tools
@@ -1623,7 +1623,7 @@ for all devices in the system.
 .LP
 .
 The \fIdevice\fR subsection recognizes the following attributes:
-.TP 17
+.TP
 .B vendor
 (Mandatory) Regular expression to match the vendor name.
 .RS
@@ -1666,7 +1666,7 @@ and \fImultipathd show paths format\fR commands. Currently only the
 The hardware handler to use for this device type.
 The following hardware handler are implemented:
 .RS
-.TP 12
+.TP
 .I 1 emc
 (Hardware-dependent)
 Hardware handler for DGC class arrays as CLARiiON CX/AX and EMC VNX families
@@ -1707,7 +1707,7 @@ section:
 .sp 1
 .PD .1v
 .RS
-.TP 18
+.TP
 .B path_grouping_policy
 .TP
 .B uid_attribute
@@ -1785,7 +1785,7 @@ the values are taken from the \fIdevices\fR or \fIdefaults\fR sections:
 .sp 1
 .PD .1v
 .RS
-.TP 18
+.TP
 .B path_grouping_policy
 .TP
 .B uid_attribute
@@ -1899,7 +1899,7 @@ which paths belong to the same device. Each path presenting the same
 WWID is assumed to point to the same device.
 .LP
 The WWID is generated by four methods (in the order of preference):
-.TP 17
+.TP
 .B uid_attrs
 The WWID is derived from udev attributes by matching the device node name; cf
 \fIuid_attrs\fR above.
@@ -1939,7 +1939,7 @@ pathgroups will not be used until all other pathgroups have been tried. At the
 time when the path would normally be reinstated, it will be returned to its
 normal pathgroup. The logic of determining \(dqshaky\(dq condition, as well as
 the logic when to reinstate, differs between the three methods.
-.TP 8
+.TP
 .B \(dqdelay_checks\(dq failure tracking
 (Deprecated) This method is \fBdeprecated\fR and mapped to the \(dqsan_path_err\(dq method.
 See the \fIdelay_watch_checks\fR and \fIdelay_wait_checks\fR options above
-- 
2.54.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 3/3] multipath-tools: remove explicit widths from macros in the remaining man pages
       [not found] <20260607103831.336833-1-xose.vazquez@gmail.com>
  2026-06-07 10:38 ` [PATCH 1/3] multipath-tools: delete obsolete information from multipath.conf.5 Xose Vazquez Perez
  2026-06-07 10:38 ` [PATCH 2/3] multipath-tools: remove explicit width from .TP macros in multipath.conf.5 Xose Vazquez Perez
@ 2026-06-07 10:38 ` Xose Vazquez Perez
  2026-06-10 23:41   ` Benjamin Marzinski
  2 siblings, 1 reply; 6+ messages in thread
From: Xose Vazquez Perez @ 2026-06-07 10:38 UTC (permalink / raw)
  Cc: Xose Vazquez Perez, Martin Wilck, Benjamin Marzinski,
	Christophe Varoqui, DM_DEVEL-ML

This allows man/groff to calculate the width automatically, preventing
excessive whitespace and improving readability on various terminal sizes.

Cc: Martin Wilck <mwilck@suse.com>
Cc: Benjamin Marzinski <bmarzins@redhat.com>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: DM_DEVEL-ML <dm-devel@lists.linux.dev>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 libmpathpersist/mpath_persistent_reserve_in.3  |  4 ++--
 libmpathpersist/mpath_persistent_reserve_out.3 |  4 ++--
 mpathpersist/mpathpersist.8.in                 |  2 +-
 multipath/multipath.8.in                       |  6 +++---
 multipath/multipath.conf.5.in                  | 10 +++++-----
 multipathd/multipathd.8.in                     |  9 ++++-----
 6 files changed, 17 insertions(+), 18 deletions(-)

diff --git a/libmpathpersist/mpath_persistent_reserve_in.3 b/libmpathpersist/mpath_persistent_reserve_in.3
index 4ac43fa3..871242d5 100644
--- a/libmpathpersist/mpath_persistent_reserve_in.3
+++ b/libmpathpersist/mpath_persistent_reserve_in.3
@@ -35,7 +35,7 @@ the DM device and gets the response.
 .TP
 .B Parameters:
 .RS
-.TP 12
+.TP
 .I fd
 The file descriptor of a multipath device. Input argument.
 .TP
@@ -57,7 +57,7 @@ Set verbosity level. Input argument. value:[0-3]. 0->Errors, 1->Warnings, 2->Inf
 .SH RETURNS
 .\" ----------------------------------------------------------------------------
 .
-.TP 12
+.TP
 .B MPATH_PR_SUCCESS
 If PR command successful.
 .TP
diff --git a/libmpathpersist/mpath_persistent_reserve_out.3 b/libmpathpersist/mpath_persistent_reserve_out.3
index ab1f923e..bf44777c 100644
--- a/libmpathpersist/mpath_persistent_reserve_out.3
+++ b/libmpathpersist/mpath_persistent_reserve_out.3
@@ -35,7 +35,7 @@ the DM device and gets the response.
 .TP
 .B Parameters:
 .RS
-.TP 12
+.TP
 .I fd
 The file descriptor of a multipath device. Input argument.
 .TP
@@ -74,7 +74,7 @@ Set verbosity level. Input argument. value: 0 to 3. 0->Errors, 1->Warnings, 2->I
 .SH RETURNS
 .\" ----------------------------------------------------------------------------
 .
-.TP 12
+.TP
 .B MPATH_PR_SUCCESS
 If PR command successful else returns any one of the status mentioned below.
 .TP
diff --git a/mpathpersist/mpathpersist.8.in b/mpathpersist/mpathpersist.8.in
index fecef0d6..9a5a716b 100644
--- a/mpathpersist/mpathpersist.8.in
+++ b/mpathpersist/mpathpersist.8.in
@@ -50,7 +50,7 @@ various options.
 .BI \-verbose|\-v " level"
 Verbosity:
 .RS
-.TP 5
+.TP
 .I 0
 Critical messages.
 .TP
diff --git a/multipath/multipath.8.in b/multipath/multipath.8.in
index b88e9a4c..0b170101 100644
--- a/multipath/multipath.8.in
+++ b/multipath/multipath.8.in
@@ -100,7 +100,7 @@ The \fBdevice\fR argument restricts \fBmultipath\fR's operation to devices match
 expression. The argument may refer either to a multipath map or to
 its components ("paths"). The expression may be in one of the following formats:
 .
-.TP 1.4i
+.TP
 .B device node
 file name of a device node, e.g. \fI/dev/dm-10\fR or \fI/dev/sda\fR. If the node refers
 to an existing device mapper device representing a multipath map, this selects
@@ -195,8 +195,8 @@ creating \fI@CONFIGFILE@\fR.
 .BI \-v " level"
 Verbosity of information printed to stdout in default and "list" operation
 modes. The default level is \fI-v 2\fR.
-.RS 1.2i
-.TP 1.2i
+.RS
+.TP
 .I 0
 Nothing is printed.
 .TP
diff --git a/multipath/multipath.conf.5.in b/multipath/multipath.conf.5.in
index ef08ed2b..6990d2bf 100644
--- a/multipath/multipath.conf.5.in
+++ b/multipath/multipath.conf.5.in
@@ -1527,7 +1527,7 @@ the same WWID in the \fIbindings_file\fR.
 The following attributes are optional; if not set the default values
 are taken from the \fIoverrides\fR, \fIdevices\fR, or \fIdefaults\fR
 section:
-.sp 1
+.sp
 .PD .1v
 .RS
 .TP
@@ -1704,7 +1704,7 @@ has no effect.
 The following attributes are optional; if not set the default values
 are taken from the \fIdefaults\fR
 section:
-.sp 1
+.sp
 .PD .1v
 .RS
 .TP
@@ -1782,7 +1782,7 @@ section:
 .
 The overrides section recognizes the following optional attributes; if not set
 the values are taken from the \fIdevices\fR or \fIdefaults\fR sections:
-.sp 1
+.sp
 .PD .1v
 .RS
 .TP
@@ -1878,7 +1878,7 @@ exactly. The protocol that a path is using can be viewed by running
 .LP
 The following attributes are optional; if not set, the default values are taken
 from the \fIoverrides\fR, \fIdevices\fR, or \fIdefaults\fR section:
-.sp 1
+.sp
 .PD .1v
 .RS
 .TP
@@ -1969,7 +1969,7 @@ increase and the threshold is never reached. Ticks are the time between
 path checks by multipathd, which is variable and controlled by the
 \fIpolling_interval\fR and \fImax_polling_interval\fR parameters.
 .
-.RS 8
+.RS
 .LP
 This algorithm is superseded by the \(dqmarginal_path\(dq failure tracking,
 but remains supported for backward compatibility.
diff --git a/multipathd/multipathd.8.in b/multipathd/multipathd.8.in
index 38a243e3..d766dc65 100644
--- a/multipathd/multipathd.8.in
+++ b/multipathd/multipathd.8.in
@@ -416,7 +416,7 @@ appropriate device information. The following wildcards are supported.
 .TP
 .B Multipath format wildcards
 .RS
-.TP 12
+.TP
 .B %n
 The device name.
 .TP
@@ -521,7 +521,7 @@ the configured one).
 .TP
 .B Path format wildcards
 .RS
-.TP 12
+.TP
 .B %w
 The device WWID (uuid).
 .TP
@@ -548,7 +548,7 @@ sysfs states, it shows "running".
 .B %T
 The multipathd path checker state of the device. The possible states are:
 .RS
-.TP 12
+.TP
 .I ready
 The device is ready to handle IO.
 .TP
@@ -693,8 +693,7 @@ Overrides the \fImax_fds\fR configuration setting.
 .
 .BR multipathc (8),
 .BR multipath (8),
-.BR kpartx (8)
-.RE
+.BR kpartx (8),
 .BR sd_notify (3),
 .BR systemd.service (5).
 .
-- 
2.54.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] multipath-tools: delete obsolete information from multipath.conf.5
  2026-06-07 10:38 ` [PATCH 1/3] multipath-tools: delete obsolete information from multipath.conf.5 Xose Vazquez Perez
@ 2026-06-10 22:46   ` Benjamin Marzinski
  0 siblings, 0 replies; 6+ messages in thread
From: Benjamin Marzinski @ 2026-06-10 22:46 UTC (permalink / raw)
  To: Xose Vazquez Perez; +Cc: Martin Wilck, Christophe Varoqui, DM_DEVEL-ML

On Sun, Jun 07, 2026 at 12:38:21PM +0200, Xose Vazquez Perez wrote:
> repeat_count (rr_min_io, rr_min_io_rq, rr_weight) has been unsupported
> since kernel 4.6 ( commits 90a4323ccfea and  21136f89d76d ).
> 
> And also clean up other deprecated entries.
> 
> Cc: Martin Wilck <mwilck@suse.com>
> Cc: Benjamin Marzinski <bmarzins@redhat.com>
> Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> Cc: DM_DEVEL-ML <dm-devel@lists.linux.dev>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> ---
>  multipath/multipath.conf.5.in | 69 +++--------------------------------
>  1 file changed, 6 insertions(+), 63 deletions(-)
> 
> diff --git a/multipath/multipath.conf.5.in b/multipath/multipath.conf.5.in
> index 84cd1a0a..e3a9811c 100644
> --- a/multipath/multipath.conf.5.in
> +++ b/multipath/multipath.conf.5.in
> @@ -6,7 +6,7 @@
>  .\" Update the date below if you make any significant change.
>  .\" ----------------------------------------------------------------------------
>  .
> -.TH MULTIPATH.CONF 5 2025-09-12 Linux
> +.TH MULTIPATH.CONF 5 2026-05-27 Linux
>  .
>  .
>  .\" ----------------------------------------------------------------------------
> @@ -198,9 +198,8 @@ kernel multipath target:
>  .TP 12
>  .I "round-robin 0"
>  Choose the path for the next bunch of I/O by looping through every path in the
> -path group, sending \fBthe same number of I/O requests\fR to each path. Some
> -aspects of behavior can be controlled with the attributes: \fIrr_min_io\fR,
> -\fIrr_min_io_rq\fR and \fIrr_weight\fR.
> +path group, sending \fBthe same number of I/O requests\fR to each path.
> +

For the path selectors, we know the number of I/O requests that will
be sent, and it's 1. So instead of saying "the next bunch of I/O", we
should probably just say "the next I/O request". And we can probably
just drop the "sending the same number of  I/O  requests to each path"
part of the round-robin definition.

>  .TP
>  .I "queue-length 0"
>  (Since 2.6.31 kernel) Choose the path for the next bunch of I/O based on \fBthe lowest
> @@ -598,24 +597,12 @@ The default is: \fBmanual\fR
>  .
>  .TP
>  .B rr_min_io
> -Number of I/O requests to route to a path before switching to the next in the
> -same path group. This is only for \fIBlock I/O\fR(BIO) based multipath and
> -only apply to \fIround-robin\fR path_selector.
> -.RS
> -.TP
> -The default is: \fB1000\fR
> -.RE
> +(Deprecated since kernel 4.6) This option is not supported anymore, and will be ignored.
>  .
>  .
>  .TP
>  .B rr_min_io_rq
> -Number of I/O requests to route to a path before switching to the next in the
> -same path group. This is only for \fIRequest\fR based multipath and
> -only apply to \fIround-robin\fR path_selector.
> -.RS
> -.TP
> -The default is: \fB1\fR
> -.RE
> +(Deprecated since kernel 4.6) This option is not supported anymore, and will be ignored.
>  .
>  .
>  .TP
> @@ -634,16 +621,7 @@ The default is: \fBmax\fR
>  .
>  .TP
>  .B rr_weight
> -If set to \fIpriorities\fR the multipath configurator will assign path weights
> -as "path prio * rr_min_io". Possible values are
> -.I priorities
> -or
> -.I uniform .
> -Only apply to \fIround-robin\fR path_selector.
> -.RS
> -.TP
> -The default is: \fBuniform\fR
> -.RE
> +(Deprecated since kernel 4.6) This option is not supported anymore, and will be ignored.
>  .
>  .
>  .TP
> @@ -821,28 +799,16 @@ The default is: \fB<unset>\fR
>  .TP
>  .B bindings_file
>  (Deprecated) This option is not supported anymore, and will be ignored.
> -.RS
> -.TP
> -The compiled-in value is: \fB@STATE_DIR@/bindings\fR
> -.RE
>  .
>  .
>  .TP
>  .B wwids_file
>  (Deprecated) This option is not supported anymore, and will be ignored.
> -.RS
> -.TP
> -The compiled-in value is: \fB@STATE_DIR@/wwids\fR
> -.RE
>  .
>  .
>  .TP
>  .B prkeys_file
>  (Deprecated) This option is not supported anymore, and will be ignored.
> -.RS
> -.TP
> -The compiled-in value is: \fB@STATE_DIR@/prkeys\fR
> -.RE
>  .
>  .
>  .TP
> @@ -1008,10 +974,6 @@ The default is: \fB<unset>\fR
>  .TP
>  .B config_dir
>  (Deprecated) This option is not supported anymore, and will be ignored.
> -.RS
> -.TP
> -The compiled-in value is: \fB@CONFIGDIR@\fR
> -.RE

We don't actually specify the location of these files and directories
elsewhere in this man page (except for the bindings file), and we do
refer to them by these config option names. We should probably
replace the references to these option names with the actual file
locations, and probably also list the files in a FILES section.

-Ben


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/3] multipath-tools: remove explicit width from .TP macros in multipath.conf.5
  2026-06-07 10:38 ` [PATCH 2/3] multipath-tools: remove explicit width from .TP macros in multipath.conf.5 Xose Vazquez Perez
@ 2026-06-10 23:41   ` Benjamin Marzinski
  0 siblings, 0 replies; 6+ messages in thread
From: Benjamin Marzinski @ 2026-06-10 23:41 UTC (permalink / raw)
  To: Xose Vazquez Perez; +Cc: Martin Wilck, Christophe Varoqui, DM_DEVEL-ML

On Sun, Jun 07, 2026 at 12:38:22PM +0200, Xose Vazquez Perez wrote:
> This allows man/groff to calculate the width automatically, preventing
> excessive whitespace and improving readability on various terminal sizes.
> 
> Cc: Martin Wilck <mwilck@suse.com>
> Cc: Benjamin Marzinski <bmarzins@redhat.com>
> Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> Cc: DM_DEVEL-ML <dm-devel@lists.linux.dev>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 3/3] multipath-tools: remove explicit widths from macros in the remaining man pages
  2026-06-07 10:38 ` [PATCH 3/3] multipath-tools: remove explicit widths from macros in the remaining man pages Xose Vazquez Perez
@ 2026-06-10 23:41   ` Benjamin Marzinski
  0 siblings, 0 replies; 6+ messages in thread
From: Benjamin Marzinski @ 2026-06-10 23:41 UTC (permalink / raw)
  To: Xose Vazquez Perez; +Cc: Martin Wilck, Christophe Varoqui, DM_DEVEL-ML

On Sun, Jun 07, 2026 at 12:38:23PM +0200, Xose Vazquez Perez wrote:
> This allows man/groff to calculate the width automatically, preventing
> excessive whitespace and improving readability on various terminal sizes.
> 
> Cc: Martin Wilck <mwilck@suse.com>
> Cc: Benjamin Marzinski <bmarzins@redhat.com>
> Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> Cc: DM_DEVEL-ML <dm-devel@lists.linux.dev>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2026-06-10 23:41 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20260607103831.336833-1-xose.vazquez@gmail.com>
2026-06-07 10:38 ` [PATCH 1/3] multipath-tools: delete obsolete information from multipath.conf.5 Xose Vazquez Perez
2026-06-10 22:46   ` Benjamin Marzinski
2026-06-07 10:38 ` [PATCH 2/3] multipath-tools: remove explicit width from .TP macros in multipath.conf.5 Xose Vazquez Perez
2026-06-10 23:41   ` Benjamin Marzinski
2026-06-07 10:38 ` [PATCH 3/3] multipath-tools: remove explicit widths from macros in the remaining man pages Xose Vazquez Perez
2026-06-10 23:41   ` Benjamin Marzinski

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.