From owner-cvs-all@FreeBSD.ORG Tue Oct 30 08:44:41 2007 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 0FF0716A420; Tue, 30 Oct 2007 08:44:41 +0000 (UTC) (envelope-from kris@FreeBSD.org) Received: from weak.local (pointyhat.freebsd.org [IPv6:2001:4f8:fff6::2b]) by mx1.freebsd.org (Postfix) with ESMTP id 2C40113C4BE; Tue, 30 Oct 2007 08:44:39 +0000 (UTC) (envelope-from kris@FreeBSD.org) Message-ID: <4726EEFB.8030309@FreeBSD.org> Date: Tue, 30 Oct 2007 09:44:43 +0100 From: Kris Kennaway User-Agent: Thunderbird 2.0.0.6 (Macintosh/20070728) MIME-Version: 1.0 To: David Xu References: <200710292101.l9TL1mAE049561@repoman.freebsd.org> <47268F17.1000106@freebsd.org> <47269AD0.3080906@freebsd.org> In-Reply-To: <47269AD0.3080906@freebsd.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Daniel Eischen , cvs-all@FreeBSD.org Subject: Re: cvs commit: src/lib/libthr/thread thr_mutex.c src/lib/libkse/thread thr_mutex.c src/include pthread.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 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, 30 Oct 2007 08:44:41 -0000 David Xu wrote: > Daniel Eischen wrote: >> On Tue, 30 Oct 2007, David Xu wrote: >> >>> I am not sure PTHREAD_MUTEX_ADAPTIVE_NP is a correct solution, in fact >>> I think this is Linux crap, shouldn't PTHREAD_PRIO_PROTECT and >>> PTHREAD_PRIO_INHERIT mutex be adaptivly spinned ? >>> also this commit does not change mutex_self_lock() to handle the >>> PTHREAD_MUTEX_ADAPTIVE_NP, what is the PTHREAD_MUTEX_ADAPTIVE_NP >>> definition when the mutex is already locked by the currect thread ? >>> deadlock or return error code ? >> >> >> I tend to agree with the "Linux crap" comment, but I hesitate >> to use those words considering the recent sensor framework >> incident ;-) >> > > Isn't this commit an incident too ? :-) if it is not, then > we should retire from FreeBSD now, as two thread library > maintainers were bypassed. Hi David, I'm honestly a bit surprised to hear that you consider yourself to be the maintainer of this code, because while you have certainly worked heavily on it in the past, I have sent several mails to you over the past year or so raising various problems and ideas I have encountered in the libthr and related code while working on performance tuning, and you have declined to reply to many of them, or to participate in the ongoing optimization work. That being said, it's only an "incident" if you choose to be offended by the commit. I'd recommend instead focusing on the technical issues, specifically the fact that there does not exist a better alternative way to do this at the present date. I have discussed the reasons why in my previous emails (in particular, an env variable is inappropriate), so please refer to those specifc points in formulating any further reply. Jeff, Attilio and I have ideas about how we might be able to improve the libthr and umtx code going forward, so we'd be delighted to have your help in working on them. Kris