database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-06 19:59:18 +03:00
Code

Fix statistics breakage from bgwriter/checkpointer process split.

Browse Source 
ForwardFsyncRequest() supposed that it could only be called in regular
backends, which used to be true; but since the splitup of bgwriter and
checkpointer, it is also called in the bgwriter.  We do not want to count
such calls in pg_stat_bgwriter.buffers_backend statistics, so fix things
so that they aren't.

(It's worth noting here that this implies an alarmingly large increase in
the expected amount of cross-process fsync request traffic, which may well
mean that the process splitup was not such a hot idea.)
This commit is contained in:
Tom Lane 2012-07-18 15:40:35 -04:00
parent d843589e5a
commit 1e9326d6a3
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/backend/postmaster/checkpointer.c
View File

@ -1135,6 +1135,7 @@ ForwardFsyncRequest(RelFileNode rnode, ForkNumber forknum, BlockNumber segno)
LWLockAcquire(CheckpointerCommLock, LW_EXCLUSIVE); LWLockAcquire(CheckpointerCommLock, LW_EXCLUSIVE);
/* Count all backend writes regardless of if they fit in the queue */ /* Count all backend writes regardless of if they fit in the queue */
if (!AmBackgroundWriterProcess())
CheckpointerShmem->num_backend_writes++; CheckpointerShmem->num_backend_writes++;
/* /*
@ -1150,6 +1151,7 @@ ForwardFsyncRequest(RelFileNode rnode, ForkNumber forknum, BlockNumber segno)
* Count the subset of writes where backends have to do their own * Count the subset of writes where backends have to do their own
* fsync * fsync
*/ */
if (!AmBackgroundWriterProcess())
CheckpointerShmem->num_backend_fsync++; CheckpointerShmem->num_backend_fsync++;
LWLockRelease(CheckpointerCommLock); LWLockRelease(CheckpointerCommLock);
return false; return false;