All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chongyun Wu <wucy11@chinatelecom.cn>
To: "mwilck@suse.com" <mwilck@suse.com>,
	"bmarzins@redhat.com" <bmarzins@redhat.com>,
	"dm-devel@redhat.com" <dm-devel@redhat.com>
Cc: yubin1@chinatelecom.cn, zhangzijian@chinatelecom.cn
Subject: [dm-devel] [PATCH v3 resend] multipathd: fix path checkint not changed when path state changed from delay to failed
Date: Wed, 6 Jan 2021 09:39:12 +0800	[thread overview]
Message-ID: <4a94424e-5fae-0f7f-b9be-56ccb5f1a0df@chinatelecom.cn> (raw)

Thank you, Ben and Martin, this is the resend version in text format.

 From 45dad5fa6a9fb42648c8f5f54d6db974974d9612 Mon Sep 17 00:00:00 2001
From: Chongyun Wu <wucy11@chinatelecom.cn>
Date: Wed, 16 Dec 2020 13:59:16 +0800
Subject: [PATCH] multipathd: fix path checkint not changed when path 
state changed from delay to failed

Check_path: when path state change back to failed from delay state, 
should change this path's check interval time to the shortest delay to 
faster path state check.

Reviewed-by: Benjamin Marzinski <bmarzins@redhat.com>
Signed-off-by: Chongyun Wu <wucy11@chinatelecom.cn>
---
  multipathd/main.c | 5 +++++
  1 file changed, 5 insertions(+)

diff --git a/multipathd/main.c b/multipathd/main.c
index a4abbb27..9fd34e97 100644
--- a/multipathd/main.c
+++ b/multipathd/main.c
@@ -2166,6 +2166,11 @@ check_path (struct vectors * vecs, struct path * 
pp, unsigned int ticks)
  			(pp->state == PATH_DELAYED)) {
  		/* If path state become failed again cancel path delay state */
  		pp->state = newstate;
+		/*
+		 * path state bad again should change the check interval time
+		 * to the shortest delay
+		 */
+		pp->checkint = checkint;
  		return 1;
  	}
  	if (!pp->mpp) {
-- 
2.29.2.windows.3

-- 
Best Regard,
Chongyun Wu

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


             reply	other threads:[~2021-01-06  1:39 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-06  1:39 Chongyun Wu [this message]
2021-01-06 15:35 ` [dm-devel] [PATCH v3 resend] multipathd: fix path checkint not changed when path state changed from delay to failed Martin Wilck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4a94424e-5fae-0f7f-b9be-56ccb5f1a0df@chinatelecom.cn \
    --to=wucy11@chinatelecom.cn \
    --cc=bmarzins@redhat.com \
    --cc=dm-devel@redhat.com \
    --cc=mwilck@suse.com \
    --cc=yubin1@chinatelecom.cn \
    --cc=zhangzijian@chinatelecom.cn \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.