From 048c640a64fd064361382a8800de05c87ff630cb Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Mon, 10 Jun 2024 11:10:14 +0200 Subject: executor: optimize waiting for child processes exit Currently we sleep only for 1 ms, which may produce some excessive CPU load (we usually have 6/8 such processes waiting). Make it sleep for 10 ms, but also make the sleep return immediately on child exit. This shuold both improve latency and reduce CPU load. --- executor/executor_bsd.h | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'executor/executor_bsd.h') diff --git a/executor/executor_bsd.h b/executor/executor_bsd.h index 6c2174360..43c2a19a9 100644 --- a/executor/executor_bsd.h +++ b/executor/executor_bsd.h @@ -41,6 +41,11 @@ static void os_init(int argc, char** argv, void* data, size_t data_size) struct rlimit rlim; rlim.rlim_cur = rlim.rlim_max = kMaxFd; setrlimit(RLIMIT_NOFILE, &rlim); + + // A SIGCHLD handler makes sleep in loop exit immediately return with EINTR with a child exits. + struct sigaction act = {}; + act.sa_handler = [](int) {}; + sigaction(SIGCHLD, &act, nullptr); } static intptr_t execute_syscall(const call_t* c, intptr_t a[kMaxArgs]) -- cgit mrf-deployment