From: Hannes Reinecke Subject: [PATCH 28/29] multipath: fix setting of fast_io_fail_tmo Date: Mon, 15 Jul 2013 15:00:29 +0200 Message-ID: <1373893230-26077-29-git-send-email-hare@suse.de> References: <1373893230-26077-1-git-send-email-hare@suse.de> Reply-To: device-mapper development Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1373893230-26077-1-git-send-email-hare@suse.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Christophe Varoqui Cc: dm-devel@redhat.com, Petr Uzel List-Id: dm-devel.ids From: Petr Uzel Setting fast_io_fail_tmo to the same value as dev_loss_tmo is not allowed by the kernel. Increase dev_loss_tmo by 1 in such cases to make it strictly greated than fast_io_fail_tmo. Signed-off-by: Petr Uzel --- libmultipath/discovery.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c index adfb1ba..e2b4f56 100644 --- a/libmultipath/discovery.c +++ b/libmultipath/discovery.c @@ -367,7 +367,7 @@ sysfs_set_rport_tmo(struct multipath *mpp, struct path *pp) goto out; } if (mpp->fast_io_fail >= tmo) { - snprintf(value, 16, "%u", mpp->fast_io_fail); + snprintf(value, 16, "%u", mpp->fast_io_fail + 1); } } else if (mpp->dev_loss > 600) { condlog(3, "%s: limiting dev_loss_tmo to 600, since " -- 1.7.10.4