From e2cd73a2ccabe8acae28719a0c3c1c03f2b5f9fb Mon Sep 17 00:00:00 2001 From: Florian Weimer Date: Fri, 13 May 2016 16:55:01 +0200 Subject: tst-mallocfork2: Fix race condition, use fewer resources The first SIGUSR1 signal could arrive when sigusr1_sender_pid was still 0. As a result, kill would send SIGSTOP to the entire process group. This would cause the test to hang before printing any output. This commit also adds a sched_yield to the signal source, so that it does not flood the parent process with signals it has never a chance to handle. Even with these changes, tst-mallocfork2 still fails reliably after the fix in commit commit 56290d6e762c1194547e73ff0b948cd79d3a1e03 (Increase fork signal safety for single-threaded processes) is backed out. --- malloc/tst-mallocfork2.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'malloc/tst-mallocfork2.c') diff --git a/malloc/tst-mallocfork2.c b/malloc/tst-mallocfork2.c index a9e3e94aad..4939938a44 100644 --- a/malloc/tst-mallocfork2.c +++ b/malloc/tst-mallocfork2.c @@ -25,6 +25,7 @@ still make fork unsafe, even in single-threaded processes. */ #include +#include #include #include #include @@ -70,7 +71,9 @@ sigusr1_handler (int signo) signals from the subprocess. */ if (sigusr1_received) return; - if (kill (sigusr1_sender_pid, SIGSTOP) != 0) + /* sigusr1_sender_pid might not be initialized in the parent when + the first SIGUSR1 signal arrives. */ + if (sigusr1_sender_pid > 0 && kill (sigusr1_sender_pid, SIGSTOP) != 0) { write_message ("error: kill (SIGSTOP)\n"); abort (); @@ -123,6 +126,9 @@ signal_sender (int signo, bool sleep) } if (sleep) usleep (1 * 1000 * 1000); + else + /* Reduce the rate at which we send signals. */ + sched_yield (); } } -- cgit v1.2.3-70-g09d2