|
 |
3c4af5 |
From 7f3b2d1d1621cbdc60b5af4a41445391010fe9e1 Mon Sep 17 00:00:00 2001
|
|
|
3c4af5 |
From: Blazej Kucman <blazej.kucman@intel.com>
|
|
|
3c4af5 |
Date: Wed, 9 Sep 2020 10:31:20 +0200
|
|
|
3c4af5 |
Subject: [PATCH 095/108] Check if other Monitor instance running before fork.
|
|
|
3c4af5 |
|
|
|
3c4af5 |
Make error message visible to the user.
|
|
|
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 | 44 ++++++++++++++++++++++++++++----------------
|
|
|
3c4af5 |
1 file changed, 28 insertions(+), 16 deletions(-)
|
|
|
3c4af5 |
|
|
|
3c4af5 |
diff --git a/Monitor.c b/Monitor.c
|
|
|
3c4af5 |
index 0fb4f77..7fd4808 100644
|
|
|
3c4af5 |
--- a/Monitor.c
|
|
|
3c4af5 |
+++ b/Monitor.c
|
|
|
3c4af5 |
@@ -63,6 +63,7 @@ struct alert_info {
|
|
|
3c4af5 |
};
|
|
|
3c4af5 |
static int make_daemon(char *pidfile);
|
|
|
3c4af5 |
static int check_one_sharer(int scan);
|
|
|
3c4af5 |
+static void write_autorebuild_pid(void);
|
|
|
3c4af5 |
static void alert(char *event, char *dev, char *disc, struct alert_info *info);
|
|
|
3c4af5 |
static int check_array(struct state *st, struct mdstat_ent *mdstat,
|
|
|
3c4af5 |
int test, struct alert_info *info,
|
|
|
3c4af5 |
@@ -153,6 +154,11 @@ int Monitor(struct mddev_dev *devlist,
|
|
|
3c4af5 |
info.mailfrom = mailfrom;
|
|
|
3c4af5 |
info.dosyslog = dosyslog;
|
|
|
3c4af5 |
|
|
|
3c4af5 |
+ if (share){
|
|
|
3c4af5 |
+ if (check_one_sharer(c->scan))
|
|
|
3c4af5 |
+ return 1;
|
|
|
3c4af5 |
+ }
|
|
|
3c4af5 |
+
|
|
|
3c4af5 |
if (daemonise) {
|
|
|
3c4af5 |
int rv = make_daemon(pidfile);
|
|
|
3c4af5 |
if (rv >= 0)
|
|
|
3c4af5 |
@@ -160,8 +166,7 @@ int Monitor(struct mddev_dev *devlist,
|
|
|
3c4af5 |
}
|
|
|
3c4af5 |
|
|
|
3c4af5 |
if (share)
|
|
|
3c4af5 |
- if (check_one_sharer(c->scan))
|
|
|
3c4af5 |
- return 1;
|
|
|
3c4af5 |
+ write_autorebuild_pid();
|
|
|
3c4af5 |
|
|
|
3c4af5 |
if (devlist == NULL) {
|
|
|
3c4af5 |
mdlist = conf_get_ident(NULL);
|
|
|
3c4af5 |
@@ -328,8 +333,8 @@ static int check_one_sharer(int scan)
|
|
|
3c4af5 |
int pid;
|
|
|
3c4af5 |
FILE *comm_fp;
|
|
|
3c4af5 |
FILE *fp;
|
|
|
3c4af5 |
- char comm_path[100];
|
|
|
3c4af5 |
- char path[100];
|
|
|
3c4af5 |
+ char comm_path[PATH_MAX];
|
|
|
3c4af5 |
+ char path[PATH_MAX];
|
|
|
3c4af5 |
char comm[20];
|
|
|
3c4af5 |
|
|
|
3c4af5 |
sprintf(path, "%s/autorebuild.pid", MDMON_DIR);
|
|
|
3c4af5 |
@@ -356,21 +361,28 @@ static int check_one_sharer(int scan)
|
|
|
3c4af5 |
}
|
|
|
3c4af5 |
fclose(fp);
|
|
|
3c4af5 |
}
|
|
|
3c4af5 |
- if (scan) {
|
|
|
3c4af5 |
- if (mkdir(MDMON_DIR, S_IRWXU) < 0 && errno != EEXIST) {
|
|
|
3c4af5 |
+ return 0;
|
|
|
3c4af5 |
+}
|
|
|
3c4af5 |
+
|
|
|
3c4af5 |
+static void write_autorebuild_pid()
|
|
|
3c4af5 |
+{
|
|
|
3c4af5 |
+ char path[PATH_MAX];
|
|
|
3c4af5 |
+ int pid;
|
|
|
3c4af5 |
+ FILE *fp;
|
|
|
3c4af5 |
+ sprintf(path, "%s/autorebuild.pid", MDMON_DIR);
|
|
|
3c4af5 |
+
|
|
|
3c4af5 |
+ if (mkdir(MDMON_DIR, S_IRWXU) < 0 && errno != EEXIST) {
|
|
|
3c4af5 |
+ pr_err("Can't create autorebuild.pid file\n");
|
|
|
3c4af5 |
+ } else {
|
|
|
3c4af5 |
+ fp = fopen(path, "w");
|
|
|
3c4af5 |
+ if (!fp)
|
|
|
3c4af5 |
pr_err("Can't create autorebuild.pid file\n");
|
|
|
3c4af5 |
- } else {
|
|
|
3c4af5 |
- fp = fopen(path, "w");
|
|
|
3c4af5 |
- if (!fp)
|
|
|
3c4af5 |
- pr_err("Cannot create autorebuild.pidfile\n");
|
|
|
3c4af5 |
- else {
|
|
|
3c4af5 |
- pid = getpid();
|
|
|
3c4af5 |
- fprintf(fp, "%d\n", pid);
|
|
|
3c4af5 |
- fclose(fp);
|
|
|
3c4af5 |
- }
|
|
|
3c4af5 |
+ else {
|
|
|
3c4af5 |
+ pid = getpid();
|
|
|
3c4af5 |
+ fprintf(fp, "%d\n", pid);
|
|
|
3c4af5 |
+ fclose(fp);
|
|
|
3c4af5 |
}
|
|
|
3c4af5 |
}
|
|
|
3c4af5 |
- return 0;
|
|
|
3c4af5 |
}
|
|
|
3c4af5 |
|
|
|
3c4af5 |
static void alert(char *event, char *dev, char *disc, struct alert_info *info)
|
|
|
3c4af5 |
--
|
|
|
3c4af5 |
2.7.5
|
|
|
3c4af5 |
|