Re: [PATCH 1/3] multipath-tools: delete obsolete information from multipath.conf.5
Xose Vazquez Perez <[email protected]>
| Newsgroups | dev.linux.lists.dm-devel |
|---|---|
| Message-ID | <[email protected]> |
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.