From owner-cvs-src@FreeBSD.ORG Sun Oct 31 15:40:48 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from green.homeunix.org (freefall.freebsd.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 2CBA416A4CE; Sun, 31 Oct 2004 15:40:48 +0000 (GMT) Received: from green.homeunix.org (green@localhost [127.0.0.1]) by green.homeunix.org (8.13.1/8.13.1) with ESMTP id i9VFelDV004867; Sun, 31 Oct 2004 10:40:47 -0500 (EST) (envelope-from green@green.homeunix.org) Received: (from green@localhost) by green.homeunix.org (8.13.1/8.13.1/Submit) id i9VFelAf004866; Sun, 31 Oct 2004 10:40:47 -0500 (EST) (envelope-from green) Date: Sun, 31 Oct 2004 10:40:46 -0500 From: Brian Fundakowski Feldman To: Scott Long Message-ID: <20041031154046.GU93831@green.homeunix.org> References: <200410310503.i9V53ofj011896@repoman.freebsd.org> <20041031050620.GQ93831@green.homeunix.org> <4184BD2D.9030209@freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4184BD2D.9030209@freebsd.org> User-Agent: Mutt/1.5.6i cc: cvs-src@freebsd.org cc: src-committers@freebsd.org cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/lib/libpthread/thread thr_mutex.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 31 Oct 2004 15:40:48 -0000 On Sun, Oct 31, 2004 at 03:23:41AM -0700, Scott Long wrote: > Brian Fundakowski Feldman wrote: > >On Sun, Oct 31, 2004 at 05:03:50AM +0000, Brian Feldman wrote: > > > >>green 2004-10-31 05:03:50 UTC > >> > >> FreeBSD src repository > >> > >> Modified files: > >> lib/libpthread/thread thr_mutex.c > >> Log: > >> Make pthread_mutex_trylock(3) return EBUSY on failure, as all software > >> packages expect and seems to be most correct according to the slightly- > >> ambiguous standards. > >> > >> MFC after: 1 month > >> Corroborated by: POSIX > >> Reviewed by: silence on threads@ > > > > > >Software such as mozilla projects (using NSPR) and Java have been > >broken in various ways by this. We need to try to be more compatible > >with the most popular interpretation of the standards (instead of just > >inventing our own) -- usually we're pretty good about this. > > > > Please define 'broken'? There are many test suites available for > pthreads. How does this affect those test suites, and have you > _directly_ talked with those who run the tests suites? It causes them to _FAIL_. Try to build a mozilla port without the FreeBSD-specific pthread_mutex_trylock(3) workaround, with debugging turned on, and without the removal of EDEADLK from the function. We break the assertions that software developers use when writing to the pthreads API, and then when we "port" applications written to standard POSIX threads implementations in Solaris, Linux, etc., we have to modify them to catch this. Have you tried any of the test suites? The POSIX-provided-for-free one does not have coverage of error return values from what I can see, and the Linux pthreads ones also don't test this because as far as they are concerned, it's a code invariant, and the implementation which they control is not going to start returning EDEADLK one day. The FreeBSD one has similar issues (lack of coverage ones) but additionally is exceedingly moldy and if it were actually a "regression" test we'd fail it because half of them do not produce expected results. What other ones do you want me to try? -- Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\ <> green@FreeBSD.org \ The Power to Serve! \ Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\