On 6/11/26 12:46 AM, Benjamin Marzinski wrote:

On Sun, Jun 07, 2026 at 12:38:21PM +0200, Xose Vazquez Perez wrote:

@@ -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.

@@ -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.
Sounds good. I will address both points and submit the updates in
separate patches.

Reply via email to