From owner-cvs-all@FreeBSD.ORG Tue Nov 2 21:26:06 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 57C0216A4CE; Tue, 2 Nov 2004 21:26:06 +0000 (GMT) Received: from mail.vicor-nb.com (bigwoop.vicor-nb.com [208.206.78.2]) by mx1.FreeBSD.org (Postfix) with ESMTP id AA47F43D5C; Tue, 2 Nov 2004 21:26:05 +0000 (GMT) (envelope-from julian@elischer.org) Received: from elischer.org (julian.vicor-nb.com [208.206.78.97]) by mail.vicor-nb.com (Postfix) with ESMTP id F00737A443; Tue, 2 Nov 2004 13:26:04 -0800 (PST) Message-ID: <4187FB6C.9020702@elischer.org> Date: Tue, 02 Nov 2004 13:26:04 -0800 From: Julian Elischer User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.3.1) Gecko/20030516 X-Accept-Language: en, hu MIME-Version: 1.0 To: John Baldwin References: <200411011049.iA1AnY8m012136@repoman.freebsd.org> <4186960C.6090805@elischer.org> <4186C114.1000004@freebsd.org> <200411021549.27511.jhb@FreeBSD.org> In-Reply-To: <200411021549.27511.jhb@FreeBSD.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit cc: David Xu cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/lib/libpthread/thread thr_private.h thr_sig.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 02 Nov 2004 21:26:06 -0000 John Baldwin wrote: >On Monday 01 November 2004 06:04 pm, David Xu wrote: > > >>Not every important, I think I have another very important history >>bug in hand, did you get my "fix famous libpthread conditional >>variable race condition" mail ? :-) >> >> > >Oooo, can I test it please? We are still having problems with mono on HEAD >here at work. I tried merging the changes in uthread_cond.c 1.32 to >libpthread but that seemed to make it worse. The problems seem to be that a >signal handler is being run when the SYNCQ sflag is set (but the thread is >not on a cv or a mutex queue), and the handler calls sem_post() which is >supposed to be signal safe. sem_post() tries to lock a mutex and then bombs >with the assertion failure. > the patch is at: http://people.freebsd.org/~davidxu/kse/thr_cond.c.diff > > > >>David Xu >> >>Julian Elischer wrote: >> >> >>>shuold there be an MFC date? >>> >>>David Xu wrote: >>> >>> >>>>davidxu 2004-11-01 10:49:34 UTC >>>> >>>> FreeBSD src repository >>>> >>>> Modified files: >>>> lib/libpthread/thread thr_private.h thr_sig.c Log: >>>> Save cancelflags in signal frame, this fixes a problem that >>>> a thread in pthread_cond_wait handled a signal can no longer >>>> be canceled. >>>> >>>> Reviewed by: deischen >>>> >>>> Revision Changes Path >>>> 1.120 +1 -0 src/lib/libpthread/thread/thr_private.h >>>> 1.82 +2 -0 src/lib/libpthread/thread/thr_sig.c >>>> >>>> > > >