LWLockRelease() currently does something like (simplifying a lot):

    acquire lwlock spinlock
    decrement lock count
    if lock is free
      if first waiter in queue is waiting for exclusive lock,
      awaken him; else, walk through the queue and awaken
      all the shared waiters until we reach an exclusive waiter
    end if
    release lwlock spinlock

This has the nice property that locks are granted in FIFO order. Is it
essential that we maintain that property? If not, we could instead walk
through the wait queue and awaken *all* the shared waiters, and get a
small improvement in throughput.

I can see that this might starve exclusive waiters; however, we allow
the following:

    Proc A => LWLockAcquire(lock, LW_SHARED); -- succeeds
    Proc B => LWLockAcquire(lock, LW_EXCLUSIVE); -- blocks
    Proc C => LWLockAcquire(lock, LW_SHARED); -- succeeds

i.e. we don't *really* follow strict FIFO order anyway.

-Neil



---------------------------(end of broadcast)---------------------------
TIP 3: if posting/reading through Usenet, please send an appropriate
      subscribe-nomail command to [EMAIL PROTECTED] so that your
      message can get through to the mailing list cleanly

Reply via email to