<div dir="ltr">Thank you for the review, the commit is in.<div><br></div><div>the user reporting this, seems to face exactly the problem mentioned in 7db0c44 :</div><div><br></div><div>commit 7db0c4466c0e5ed2179710f93d1d14a0bf92922a<br>...</div><div>    And the default timeout should be increased to 4 seconds to ensure<br>    multipath runs correctly on large installations.<br><br></div><div>He clearly qualifies as "large installation", and reach that timeout when submitting a large number of mpathpersist registers in parallel on boot.</div><div>Regardless, needing a uxsock_timeout greater than 4 seconds hints we do something non-optimal there.</div><div>Have you already had a chance to investigate possible enhancements on this ?</div><div><br></div><div>Christophe</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 14, 2020 at 10:39 AM Martin Wilck <<a href="mailto:mwilck@suse.com">mwilck@suse.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, 2020-12-14 at 08:56 +0100, Christophe Varoqui wrote:<br>
> Hello,<br>
> <br>
> a user brought to my attention "multipathd show config | grep<br>
> usock_timeout" does not agree with the default value stated in the<br>
> manpage : 4000 instead of 1000.<br>
> <br>
> And indeed,<br>
> <br>
> ./libmpathcmd/mpath_cmd.h:#define DEFAULT_REPLY_TIMEOUT 4000<br>
> <br>
> Can you confirm this change is valid ?<br>
<br>
Yes. <br>
Reviewed-by: Martin Wilck <<a href="mailto:mwilck@suse.com" target="_blank">mwilck@suse.com</a>><br>
<br>
You may want to add<br>
<br>
Fixes: 7db0c44 ("multipathd: Set CLI timeout correctly")<br>
<br>
because that commit changed the timeout without changing the man page.<br>
<br>
Martin<br>
<br>
<br>
> <br>
> diff --git a/multipath/multipath.conf.5 b/multipath/multipath.conf.5<br>
> index d2101ed6..7242d39b 100644<br>
> --- a/multipath/multipath.conf.5<br>
> +++ b/multipath/multipath.conf.5<br>
> @@ -1153,7 +1153,7 @@ In these cases it is recommended to increase<br>
> the CLI timeout to avoid<br>
>  those issues.<br>
>  .RS<br>
>  .TP<br>
> -The default is: \fB1000\fR<br>
> +The default is: \fB4000\fR<br>
>  .RE<br>
>  .<br>
>  .<br>
> <br>
> Best Regards,<br>
> Christophe<br>
<br>
<br>
</blockquote></div>