NotificationsYou must be signed in to change notification settings
Fork5.2k
Star19k

Commitbae5078

committed

Limit checkpointer requests queue size

If the number of sync requests is big enough, the palloc() call inAbsorbSyncRequests() will attempt to allocate more than 1 GB of memory,resulting in failure. This can lead to an infinite loop in the checkpointerprocess, as it repeatedly fails to absorb the pending requests.This commit limits the checkpointer requests queue size to 10M items. Inaddition to preventing the palloc() failure, this change helps to avoid longqueue processing time.Also, this commit is for backpathing only. The master branch receivesa more invasive yet comprehensive fix for this problem.Discussion:https://postgr.es/m/db4534f83a22a29ab5ee2566ad86ca92%40postgrespro.ruBackpatch-through: 13

1 parent75f633f commitbae5078Copy full SHA for bae5078

File tree

1 file changed

-1

lines changed

src/backend/postmaster
- checkpointer.c

1 file changed

-1

lines changed

`‎src/backend/postmaster/checkpointer.c‎`

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -135,6 +135,9 @@ static CheckpointerShmemStruct *CheckpointerShmem;`
`135`	`135`	`/* interval for calling AbsorbSyncRequests in CheckpointWriteDelay */`
`136`	`136`	`#defineWRITES_PER_ABSORB1000`
`137`	`137`
	`138`	`+/* Max number of requests the checkpointer request queue can hold */`
	`139`	`+#defineMAX_CHECKPOINT_REQUESTS 10000000`
	`140`	`+`
`138`	`141`	`/*`
`139`	`142`	`* GUC parameters`
`140`	`143`	`*/`
`@@ -970,7 +973,7 @@ CheckpointerShmemInit(void)`
`970`	`973`	`*/`
`971`	`974`	`MemSet(CheckpointerShmem,0,size);`
`972`	`975`	`SpinLockInit(&CheckpointerShmem->ckpt_lck);`
`973`		`-CheckpointerShmem->max_requests=NBuffers;`
	`976`	`+CheckpointerShmem->max_requests=Min(NBuffers,MAX_CHECKPOINT_REQUESTS);`
`974`	`977`	`ConditionVariableInit(&CheckpointerShmem->start_cv);`
`975`	`978`	`ConditionVariableInit(&CheckpointerShmem->done_cv);`
`976`	`979`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitbae5078

File tree

1 file changed

1 file changed

`‎src/backend/postmaster/checkpointer.c‎`

0 commit comments