[dm-devel] SUSE multipath-tools patch resync

Hannes Reinecke hare at suse.de
Wed May 25 09:08:58 UTC 2011


On 05/25/2011 09:51 AM, Christophe Varoqui wrote:
> On mer., 2011-05-18 at 17:03 +0200, Hannes Reinecke wrote:
>> git.kernel.org:/pub/scm/linux/kernel/git/hare/multipath-tools.git
>
> I started the merge. I'll post comments along the reading. It seems
> there won't be much : this patchset is clearly a must-have.
>
:-)

With some things so bloody obvious that I keep wondering if I'm the
only one seeing these ...

> ...
>
> Shouldn't this one be reverted ? Seems the real thing is
> 10d68a92b2a0027d5468cf76656692f34f6dbc54
>
>
> commit efc8ace4b335e752a7d28aca6040af0f9fe37530
> Author: Hannes Reinecke<hare at suse.de>
> Date:   Mon Feb 1 09:46:57 2010 +0100
>
>      Add 'max_polling_interval' config variable
>
>      We should be able to set the 'max_polling_interval' variable
>      manually. Especially systems requiring precise failover timing
>      will want to disable the automatic polling interval increase.
>
>      Signed-off-by: Hannes Reinecke<hare at suse.de>
>
> diff --git a/multipathd/main.c b/multipathd/main.c
> index 7659bb2..6867f23 100644
> --- a/multipathd/main.c
> +++ b/multipathd/main.c
> @@ -1096,9 +1096,9 @@ check_path (struct vectors * vecs, struct path *
> pp)
>                           * max at conf->max_checkint
>                           */
>                          if (pp->checkint<  (conf->max_checkint / 2))
> -                               pp->checkint = 2 * pp->checkint;
> +                           pp->checkint = 2 * pp->checkint;
>                          else
> -                               pp->checkint = conf->max_checkint;
> +                           pp->checkint = conf->max_checkint;
>
>                          pp->tick = pp->checkint;
>                          condlog(4, "%s: delay next check %is",
>
>
>
Oh, yes. Merge error. Please do not include this.
Thanks for your effort.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare at suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)




More information about the dm-devel mailing list