From mboxrd@z Thu Jan 1 00:00:00 1970 From: mwilck@suse.com Subject: [PATCH v2 02/21] multipathd: avoid sending "READY=1" to systemd on early shutdown Date: Thu, 24 Sep 2020 15:36:57 +0200 Message-ID: <20200924133716.14120-3-mwilck@suse.com> References: <20200924133716.14120-1-mwilck@suse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20200924133716.14120-1-mwilck@suse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Christophe Varoqui , Benjamin Marzinski Cc: dm-devel@redhat.com, Martin Wilck List-Id: dm-devel.ids From: Martin Wilck If multipathd gets a shutdown request during initial reconfigure(), it shouldn't send "READY=1" to systemd. Ensure this by sending "READY=1" via post_config_state(). Reviewed-by: Benjamin Marzinski Signed-off-by: Martin Wilck --- multipathd/main.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/multipathd/main.c b/multipathd/main.c index fa53e96..53a22a4 100644 --- a/multipathd/main.c +++ b/multipathd/main.c @@ -189,6 +189,8 @@ static void do_sd_notify(enum daemon_status old_state, { char notify_msg[MSG_SIZE]; const char *msg; + static bool startup_done = false; + /* * Checkerloop switches back and forth between idle and running state. * No need to tell systemd each time. @@ -205,6 +207,11 @@ static void do_sd_notify(enum daemon_status old_state, if (msg && !safe_sprintf(notify_msg, "STATUS=%s", msg)) sd_notify(0, notify_msg); + + if (new_state == DAEMON_IDLE && !startup_done) { + sd_notify(0, "READY=1"); + startup_done = true; + } } #endif @@ -2903,9 +2910,6 @@ child (__attribute__((unused)) void *param) struct vectors * vecs; struct multipath * mpp; int i; -#ifdef USE_SYSTEMD - int startup_done = 0; -#endif int rc; int pid_fd = -1; struct config *conf; @@ -3065,12 +3069,6 @@ child (__attribute__((unused)) void *param) } lock_cleanup_pop(vecs->lock); post_config_state(DAEMON_IDLE); -#ifdef USE_SYSTEMD - if (!startup_done) { - sd_notify(0, "READY=1"); - startup_done = 1; - } -#endif } } -- 2.28.0