From d8cdfda31e549353cad281ac112fcde6db6e521b Mon Sep 17 00:00:00 2001 From: Jakub Jelinek Date: Thu, 12 Jul 2007 15:40:54 +0000 Subject: [PATCH] 2007-02-25 Ulrich Drepper * sysdeps/unix/sysv/linux/fork.c (__libc_fork): Reset refcntr in new thread, don't just decrement it. Patch by Suzuki K P . --- nptl/ChangeLog | 6 ++++++ nptl/sysdeps/unix/sysv/linux/fork.c | 7 +++++-- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/nptl/ChangeLog b/nptl/ChangeLog index 6fa76ca4e57..a3ba15dc8d1 100644 --- a/nptl/ChangeLog +++ b/nptl/ChangeLog @@ -186,6 +186,12 @@ * tst-cancel-wrappers.sh: Set C["fdatasync"] to 1. * tst-cancel4.c (tf_fdatasync): New test. +2007-02-25 Ulrich Drepper + + * sysdeps/unix/sysv/linux/fork.c (__libc_fork): Reset refcntr in + new thread, don't just decrement it. + Patch by Suzuki K P . + 2007-02-09 Jakub Jelinek * sysdeps/unix/sysv/linux/x86_64/lowlevellock.S diff --git a/nptl/sysdeps/unix/sysv/linux/fork.c b/nptl/sysdeps/unix/sysv/linux/fork.c index 840974401bc..f6691e13446 100644 --- a/nptl/sysdeps/unix/sysv/linux/fork.c +++ b/nptl/sysdeps/unix/sysv/linux/fork.c @@ -167,8 +167,11 @@ __libc_fork (void) allp->handler->child_handler (); /* Note that we do not have to wake any possible waiter. - This is the only thread in the new process. */ - --allp->handler->refcntr; + This is the only thread in the new process. The count + may have been bumped up by other threads doing a fork. + We reset it to 1, to avoid waiting for non-existing + thread(s) to release the count. */ + allp->handler->refcntr = 1; /* XXX We could at this point look through the object pool and mark all objects not on the __fork_handlers list as -- 2.47.2