From owner-cvs-all@FreeBSD.ORG  Sat Nov 27 18:15:26 2004
Return-Path: <owner-cvs-all@FreeBSD.ORG>
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 2942416A4CF; Sat, 27 Nov 2004 18:15:25 +0000 (GMT)
Received: from ylpvm43.prodigy.net (ylpvm43-ext.prodigy.net [207.115.57.74])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id AA15F43D48; Sat, 27 Nov 2004 18:15:15 +0000 (GMT)
	(envelope-from nate@root.org)
Received: from [10.0.5.51] (adsl-64-171-186-185.dsl.snfc21.pacbell.net
	[64.171.186.185])iARIFIp8020723;	Sat, 27 Nov 2004 13:15:18 -0500
Message-ID: <41A8C429.1090406@root.org>
Date: Sat, 27 Nov 2004 10:15:05 -0800
From: Nate Lawson <nate@root.org>
User-Agent: Mozilla Thunderbird 0.7.3 (X11/20040901)
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: David Xu <davidxu@FreeBSD.org>
References: <20041127114337.95FC716A4EE@hub.freebsd.org>
In-Reply-To: <20041127114337.95FC716A4EE@hub.freebsd.org>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
cc: cvs-src@FreeBSD.org
cc: src-committers@FreeBSD.org
cc: cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/kern kern_synch.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sat, 27 Nov 2004 18:15:26 -0000

David Xu wrote:
> davidxu     2004-11-27 11:43:31 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/kern             kern_synch.c 
>   Log:
>   Unlock mutex if PDROP was set by caller.
>   
>   Revision  Changes    Path
>   1.265     +2 -0      src/sys/kern/kern_synch.c
> 
> 
> Index: src/sys/kern/kern_synch.c
> diff -u src/sys/kern/kern_synch.c:1.264 src/sys/kern/kern_synch.c:1.265
> --- src/sys/kern/kern_synch.c:1.264	Sat Oct 16 06:38:22 2004
> +++ src/sys/kern/kern_synch.c	Sat Nov 27 11:43:31 2004
> @@ -175,6 +175,8 @@
>  		mtx_unlock_spin(&sched_lock);
>  		if (rval != 0) {
>  			sleepq_release(ident);
> +			if (mtx != NULL && priority & PDROP)
> +				mtx_unlock(mtx);
>  			return (rval);
>  		}
>  	}

Are you serious?  PDROP was unimplemented until now?

-Nate