<div dir="ltr">Could you please rebase this one ?<div>Thanks.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 12, 2016 at 4:24 PM, Xose Vazquez Perez <span dir="ltr"><<a href="mailto:xose.vazquez@gmail.com" target="_blank">xose.vazquez@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">New undocumented feature, added in e83068a5f:<br>
<br>
    dm mpath: add optional "queue_mode" feature<br>
<br>
    Allow a user to specify an optional feature 'queue_mode <mode>' where<br>
    <mode> may be "bio", "rq" or "mq" -- which corresponds to bio-based,<br>
    request_fn rq-based, and blk-mq rq-based respectively.<br>
<br>
    If the queue_mode feature isn't specified the default for the<br>
    "multipath" target is still "rq" but if dm_mod.use_blk_mq is set to Y<br>
    it'll default to mode "mq".<br>
<br>
    This new queue_mode feature introduces the ability for each multipath<br>
    device to have its own queue_mode (whereas before this feature all<br>
    multipath devices effectively had to have the same queue_mode).<br>
<br>
<br>
Also add more info to other features.<br>
<br>
Cc: Mike Snitzer <<a href="mailto:snitzer@redhat.com">snitzer@redhat.com</a>><br>
Cc: Christophe Varoqui <<a href="mailto:christophe.varoqui@opensvc.com">christophe.varoqui@opensvc.<wbr>com</a>><br>
Cc: device-mapper development <<a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a>><br>
Signed-off-by: Xose Vazquez Perez <<a href="mailto:xose.vazquez@gmail.com">xose.vazquez@gmail.com</a>><br>
---<br>
 multipath/multipath.conf.5 | 36 ++++++++++++++++++++++++------<wbr>------<br>
 1 file changed, 24 insertions(+), 12 deletions(-)<br>
<br>
diff --git a/multipath/multipath.conf.5 b/multipath/multipath.conf.5<br>
index fd4b1f2..40df315 100644<br>
--- a/multipath/multipath.conf.5<br>
+++ b/multipath/multipath.conf.5<br>
@@ -3,11 +3,11 @@<br>
 .\" Make sure there are no errors with:<br>
 .\" groff -z -wall -b -e -t multipath/multipath.conf.5<br>
 .\"<br>
-.\" TODO: Look for XXX<br>
+.\" TODO: Look for XXX and ???<br>
 .\"<br>
 .\" ------------------------------<wbr>------------------------------<wbr>----------------<br>
 .<br>
-.TH MULTIPATH.CONF 5 2016-08-09 "Linux"<br>
+.TH MULTIPATH.CONF 5 2016-08-12 "Linux"<br>
 .<br>
 .<br>
 .\" ------------------------------<wbr>------------------------------<wbr>----------------<br>
@@ -166,12 +166,15 @@ Loop through every path in the path group, sending the same amount of IO to<br>
 each. Some aspects of behavior can be controlled with the attributes:<br>
 \fIrr_min_io\fR, \fIrr_min_io_rq\fR and \fIrr_weight\fR.<br>
 .TP<br>
+.\" XXX<br>
 .I "queue-length 0"<br>
-Send the next bunch of IO down the path with the least amount of outstanding IO.<br>
+(Since ??? kernel) Choose the path for the next bunch of IO based on the amount<br>
+of outstanding IO to the path.<br>
 .TP<br>
+.\" XXX<br>
 .I "service-time 0"<br>
-Choose the path for the next bunch of IO based on the amount of outstanding IO<br>
-to the path and its relative throughput.<br>
+(Since ??? kernel) Choose the path for the next bunch of IO based on the amount<br>
+of outstanding IO to the path and its relative throughput.<br>
 .TP<br>
 Default value is: \fBservice-time 0\fR<br>
 .RE<br>
@@ -333,22 +336,31 @@ Default value is: \fB<unset>\fR<br>
 .TP<br>
 .B features<br>
 Specify any device-mapper features to be used. Syntax is \fInum list\fR<br>
-where \fInum\fR is the number, between 0 and 6, of features in \fIlist\fR.<br>
+where \fInum\fR is the number, between 0 and 8, of features in \fIlist\fR.<br>
 Possible values for the feature list are:<br>
 .RS<br>
 .TP 12<br>
+.\" XXX<br>
 .I queue_if_no_path<br>
-(Superseded by \fIno_path_retry\fR) Queue IO if no path is active. Identical to<br>
-the \fIno_path_retry\fR with \fIqueue\fR value. See KNOWN ISSUES.<br>
+(Superseded by \fIno_path_retry\fR) (Since ??? kernel) Queue IO if no path is active.<br>
+Identical to the \fIno_path_retry\fR with \fIqueue\fR value. See KNOWN ISSUES.<br>
 .TP<br>
 .I no_partitions<br>
 Disable automatic partitions generation via kpartx.<br>
 .TP<br>
-.I pg_init_retries<br>
-Number of times to retry pg_init, it must be between 1 and 50.<br>
+.\" XXX<br>
+.I pg_init_retries <times><br>
+(Since ??? kernel) Number of times to retry pg_init, it must be between 1 and 50.<br>
 .TP<br>
-.I pg_init_delay_msecs<br>
-Number of msecs before pg_init retry, it must be between 0 and 60000.<br>
+.\" XXX<br>
+.I pg_init_delay_msecs <msecs><br>
+(Since ??? kernel) Number of msecs before pg_init retry, it must be between 0 and 60000.<br>
+.TP<br>
+.\" XXX<br>
+.I queue_mode <mode><br>
+(Since ??? kernel) Select the the queue_mode per multipath device.<br>
+Where <mode> can be \fIbio\fR, \fIrq\fR or \fImq\fR. Which corresponds to<br>
+bio-based, request_fn rq-based, and blk-mq rq-based respectively.<br>
 .TP<br>
 Default value is: \fB0\fR<br>
 .RE<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.7.4<br>
<br>
</font></span></blockquote></div><br></div>