Date: Sat, 27 Nov 2004 11:55:17 -0700 From: Scott Long <scottl@freebsd.org> To: Nate Lawson <nate@root.org> Cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/kern kern_synch.c Message-ID: <41A8CD95.3050005@freebsd.org> In-Reply-To: <41A8C429.1090406@root.org> References: <20041127114337.95FC716A4EE@hub.freebsd.org> <41A8C429.1090406@root.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Nate Lawson wrote:
> 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
No, this just fixes an edge-case that wasn't covered before.
Scott
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?41A8CD95.3050005>