Re: SUSE multipath-tools patch resync

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxx>
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@xxxxxxx>

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@xxxxxxx			      +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)

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel



[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux