On 10/14/2014 03:59 PM, MauMau wrote:
> BTW, in LWLockWaitForVar(), the first line of the following code fragment is
> not necessary, because lwWaitLink is set to head immediately. I think it
> would be good to eliminate as much unnecessary code as possible from the
> spinlock section.
>
> proc->lwWaitLink = NULL;
>
> /* waiters are added to the front of the queue */
> proc->lwWaitLink = lock->head;
Thanks, fixed!
- Heikki