dcavalca / rpms / mdadm

Forked from rpms/mdadm 3 years ago
Clone

Blame SOURCES/0094-mdmonitor-set-small-delay-once.patch

3c4af5
From cab9c67d461c65a1138359f9f6d39636466b90e4 Mon Sep 17 00:00:00 2001
3c4af5
From: Blazej Kucman <blazej.kucman@intel.com>
3c4af5
Date: Wed, 9 Sep 2020 10:31:19 +0200
3c4af5
Subject: [PATCH 094/108] mdmonitor: set small delay once
3c4af5
3c4af5
If mdmonitor is awakened by event, set small delay once
3c4af5
to deal with udev and mdadm.
3c4af5
3c4af5
Signed-off-by: Blazej Kucman <blazej.kucman@intel.com>
3c4af5
Signed-off-by: Mariusz Tkaczyk <mariusz.tkaczyk@intel.com>
3c4af5
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
3c4af5
---
3c4af5
 Monitor.c | 14 +++++++++++++-
3c4af5
 mdadm.h   |  2 +-
3c4af5
 mdstat.c  | 18 +++++++++++++++---
3c4af5
 3 files changed, 29 insertions(+), 5 deletions(-)
3c4af5
3c4af5
diff --git a/Monitor.c b/Monitor.c
3c4af5
index aed7a69..0fb4f77 100644
3c4af5
--- a/Monitor.c
3c4af5
+++ b/Monitor.c
3c4af5
@@ -128,6 +128,7 @@ int Monitor(struct mddev_dev *devlist,
3c4af5
 	char *mailfrom;
3c4af5
 	struct alert_info info;
3c4af5
 	struct mddev_ident *mdlist;
3c4af5
+	int delay_for_event = c->delay;
3c4af5
 
3c4af5
 	if (!mailaddr) {
3c4af5
 		mailaddr = conf_get_mailaddr();
3c4af5
@@ -249,7 +250,18 @@ int Monitor(struct mddev_dev *devlist,
3c4af5
 				break;
3c4af5
 			}
3c4af5
 			else {
3c4af5
-				mdstat_wait(c->delay);
3c4af5
+				int wait_result = mdstat_wait(delay_for_event);
3c4af5
+
3c4af5
+				/*
3c4af5
+				 * If mdmonitor is awaken by event, set small delay once
3c4af5
+				 * to deal with udev and mdadm.
3c4af5
+				 */
3c4af5
+				if (wait_result != 0) {
3c4af5
+					if (c->delay > 5)
3c4af5
+						delay_for_event = 5;
3c4af5
+				} else
3c4af5
+					delay_for_event = c->delay;
3c4af5
+
3c4af5
 				mdstat_close();
3c4af5
 			}
3c4af5
 		}
3c4af5
diff --git a/mdadm.h b/mdadm.h
3c4af5
index 399478b..4961c0f 100644
3c4af5
--- a/mdadm.h
3c4af5
+++ b/mdadm.h
3c4af5
@@ -628,7 +628,7 @@ struct mdstat_ent {
3c4af5
 extern struct mdstat_ent *mdstat_read(int hold, int start);
3c4af5
 extern void mdstat_close(void);
3c4af5
 extern void free_mdstat(struct mdstat_ent *ms);
3c4af5
-extern void mdstat_wait(int seconds);
3c4af5
+extern int mdstat_wait(int seconds);
3c4af5
 extern void mdstat_wait_fd(int fd, const sigset_t *sigmask);
3c4af5
 extern int mddev_busy(char *devnm);
3c4af5
 extern struct mdstat_ent *mdstat_by_component(char *name);
3c4af5
diff --git a/mdstat.c b/mdstat.c
3c4af5
index 48559e6..dd96cca 100644
3c4af5
--- a/mdstat.c
3c4af5
+++ b/mdstat.c
3c4af5
@@ -302,7 +302,17 @@ void mdstat_close(void)
3c4af5
 	mdstat_fd = -1;
3c4af5
 }
3c4af5
 
3c4af5
-void mdstat_wait(int seconds)
3c4af5
+/*
3c4af5
+ * function: mdstat_wait
3c4af5
+ * Description: Function waits for event on mdstat.
3c4af5
+ * Parameters:
3c4af5
+ *		seconds - timeout for waiting
3c4af5
+ * Returns:
3c4af5
+ *		> 0 - detected event
3c4af5
+ *		0 - timeout
3c4af5
+ *		< 0 - detected error
3c4af5
+ */
3c4af5
+int mdstat_wait(int seconds)
3c4af5
 {
3c4af5
 	fd_set fds;
3c4af5
 	struct timeval tm;
3c4af5
@@ -312,10 +322,12 @@ void mdstat_wait(int seconds)
3c4af5
 		FD_SET(mdstat_fd, &fds);
3c4af5
 		maxfd = mdstat_fd;
3c4af5
 	} else
3c4af5
-		return;
3c4af5
+		return -1;
3c4af5
+
3c4af5
 	tm.tv_sec = seconds;
3c4af5
 	tm.tv_usec = 0;
3c4af5
-	select(maxfd + 1, NULL, NULL, &fds, &tm;;
3c4af5
+
3c4af5
+	return select(maxfd + 1, NULL, NULL, &fds, &tm;;
3c4af5
 }
3c4af5
 
3c4af5
 void mdstat_wait_fd(int fd, const sigset_t *sigmask)
3c4af5
-- 
3c4af5
2.7.5
3c4af5