[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