[Tarantool-patches] [PATCH 2/2] popen: add ability to keep child on deletion

Cyrill Gorcunov gorcunov at gmail.com
Fri Apr 10 17:40:21 MSK 2020


Currently popen_delete kills all children process.
Moreover we use popen_delete on tarantool exit.

Alexander pointed out that keep children running
even if tarantool is exited is still needed.

Reported-by: Alexander Turenko <alexander.turenko at tarantool.org>
Signed-off-by: Cyrill Gorcunov <gorcunov at gmail.com>
---
 src/lib/core/popen.c | 5 ++++-
 src/lib/core/popen.h | 6 ++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/lib/core/popen.c b/src/lib/core/popen.c
index 200b31b21..50b40b3a4 100644
--- a/src/lib/core/popen.c
+++ b/src/lib/core/popen.c
@@ -489,8 +489,11 @@ popen_delete(struct popen_handle *handle)
 		return -1;
 	}
 
-	if (popen_send_signal(handle, SIGKILL) && errno != ESRCH)
+	if ((handle->flags & POPEN_FLAG_KEEP_CHILD) == 0) {
+		if (popen_send_signal(handle, SIGKILL) &&
+		    errno != ESRCH)
 		return -1;
+	}
 
 	for (i = 0; i < lengthof(handle->ios); i++) {
 		if (handle->ios[i].fd != -1)
diff --git a/src/lib/core/popen.h b/src/lib/core/popen.h
index f9dd0ff45..2e87e11ff 100644
--- a/src/lib/core/popen.h
+++ b/src/lib/core/popen.h
@@ -102,6 +102,12 @@ enum popen_flag_bits {
 	 */
 	POPEN_FLAG_GROUP_SIGNAL_BIT	= 16,
 	POPEN_FLAG_GROUP_SIGNAL		= (1 << POPEN_FLAG_GROUP_SIGNAL_BIT),
+
+	/*
+	 * Keep child rinning on delete.
+	 */
+	POPEN_FLAG_KEEP_CHILD_BIT	= 17,
+	POPEN_FLAG_KEEP_CHILD		= (1 << POPEN_FLAG_KEEP_CHILD_BIT),
 };
 
 /**
-- 
2.20.1



More information about the Tarantool-patches mailing list