]> granicus.if.org Git - postgresql/commit
Lockless StrategyGetBuffer clock sweep hot path.
authorAndres Freund <andres@anarazel.de>
Thu, 25 Dec 2014 17:24:20 +0000 (18:24 +0100)
committerAndres Freund <andres@anarazel.de>
Thu, 25 Dec 2014 17:26:25 +0000 (18:26 +0100)
commitd72731a70450b5e7084991b9caa15cb58a2820df
tree9b7cea630143e09daed98f0f39019ae5f81115ee
parentab5194e6f617a9a9e7aadb3dd1cee948a42d0755
Lockless StrategyGetBuffer clock sweep hot path.

StrategyGetBuffer() has proven to be a bottleneck in a number of
buffer acquisition heavy workloads. To some degree this has already
been alleviated by 5d7962c6, but it still can be quite a heavy
bottleneck.  The problem is that in unfortunate usage patterns a
single StrategyGetBuffer() call will have to look at a large number of
buffers - in turn making it likely that the process will be put to
sleep while still holding the spinlock.

Replace most of the usage of the buffer_strategy_lock spinlock for the
clock sweep by a atomic nextVictimBuffer variable. That variable,
modulo NBuffers, is the current hand of the clock sweep. The buffer
clock-sweep then only needs to acquire the spinlock after a
wraparound. And even then only in the process that did the wrapping
around. That alleviates nearly all the contention on the relevant
spinlock, although significant contention on the cacheline can still
exist.

Reviewed-By: Robert Haas and Amit Kapila
Discussion: 20141010160020.GG6670@alap3.anarazel.de,
    20141027133218.GA2639@awork2.anarazel.de
src/backend/postmaster/bgwriter.c
src/backend/storage/buffer/freelist.c
src/include/storage/buf_internals.h