diff options
author | Coly Li <colyli@suse.de> | 2021-09-02 09:32:20 +0200 |
---|---|---|
committer | Jes Sorensen <jsorensen@fb.com> | 2021-12-08 15:44:14 +0100 |
commit | 8c80d3055bcaab4c449abb7b666aad69d054347a (patch) | |
tree | 7257906c599a051e9288c6c1fa2c71d956011ca6 | |
parent | mdadm: block creation with long names (diff) | |
download | mdadm-8c80d3055bcaab4c449abb7b666aad69d054347a.tar.xz mdadm-8c80d3055bcaab4c449abb7b666aad69d054347a.zip |
Monitor: print message before quit for no array to monitor
If there is no array device to monitor, Monitor() will stop monitoring
at line 261 from the following code block,
257 if (!new_found) {
258 if (oneshot)
259 break;
260 else if (!anyredundant) {
261 break;
262 }
This change was introduced by commit 007087d0898a ("Monitor: stop
notifing about containers"). Before this commit, Monitor() will continue
and won't quit even there is no array to monitor.
It is fine to quit without any array device to monitor, but users may
wonder whether there is something wrong with mdadm program or their
configuration to make mdadm quit monitoring.
This patch adds a simple error message to indicate Monitor() quits for
array device to monitor, which makes users have hint to understand why
mdadm stops monitoring.
Reported-by: George Gkioulis <ggkioulis@suse.com>
Suggested-by: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>
Signed-off-by: Coly Li <colyli@suse.de>
Cc: Jes Sorensen <jsorensen@fb.com>
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
-rw-r--r-- | Monitor.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -258,6 +258,7 @@ int Monitor(struct mddev_dev *devlist, if (oneshot) break; else if (!anyredundant) { + pr_err("No array with redundancy detected, stopping\n"); break; } else { |