[dm-devel] [PATCH] multipathd: not skip paths not belonging to a mpp in check_path

Christophe Varoqui christophe.varoqui at opensvc.com
Thu Feb 27 08:14:50 UTC 2014


Applied.

Indeed, it seems I messed up the merge.

Thanks.

On Sat, Feb 22, 2014 at 1:34 PM, Vaughan Cao <vaughan.cao at oracle.com> wrote:
> From: Vaughan Cao <Vaughan.Cao at oracle.com>
>
> commit bf4c80aee4e977e89d6925df569647acd3d5efef
> multipathd: add path when transitioned from 'blocked' state
> intends to check paths not belonging to a mpp usually because of
> they are in 'blocked' state and we cannot get the path uuid.
> So these paths ought not to be skipped at the start of the function.
> It could be a fault when merge the original patch -
> http://www.redhat.com/archives/dm-devel/2013-December/msg00085.html
>
> Signed-off-by: Vaughan Cao <Vaughan.Cao at oracle.com>
> ---
>  multipathd/main.c |    3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/multipathd/main.c b/multipathd/main.c
> index af93f32..56d00d3 100644
> --- a/multipathd/main.c
> +++ b/multipathd/main.c
> @@ -1118,9 +1118,6 @@ check_path (struct vectors * vecs, struct path * pp)
>         int chkr_new_path_up = 0;
>         int oldchkrstate = pp->chkrstate;
>
> -       if (!pp->mpp)
> -               return 0;
> -
>         if (pp->tick && --pp->tick)
>                 return 0; /* don't check this path yet */
>
> --
> 1.7.9.5
>
> --
> dm-devel mailing list
> dm-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel




More information about the dm-devel mailing list