Make the updates of the tid ring buffer' head and tail pointers

explicit by moving them into separate statements from the buffer
element accesses.

Requested by:	jhb
MFC after:	3 days
This commit is contained in:
Konstantin Belousov 2012-09-26 09:25:11 +00:00
parent 43f3d8e372
commit 94cb35459d

View file

@ -102,8 +102,8 @@ tid_alloc(void)
mtx_unlock(&tid_lock);
return (-1);
}
tid = tid_buffer[tid_head++];
tid_head %= TID_BUFFER_SIZE;
tid = tid_buffer[tid_head];
tid_head = (tid_head + 1) % TID_BUFFER_SIZE;
mtx_unlock(&tid_lock);
return (tid);
}
@ -115,11 +115,11 @@ tid_free(lwpid_t tid)
mtx_lock(&tid_lock);
if ((tid_tail + 1) % TID_BUFFER_SIZE == tid_head) {
tmp_tid = tid_buffer[tid_head++];
tid_head %= TID_BUFFER_SIZE;
tmp_tid = tid_buffer[tid_head];
tid_head = (tid_head + 1) % TID_BUFFER_SIZE;
}
tid_buffer[tid_tail++] = tid;
tid_tail %= TID_BUFFER_SIZE;
tid_buffer[tid_tail] = tid;
tid_tail = (tid_tail + 1) % TID_BUFFER_SIZE;
mtx_unlock(&tid_lock);
if (tmp_tid != -1)
free_unr(tid_unrhdr, tmp_tid);