[Tarantool-patches] [PATCH 02/12] popen: add ability to keep child on deletion
Alexander Turenko
alexander.turenko at tarantool.org
Tue Apr 14 14:38:11 MSK 2020
From: Cyrill Gorcunov <gorcunov at gmail.com>
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.
Part of #4031
Reported-by: Alexander Turenko <alexander.turenko at tarantool.org>
Acked-by: Alexander Turenko <alexander.turenko at tarantool.org>
Signed-off-by: Cyrill Gorcunov <gorcunov at gmail.com>
---
src/lib/core/popen.c | 15 +++++++++------
src/lib/core/popen.h | 6 ++++++
2 files changed, 15 insertions(+), 6 deletions(-)
diff --git a/src/lib/core/popen.c b/src/lib/core/popen.c
index a5a305013..30be74a5f 100644
--- a/src/lib/core/popen.c
+++ b/src/lib/core/popen.c
@@ -585,12 +585,15 @@ popen_delete(struct popen_handle *handle)
assert(handle != NULL);
- /*
- * Unable to kill the process -- give an error.
- * The process is not exist already -- pass over.
- */
- if (popen_send_signal(handle, SIGKILL) != 0 && errno != ESRCH)
- return -1;
+ if ((handle->flags & POPEN_FLAG_KEEP_CHILD) == 0) {
+ /*
+ * Unable to kill the process -- give an error.
+ * The process is not exist already -- pass over.
+ */
+ if (popen_send_signal(handle, SIGKILL) != 0 &&
+ 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 623d826b9..570376d33 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 running on delete.
+ */
+ POPEN_FLAG_KEEP_CHILD_BIT = 17,
+ POPEN_FLAG_KEEP_CHILD = (1 << POPEN_FLAG_KEEP_CHILD_BIT),
};
/**
--
2.25.0
More information about the Tarantool-patches
mailing list