diff mbox series

multipath-tools: multipath.conf man page housekeeping

Message ID 20221105213733.182458-1-xose.vazquez@gmail.com (mailing list archive)
State Not Applicable, archived
Delegated to: christophe varoqui
Headers show
Series multipath-tools: multipath.conf man page housekeeping | expand

Commit Message

Xose Vazquez Perez Nov. 5, 2022, 9:37 p.m. UTC
Mark deprecated options as (Deprecated), and standardize the description
Document pg_timeout
Default value of "features" is "0"
directio is also an alternative to readsector0

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@redhat.com>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 multipath/multipath.conf.5 | 41 ++++++++++++++++++++------------------
 1 file changed, 22 insertions(+), 19 deletions(-)

Comments

Benjamin Marzinski Nov. 8, 2022, 11:11 p.m. UTC | #1
On Sat, Nov 05, 2022 at 10:37:33PM +0100, Xose Vazquez Perez wrote:
> Mark deprecated options as (Deprecated), and standardize the description
> Document pg_timeout
> Default value of "features" is "0"
> directio is also an alternative to readsector0
> 
> 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@redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
Reviewed-by: Benjamin Marzinski <bmarzins@redhat.com>
> ---
>  multipath/multipath.conf.5 | 41 ++++++++++++++++++++------------------
>  1 file changed, 22 insertions(+), 19 deletions(-)
> 
> diff --git a/multipath/multipath.conf.5 b/multipath/multipath.conf.5
> index 8e28ba12..063b05a7 100644
> --- a/multipath/multipath.conf.5
> +++ b/multipath/multipath.conf.5
> @@ -178,7 +178,7 @@ The default is: \fBno\fR
>  .
>  .TP
>  .B multipath_dir
> -This option is not supported any more. The value is ignored.
> +(Deprecated) This option is not supported any more, and the value is ignored.
>  .
>  .
>  .TP
> @@ -242,6 +242,11 @@ The default is: \fBfailover\fR
>  .
>  .
>  .TP
> +.B pg_timeout
> +(Deprecated) This option is not supported any more, and the value is ignored.
> +.
> +.
> +.TP
>  .B uid_attrs
>  .
>  Setting this option activates \fBmerging uevents\fR by WWID, which may improve
> @@ -283,7 +288,7 @@ The default is: \fBID_WWN\fR, for NVMe devices
>  .
>  .TP
>  .B getuid_callout
> -This option is not supported any more. The value is ignored.
> +(Deprecated) This option is not supported any more, and the value is ignored.
>  .
>  .
>  .TP
> @@ -468,7 +473,7 @@ cannot be changed. \fInvme:tcp\fR paths are only supported in multipath
>  devices with queue_mode set to \fIbio\fR. multipath will automatically
>  set this when creating a device with \fInvme:tcp\fR paths.
>  .TP
> -The default is: \fB<unset>\fR
> +The default is: \fB0\fR
>  .RE
>  .
>  .
> @@ -484,7 +489,7 @@ second, until \fIchecker_timeout\fR seconds have elapsed. Possible values are:
>  .TP 12
>  .I readsector0
>  (Deprecated) Read the first sector of the device. This checker is being
> -deprecated, please use \fItur\fR instead.
> +deprecated, please use \fItur\fR or \fIdirectio\fR instead.
>  .TP
>  .I tur
>  Issue a \fITEST UNIT READY\fR command to the device.
> @@ -556,7 +561,7 @@ The default is: \fBmanual\fR
>  .
>  .
>  .TP
> -.B  rr_min_io
> +.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.
> @@ -740,7 +745,7 @@ The default is: \fB<unset>\fR
>  .
>  .TP
>  .B bindings_file
> -This option is deprecated, and will be removed in a future release.
> +(Deprecated) This option is deprecated, and will be removed in a future release.
>  The full pathname of the binding file to be used when the user_friendly_names
>  option is set.
>  .RS
> @@ -751,7 +756,7 @@ The default is: \fB/etc/multipath/bindings\fR
>  .
>  .TP
>  .B wwids_file
> -This option is deprecated, and will be removed in a future release.
> +(Deprecated) This option is deprecated, and will be removed in a future release.
>  The full pathname of the WWIDs file, which is used by multipath to keep track
>  of the WWIDs for LUNs it has created multipath devices on in the past.
>  .RS
> @@ -762,7 +767,7 @@ The default is: \fB/etc/multipath/wwids\fR
>  .
>  .TP
>  .B prkeys_file
> -This option is deprecated, and will be removed in a future release.
> +(Deprecated) This option is deprecated, and will be removed in a future release.
>  The full pathname of the prkeys file, which is used by multipathd to keep
>  track of the persistent reservation key used for a specific WWID, when
>  \fIreservation_key\fR is set to \fBfile\fR.
> @@ -934,7 +939,7 @@ The default is: \fB<unset>\fR
>  .
>  .TP
>  .B config_dir
> -This option is not supported any more. The value is ignored.
> +(Deprecated) This option is not supported any more, and the value is ignored.
>  .
>  .
>  .TP
> @@ -1047,7 +1052,7 @@ The default is: \fBno\fR
>  .
>  .TP
>  .B delay_watch_checks
> -This option is \fBdeprecated\fR, and mapped to \fIsan_path_err_forget_rate\fR.
> +(Deprecated) This option is \fBdeprecated\fR, and mapped to \fIsan_path_err_forget_rate\fR.
>  If this is set to a value greater than 0 and no \fIsan_path_err\fR options
>  are set, \fIsan_path_err_forget_rate\fR will be set to the value of
>  \fIdelay_watch_checks\fR and \fIsan_path_err_threshold\fR will be set to 1.
> @@ -1061,7 +1066,7 @@ The default is: \fBno\fR
>  .
>  .TP
>  .B delay_wait_checks
> -This option is \fBdeprecated\fR, and mapped to \fIsan_path_err_recovery_time\fR.
> +(Deprecated) This option is \fBdeprecated\fR, and mapped to \fIsan_path_err_recovery_time\fR.
>  If this is set to a value greater than 0 and no \fIsan_path_err\fR options
>  are set, \fIsan_path_err_recovery_time\fR will be set to the value of
>  \fIdelay_wait_checks\fR times \fImax_polling_interval\fR. This will give
> @@ -1228,8 +1233,7 @@ The default is: \fBno\fR
>  .
>  .TP
>  .B disable_changed_wwids
> -This option is deprecated and ignored. If the WWID of a path suddenly changes,
> -multipathd handles it as if it was removed and then added again.
> +(Deprecated) This option is not supported any more, and the value is ignored.
>  .RE
>  .
>  .
> @@ -1848,7 +1852,7 @@ normal pathgroup. The logic of determining \(dqshaky\(dq condition, as well as
>  the logic when to reinstate, differs between the three methods.
>  .TP 8
>  .B \(dqdelay_checks\(dq failure tracking
> -This method is \fBdeprecated\fR and mapped to the \(dqsan_path_err\(dq method.
> +(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
>  for more information.
>  .
> @@ -1878,16 +1882,15 @@ path checks by multipathd, which is variable and controlled by the
>  .
>  .TP
>  .B \(dqFPIN \(dq failure tracking
> +(Deprecated) This method is \fBdeprecated\fR in favor of the \(dqmarginal_path\(dq
> +failure tracking method, and only offered for backward compatibility.
> +.RS 8
> +.LP
>  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
> -tracking method, and only offered for backward compatibility.
> -.
>  .RE
>  .LP
>  See the documentation
> -- 
> 2.38.1
--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel
Martin Wilck Nov. 30, 2022, 9:35 p.m. UTC | #2
On Sat, 2022-11-05 at 22:37 +0100, Xose Vazquez Perez wrote:
> Mark deprecated options as (Deprecated), and standardize the
> description
> Document pg_timeout
> Default value of "features" is "0"
> directio is also an alternative to readsector0
> 
> 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@redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> ---
>  multipath/multipath.conf.5 | 41 ++++++++++++++++++++----------------
> --
>  1 file changed, 22 insertions(+), 19 deletions(-)
> 
> diff --git a/multipath/multipath.conf.5 b/multipath/multipath.conf.5
> index 8e28ba12..063b05a7 100644
> 
>  .
>  .TP
>  .B \(dqFPIN \(dq failure tracking
> +(Deprecated) This method is \fBdeprecated\fR in favor of the
> \(dqmarginal_path\(dq
> +failure tracking method, and only offered for backward
> compatibility.
> +.RS 8
> +.LP

This is wrong. FPIN is definitely *not* deprecated.
The mistake came in with cfff03e ("multipathd: handle fpin events").
The "deprecated" paragraph that you moved from below belongs to the
"san_path_err" paragraph instead.

Martin

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel
diff mbox series

Patch

diff --git a/multipath/multipath.conf.5 b/multipath/multipath.conf.5
index 8e28ba12..063b05a7 100644
--- a/multipath/multipath.conf.5
+++ b/multipath/multipath.conf.5
@@ -178,7 +178,7 @@  The default is: \fBno\fR
 .
 .TP
 .B multipath_dir
-This option is not supported any more. The value is ignored.
+(Deprecated) This option is not supported any more, and the value is ignored.
 .
 .
 .TP
@@ -242,6 +242,11 @@  The default is: \fBfailover\fR
 .
 .
 .TP
+.B pg_timeout
+(Deprecated) This option is not supported any more, and the value is ignored.
+.
+.
+.TP
 .B uid_attrs
 .
 Setting this option activates \fBmerging uevents\fR by WWID, which may improve
@@ -283,7 +288,7 @@  The default is: \fBID_WWN\fR, for NVMe devices
 .
 .TP
 .B getuid_callout
-This option is not supported any more. The value is ignored.
+(Deprecated) This option is not supported any more, and the value is ignored.
 .
 .
 .TP
@@ -468,7 +473,7 @@  cannot be changed. \fInvme:tcp\fR paths are only supported in multipath
 devices with queue_mode set to \fIbio\fR. multipath will automatically
 set this when creating a device with \fInvme:tcp\fR paths.
 .TP
-The default is: \fB<unset>\fR
+The default is: \fB0\fR
 .RE
 .
 .
@@ -484,7 +489,7 @@  second, until \fIchecker_timeout\fR seconds have elapsed. Possible values are:
 .TP 12
 .I readsector0
 (Deprecated) Read the first sector of the device. This checker is being
-deprecated, please use \fItur\fR instead.
+deprecated, please use \fItur\fR or \fIdirectio\fR instead.
 .TP
 .I tur
 Issue a \fITEST UNIT READY\fR command to the device.
@@ -556,7 +561,7 @@  The default is: \fBmanual\fR
 .
 .
 .TP
-.B  rr_min_io
+.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.
@@ -740,7 +745,7 @@  The default is: \fB<unset>\fR
 .
 .TP
 .B bindings_file
-This option is deprecated, and will be removed in a future release.
+(Deprecated) This option is deprecated, and will be removed in a future release.
 The full pathname of the binding file to be used when the user_friendly_names
 option is set.
 .RS
@@ -751,7 +756,7 @@  The default is: \fB/etc/multipath/bindings\fR
 .
 .TP
 .B wwids_file
-This option is deprecated, and will be removed in a future release.
+(Deprecated) This option is deprecated, and will be removed in a future release.
 The full pathname of the WWIDs file, which is used by multipath to keep track
 of the WWIDs for LUNs it has created multipath devices on in the past.
 .RS
@@ -762,7 +767,7 @@  The default is: \fB/etc/multipath/wwids\fR
 .
 .TP
 .B prkeys_file
-This option is deprecated, and will be removed in a future release.
+(Deprecated) This option is deprecated, and will be removed in a future release.
 The full pathname of the prkeys file, which is used by multipathd to keep
 track of the persistent reservation key used for a specific WWID, when
 \fIreservation_key\fR is set to \fBfile\fR.
@@ -934,7 +939,7 @@  The default is: \fB<unset>\fR
 .
 .TP
 .B config_dir
-This option is not supported any more. The value is ignored.
+(Deprecated) This option is not supported any more, and the value is ignored.
 .
 .
 .TP
@@ -1047,7 +1052,7 @@  The default is: \fBno\fR
 .
 .TP
 .B delay_watch_checks
-This option is \fBdeprecated\fR, and mapped to \fIsan_path_err_forget_rate\fR.
+(Deprecated) This option is \fBdeprecated\fR, and mapped to \fIsan_path_err_forget_rate\fR.
 If this is set to a value greater than 0 and no \fIsan_path_err\fR options
 are set, \fIsan_path_err_forget_rate\fR will be set to the value of
 \fIdelay_watch_checks\fR and \fIsan_path_err_threshold\fR will be set to 1.
@@ -1061,7 +1066,7 @@  The default is: \fBno\fR
 .
 .TP
 .B delay_wait_checks
-This option is \fBdeprecated\fR, and mapped to \fIsan_path_err_recovery_time\fR.
+(Deprecated) This option is \fBdeprecated\fR, and mapped to \fIsan_path_err_recovery_time\fR.
 If this is set to a value greater than 0 and no \fIsan_path_err\fR options
 are set, \fIsan_path_err_recovery_time\fR will be set to the value of
 \fIdelay_wait_checks\fR times \fImax_polling_interval\fR. This will give
@@ -1228,8 +1233,7 @@  The default is: \fBno\fR
 .
 .TP
 .B disable_changed_wwids
-This option is deprecated and ignored. If the WWID of a path suddenly changes,
-multipathd handles it as if it was removed and then added again.
+(Deprecated) This option is not supported any more, and the value is ignored.
 .RE
 .
 .
@@ -1848,7 +1852,7 @@  normal pathgroup. The logic of determining \(dqshaky\(dq condition, as well as
 the logic when to reinstate, differs between the three methods.
 .TP 8
 .B \(dqdelay_checks\(dq failure tracking
-This method is \fBdeprecated\fR and mapped to the \(dqsan_path_err\(dq method.
+(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
 for more information.
 .
@@ -1878,16 +1882,15 @@  path checks by multipathd, which is variable and controlled by the
 .
 .TP
 .B \(dqFPIN \(dq failure tracking
+(Deprecated) This method is \fBdeprecated\fR in favor of the \(dqmarginal_path\(dq
+failure tracking method, and only offered for backward compatibility.
+.RS 8
+.LP
 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
-tracking method, and only offered for backward compatibility.
-.
 .RE
 .LP
 See the documentation