[dm-devel] [PATCH] libmultipath: ensure dev_loss_tmo will be update to MAX_DEV_LOSS_TMO if no_path_retry set to queue

Christophe Varoqui christophe.varoqui at opensvc.com
Sat Nov 26 09:05:10 UTC 2016


Applied, thanks.

On Fri, Nov 25, 2016 at 7:36 AM, <peng.liang5 at zte.com.cn> wrote:

> From: PengLiang <peng.liang5 at zte.com.cn>
>
> If no_path_retry set to queue, we should make sure dev_loss_tmo update to
> MAX_DEV_LOSS_TMO.
> But, it will be limit to 600 if fast_io_fail_tmo set to off or 0 meanwhile.
>
> Signed-off-by: PengLiang <peng.liang5 at zte.com.cn>
> ---
>  libmultipath/discovery.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c
> index aaa915c..05b0842 100644
> --- a/libmultipath/discovery.c
> +++ b/libmultipath/discovery.c
> @@ -608,7 +608,8 @@ sysfs_set_rport_tmo(struct multipath *mpp, struct path
> *pp)
>                                 goto out;
>                         }
>                 }
> -       } else if (mpp->dev_loss > DEFAULT_DEV_LOSS_TMO) {
> +       } else if (mpp->dev_loss > DEFAULT_DEV_LOSS_TMO &&
> +               mpp->no_path_retry != NO_PATH_RETRY_QUEUE) {
>                 condlog(3, "%s: limiting dev_loss_tmo to %d, since "
>                         "fast_io_fail is not set",
>                         rport_id, DEFAULT_DEV_LOSS_TMO);
> --
> 2.8.1.windows.1
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20161126/2bc09df0/attachment.htm>


More information about the dm-devel mailing list