* [PATCH] Monitor: remove autorebuild pidfile when it exits
@ 2020-03-27 1:28 Lidong Zhong
2020-04-27 14:40 ` Jes Sorensen
0 siblings, 1 reply; 5+ messages in thread
From: Lidong Zhong @ 2020-03-27 1:28 UTC (permalink / raw)
To: jes; +Cc: linux-raid, Lidong Zhong
mdadm monitor is supposed to run in background and never exit. However
if it is killed by accident and the pid stored in autorebuild.pid is
taken by some other process, it reports an error when restarting
"mdadm: Only one autorebuild process allowed in scan mode, aborting"
even though no autorebuild process exists. With the patch, this file
will be removed at the end to fix this scenario.
Signed-off-by: Lidong Zhong <lidong.zhong@suse.com>
---
Hi Jes,
Sorry to ping you, but could you please share your opinion about this patch? TIA.
---
Monitor.c | 39 ++++++++++++++++++++++++++++++++++++++-
1 file changed, 38 insertions(+), 1 deletion(-)
diff --git a/Monitor.c b/Monitor.c
index b527165..4cb113d 100644
--- a/Monitor.c
+++ b/Monitor.c
@@ -62,6 +62,7 @@ struct alert_info {
int dosyslog;
};
static int make_daemon(char *pidfile);
+static int cleanup_sharer_pidfile();
static int check_one_sharer(int scan);
static void alert(char *event, char *dev, char *disc, struct alert_info *info);
static int check_array(struct state *st, struct mdstat_ent *mdstat,
@@ -71,6 +72,12 @@ static int add_new_arrays(struct mdstat_ent *mdstat, struct state **statelist,
int test, struct alert_info *info);
static void try_spare_migration(struct state *statelist, struct alert_info *info);
static void link_containers_with_subarrays(struct state *list);
+static volatile sig_atomic_t finished = 0;
+
+static void _exit_handler(int sig)
+{
+ finished= 1;
+}
int Monitor(struct mddev_dev *devlist,
char *mailaddr, char *alert_cmd,
@@ -123,7 +130,6 @@ int Monitor(struct mddev_dev *devlist,
struct state *statelist = NULL;
struct state *st2;
- int finished = 0;
struct mdstat_ent *mdstat = NULL;
char *mailfrom;
struct alert_info info;
@@ -157,6 +163,11 @@ int Monitor(struct mddev_dev *devlist,
if (rv >= 0)
return rv;
}
+
+ signal(SIGTERM, &_exit_handler);
+ signal(SIGKILL, &_exit_handler);
+ signal(SIGINT, &_exit_handler);
+ signal(SIGQUIT, &_exit_handler);
if (share)
if (check_one_sharer(c->scan))
@@ -260,6 +271,8 @@ int Monitor(struct mddev_dev *devlist,
if (pidfile)
unlink(pidfile);
+ if (cleanup_sharer_pidfile())
+ pr_err("Cannot remove autorebuild.pidfile\n");
return 0;
}
@@ -299,6 +312,30 @@ static int make_daemon(char *pidfile)
return -1;
}
+static int cleanup_sharer_pidfile()
+{
+ int pid, rv;
+ FILE *fp;
+ char path[100];
+ char dir[20];
+ struct stat buf;
+ sprintf(path, "%s/autorebuild.pid", MDMON_DIR);
+ fp = fopen(path, "r");
+ if (fp) {
+ if (fscanf(fp, "%d", &pid) != 1)
+ pid = -1;
+ sprintf(dir, "/proc/%d", pid);
+ rv = stat(dir, &buf);
+ if (rv != -1) {
+ fclose(fp);
+ unlink(path);
+ } else
+ return 1;
+ }
+
+ return 0;
+}
+
static int check_one_sharer(int scan)
{
int pid, rv;
--
2.16.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] Monitor: remove autorebuild pidfile when it exits
2020-03-27 1:28 [PATCH] Monitor: remove autorebuild pidfile when it exits Lidong Zhong
@ 2020-04-27 14:40 ` Jes Sorensen
2020-04-27 16:19 ` Zhong Lidong
0 siblings, 1 reply; 5+ messages in thread
From: Jes Sorensen @ 2020-04-27 14:40 UTC (permalink / raw)
To: Lidong Zhong; +Cc: linux-raid
On 3/26/20 9:28 PM, Lidong Zhong wrote:
> mdadm monitor is supposed to run in background and never exit. However
> if it is killed by accident and the pid stored in autorebuild.pid is
> taken by some other process, it reports an error when restarting
> "mdadm: Only one autorebuild process allowed in scan mode, aborting"
> even though no autorebuild process exists. With the patch, this file
> will be removed at the end to fix this scenario.
>
> Signed-off-by: Lidong Zhong <lidong.zhong@suse.com>
> ---
> Hi Jes,
>
> Sorry to ping you, but could you please share your opinion about this patch? TIA.
> ---
> Monitor.c | 39 ++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 38 insertions(+), 1 deletion(-)
>
> diff --git a/Monitor.c b/Monitor.c
> index b527165..4cb113d 100644
> --- a/Monitor.c
> +++ b/Monitor.c
> @@ -62,6 +62,7 @@ struct alert_info {
> int dosyslog;
> };
> static int make_daemon(char *pidfile);
> +static int cleanup_sharer_pidfile();
> static int check_one_sharer(int scan);
> static void alert(char *event, char *dev, char *disc, struct alert_info *info);
> static int check_array(struct state *st, struct mdstat_ent *mdstat,
> @@ -71,6 +72,12 @@ static int add_new_arrays(struct mdstat_ent *mdstat, struct state **statelist,
> int test, struct alert_info *info);
> static void try_spare_migration(struct state *statelist, struct alert_info *info);
> static void link_containers_with_subarrays(struct state *list);
> +static volatile sig_atomic_t finished = 0;
> +
> +static void _exit_handler(int sig)
> +{
> + finished= 1;
Please respect formatting rules.
Also this patch fails to apply cleanly:
Applying: Monitor: remove autorebuild pidfile when it exits
Using index info to reconstruct a base tree...
M Monitor.c
.git/rebase-apply/patch:27: trailing whitespace.
static void _exit_handler(int sig)
.git/rebase-apply/patch:46: trailing whitespace.
.git/rebase-apply/patch:47: trailing whitespace.
signal(SIGTERM, &_exit_handler);
.git/rebase-apply/patch:48: trailing whitespace.
signal(SIGKILL, &_exit_handler);
warning: 4 lines add whitespace errors.
Falling back to patching base and 3-way merge...
Auto-merging Monitor.c
Please fix and resubmit a v2.
Jes
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] Monitor: remove autorebuild pidfile when it exits
2020-04-27 14:40 ` Jes Sorensen
@ 2020-04-27 16:19 ` Zhong Lidong
0 siblings, 0 replies; 5+ messages in thread
From: Zhong Lidong @ 2020-04-27 16:19 UTC (permalink / raw)
To: Jes Sorensen; +Cc: linux-raid
On 4/27/20 10:40 PM, Jes Sorensen wrote:
> On 3/26/20 9:28 PM, Lidong Zhong wrote:
>> mdadm monitor is supposed to run in background and never exit. However
>> if it is killed by accident and the pid stored in autorebuild.pid is
>> taken by some other process, it reports an error when restarting
>> "mdadm: Only one autorebuild process allowed in scan mode, aborting"
>> even though no autorebuild process exists. With the patch, this file
>> will be removed at the end to fix this scenario.
>>
>> Signed-off-by: Lidong Zhong <lidong.zhong@suse.com>
>> ---
>> Hi Jes,
>>
>> Sorry to ping you, but could you please share your opinion about this patch? TIA.
>> ---
>> Monitor.c | 39 ++++++++++++++++++++++++++++++++++++++-
>> 1 file changed, 38 insertions(+), 1 deletion(-)
>>
>> diff --git a/Monitor.c b/Monitor.c
>> index b527165..4cb113d 100644
>> --- a/Monitor.c
>> +++ b/Monitor.c
>> @@ -62,6 +62,7 @@ struct alert_info {
>> int dosyslog;
>> };
>> static int make_daemon(char *pidfile);
>> +static int cleanup_sharer_pidfile();
>> static int check_one_sharer(int scan);
>> static void alert(char *event, char *dev, char *disc, struct alert_info *info);
>> static int check_array(struct state *st, struct mdstat_ent *mdstat,
>> @@ -71,6 +72,12 @@ static int add_new_arrays(struct mdstat_ent *mdstat, struct state **statelist,
>> int test, struct alert_info *info);
>> static void try_spare_migration(struct state *statelist, struct alert_info *info);
>> static void link_containers_with_subarrays(struct state *list);
>> +static volatile sig_atomic_t finished = 0;
>> +
>> +static void _exit_handler(int sig)
>> +{
>> + finished= 1;
>
> Please respect formatting rules.
>
> Also this patch fails to apply cleanly:
>
> Applying: Monitor: remove autorebuild pidfile when it exits
> Using index info to reconstruct a base tree...
> M Monitor.c
> .git/rebase-apply/patch:27: trailing whitespace.
> static void _exit_handler(int sig)
>
> .git/rebase-apply/patch:46: trailing whitespace.
>
> .git/rebase-apply/patch:47: trailing whitespace.
> signal(SIGTERM, &_exit_handler);
> .git/rebase-apply/patch:48: trailing whitespace.
> signal(SIGKILL, &_exit_handler);
> warning: 4 lines add whitespace errors.
> Falling back to patching base and 3-way merge...
> Auto-merging Monitor.c
>
> Please fix and resubmit a v2.
>
Hi Jes,
Thanks for your reply. Please ignore this one since Coly's patch
Monitor: improve check_one_sharer() for checking duplicated process
is already applied. They are aimed to fix the same issue.
Regards,
Lidong
> Jes
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] Monitor: remove autorebuild pidfile when it exits
@ 2020-03-09 3:24 Lidong Zhong
0 siblings, 0 replies; 5+ messages in thread
From: Lidong Zhong @ 2020-03-09 3:24 UTC (permalink / raw)
To: jsorensen; +Cc: linux-raid
Though the mdadm monitor is supposed to run in background and never
exit, user may want to restart it and kill the daemon forcefully. If
the pid stored in autorebuild.pid is taken by some other process, it
reports that
"mdadm: Only one autorebuild process allowed in scan mode, aborting"
even though no autorebuild process exists. With this patch this file
will be removed at the end to fix this error.
Signed-off-by: Lidong Zhong <lidong.zhong@suse.com>
---
Hi Jes,
could you please share your opinion about this patch? TIA.
---
Monitor.c | 39 ++++++++++++++++++++++++++++++++++++++-
1 file changed, 38 insertions(+), 1 deletion(-)
diff --git a/Monitor.c b/Monitor.c
index b527165..4cb113d 100644
--- a/Monitor.c
+++ b/Monitor.c
@@ -62,6 +62,7 @@ struct alert_info {
int dosyslog;
};
static int make_daemon(char *pidfile);
+static int cleanup_sharer_pidfile();
static int check_one_sharer(int scan);
static void alert(char *event, char *dev, char *disc, struct alert_info *info);
static int check_array(struct state *st, struct mdstat_ent *mdstat,
@@ -71,6 +72,12 @@ static int add_new_arrays(struct mdstat_ent *mdstat, struct state **statelist,
int test, struct alert_info *info);
static void try_spare_migration(struct state *statelist, struct alert_info *info);
static void link_containers_with_subarrays(struct state *list);
+static volatile sig_atomic_t finished = 0;
+
+static void _exit_handler(int sig)
+{
+ finished= 1;
+}
int Monitor(struct mddev_dev *devlist,
char *mailaddr, char *alert_cmd,
@@ -123,7 +130,6 @@ int Monitor(struct mddev_dev *devlist,
struct state *statelist = NULL;
struct state *st2;
- int finished = 0;
struct mdstat_ent *mdstat = NULL;
char *mailfrom;
struct alert_info info;
@@ -157,6 +163,11 @@ int Monitor(struct mddev_dev *devlist,
if (rv >= 0)
return rv;
}
+
+ signal(SIGTERM, &_exit_handler);
+ signal(SIGKILL, &_exit_handler);
+ signal(SIGINT, &_exit_handler);
+ signal(SIGQUIT, &_exit_handler);
if (share)
if (check_one_sharer(c->scan))
@@ -260,6 +271,8 @@ int Monitor(struct mddev_dev *devlist,
if (pidfile)
unlink(pidfile);
+ if (cleanup_sharer_pidfile())
+ pr_err("Cannot remove autorebuild.pidfile\n");
return 0;
}
@@ -299,6 +312,30 @@ static int make_daemon(char *pidfile)
return -1;
}
+static int cleanup_sharer_pidfile()
+{
+ int pid, rv;
+ FILE *fp;
+ char path[100];
+ char dir[20];
+ struct stat buf;
+ sprintf(path, "%s/autorebuild.pid", MDMON_DIR);
+ fp = fopen(path, "r");
+ if (fp) {
+ if (fscanf(fp, "%d", &pid) != 1)
+ pid = -1;
+ sprintf(dir, "/proc/%d", pid);
+ rv = stat(dir, &buf);
+ if (rv != -1) {
+ fclose(fp);
+ unlink(path);
+ } else
+ return 1;
+ }
+
+ return 0;
+}
+
static int check_one_sharer(int scan)
{
int pid, rv;
--
2.16.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH] Monitor: remove autorebuild pidfile when it exits
@ 2020-02-29 0:11 Lidong Zhong
0 siblings, 0 replies; 5+ messages in thread
From: Lidong Zhong @ 2020-02-29 0:11 UTC (permalink / raw)
To: linux-raid
Though the mdadm monitor is supposed to run in background and never
exit, user may want to restart it and kill the daemon forcefully. If
the pid stored in autorebuild.pid is taken by some other process, it
reports that
"mdadm: Only one autorebuild process allowed in scan mode, aborting"
even though no autorebuild process exists. With this patch this file
will be removed at the end to fix this error.
Signed-off-by: Lidong Zhong <lidong.zhong@suse.com>
---
Monitor.c | 39 ++++++++++++++++++++++++++++++++++++++-
1 file changed, 38 insertions(+), 1 deletion(-)
diff --git a/Monitor.c b/Monitor.c
index b527165..4cb113d 100644
--- a/Monitor.c
+++ b/Monitor.c
@@ -62,6 +62,7 @@ struct alert_info {
int dosyslog;
};
static int make_daemon(char *pidfile);
+static int cleanup_sharer_pidfile();
static int check_one_sharer(int scan);
static void alert(char *event, char *dev, char *disc, struct alert_info *info);
static int check_array(struct state *st, struct mdstat_ent *mdstat,
@@ -71,6 +72,12 @@ static int add_new_arrays(struct mdstat_ent *mdstat, struct state **statelist,
int test, struct alert_info *info);
static void try_spare_migration(struct state *statelist, struct alert_info *info);
static void link_containers_with_subarrays(struct state *list);
+static volatile sig_atomic_t finished = 0;
+
+static void _exit_handler(int sig)
+{
+ finished= 1;
+}
int Monitor(struct mddev_dev *devlist,
char *mailaddr, char *alert_cmd,
@@ -123,7 +130,6 @@ int Monitor(struct mddev_dev *devlist,
struct state *statelist = NULL;
struct state *st2;
- int finished = 0;
struct mdstat_ent *mdstat = NULL;
char *mailfrom;
struct alert_info info;
@@ -157,6 +163,11 @@ int Monitor(struct mddev_dev *devlist,
if (rv >= 0)
return rv;
}
+
+ signal(SIGTERM, &_exit_handler);
+ signal(SIGKILL, &_exit_handler);
+ signal(SIGINT, &_exit_handler);
+ signal(SIGQUIT, &_exit_handler);
if (share)
if (check_one_sharer(c->scan))
@@ -260,6 +271,8 @@ int Monitor(struct mddev_dev *devlist,
if (pidfile)
unlink(pidfile);
+ if (cleanup_sharer_pidfile())
+ pr_err("Cannot remove autorebuild.pidfile\n");
return 0;
}
@@ -299,6 +312,30 @@ static int make_daemon(char *pidfile)
return -1;
}
+static int cleanup_sharer_pidfile()
+{
+ int pid, rv;
+ FILE *fp;
+ char path[100];
+ char dir[20];
+ struct stat buf;
+ sprintf(path, "%s/autorebuild.pid", MDMON_DIR);
+ fp = fopen(path, "r");
+ if (fp) {
+ if (fscanf(fp, "%d", &pid) != 1)
+ pid = -1;
+ sprintf(dir, "/proc/%d", pid);
+ rv = stat(dir, &buf);
+ if (rv != -1) {
+ fclose(fp);
+ unlink(path);
+ } else
+ return 1;
+ }
+
+ return 0;
+}
+
static int check_one_sharer(int scan)
{
int pid, rv;
--
2.16.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-04-27 16:19 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-03-27 1:28 [PATCH] Monitor: remove autorebuild pidfile when it exits Lidong Zhong
2020-04-27 14:40 ` Jes Sorensen
2020-04-27 16:19 ` Zhong Lidong
-- strict thread matches above, loose matches on Subject: below --
2020-03-09 3:24 Lidong Zhong
2020-02-29 0:11 Lidong Zhong
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).