qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-25 15:08:02 -07:00
Code Releases Activity

runtime: wait to allocate mach semaphores backing Locks until needed

Browse Source 
need better management of mach semaphores eventually
but this avoids allocating them for uncontended Locks.

R=r
CC=agl1, golang-dev
https://golang.org/cl/190079
This commit is contained in:
Russ Cox 2010-01-19 21:14:15 -08:00
parent b2beb8abf0
commit 0365b989a4
1 changed files with 18 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/pkg/runtime/darwin/thread.c
View File

@ -60,12 +60,12 @@ lock(Lock *l)
throw("lock count"); throw("lock count");
m->locks++; m->locks++;
// Allocate semaphore if needed. if(xadd(&l->key, 1) > 1) { // someone else has it; wait
if(l->sema == 0) // Allocate semaphore if needed.
initsema(&l->sema); if(l->sema == 0)
initsema(&l->sema);
if(xadd(&l->key, 1) > 1) // someone else has it; wait
mach_semacquire(l->sema); mach_semacquire(l->sema);
}
} }
void void
@ -75,8 +75,12 @@ unlock(Lock *l)
if(m->locks < 0) if(m->locks < 0)
throw("lock count"); throw("lock count");
if(xadd(&l->key, -1) > 0) // someone else is waiting if(xadd(&l->key, -1) > 0) { // someone else is waiting
// Allocate semaphore if needed.
if(l->sema == 0)
initsema(&l->sema);
mach_semrelease(l->sema); mach_semrelease(l->sema);
}
} }
@ -87,15 +91,21 @@ unlock(Lock *l)
void void
usemacquire(Usema *s) usemacquire(Usema *s)
{ {
if((int32)xadd(&s->u, -1) < 0) if((int32)xadd(&s->u, -1) < 0) {
if(s->k == 0)
initsema(&s->k);
mach_semacquire(s->k); mach_semacquire(s->k);
}
} }
void void
usemrelease(Usema *s) usemrelease(Usema *s)
{ {
if((int32)xadd(&s->u, 1) <= 0) if((int32)xadd(&s->u, 1) <= 0) {
if(s->k == 0)
initsema(&s->k);
mach_semrelease(s->k); mach_semrelease(s->k);
}
} }
@ -109,8 +119,6 @@ noteclear(Note *n)
void void
notesleep(Note *n) notesleep(Note *n)
{ {
if(n->sema.k == 0)
initsema(&n->sema.k);
while(!n->wakeup) while(!n->wakeup)
usemacquire(&n->sema); usemacquire(&n->sema);
} }
@ -118,8 +126,6 @@ notesleep(Note *n)
void void
notewakeup(Note *n) notewakeup(Note *n)
{ {
if(n->sema.k == 0)
initsema(&n->sema.k);
n->wakeup = 1; n->wakeup = 1;
usemrelease(&n->sema); usemrelease(&n->sema);
} }