Re: [dm-devel] [PATCH] multipath-tools: Manual failback fix whenpriority changes

[Christophe Varoqui <christophe varoqui gmail com>]

On lun., 2011-06-06 at 18:55 -0400, Moger, Babu wrote:
> Christophe,
>  Did you get a chance to look at this patch?  Let me know if you have any feedback.   
> 
Thank you for the reminder. This patch indeed escaped my attention.
Now merged.

> > -----Original Message-----
> > From: Moger, Babu
> > Sent: Friday, May 27, 2011 9:30 AM
> > To: dm-devel redhat com
> > Subject: [dm-devel] [PATCH] multipath-tools: Manual failback fix
> > whenpriority changes
> > 
> > Current code switches the path-group when there is a change in priority.
> > However,
> > this is not the right thing to do when failback is set to manual. This
> > patch fixes
> > this problem. Call update_path_groups only if failback is immediate.
> > 
> > Signed-off-by: Babu Moger <babu moger netapp com>
> > 
> > ---
> > --- multipath-tools/multipathd/main.c.orig	2011-05-26 08:06:08.000000000
> > -0500
> > +++ multipath-tools/multipathd/main.c	2011-05-26 08:21:39.000000000
> > -0500
> > @@ -1189,7 +1189,8 @@ check_path (struct vectors * vecs, struc
> >  	condlog(4, "path prio refresh");
> > 
> >  	if (update_prio(pp, new_path_up) &&
> > -	    pp->mpp->pgpolicyfn == (pgpolicyfn *)group_by_prio)
> > +	    (pp->mpp->pgpolicyfn == (pgpolicyfn *)group_by_prio) &&
> > +	     pp->mpp->pgfailback == -FAILBACK_IMMEDIATE)
> >  		update_path_groups(pp->mpp, vecs, !new_path_up);
> >  	else if (need_switch_pathgroup(pp->mpp, 0)) {
> >  		if (pp->mpp->pgfailback > 0 &&
> > 
> > 

-- 
Christophe Varoqui
OpenSVC - Tools to scale
http://www.opensvc.com/