NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit806a2ae

committed

Split work of bgwriter between 2 processes: bgwriter and checkpointer.

bgwriter is now a much less important process, responsible for pagecleaning duties only. checkpointer is now responsible for checkpointsand so has a key role in shutdown. Later patches will correct docreferences to the now old idea that bgwriter performs checkpoints.Has beneficial effect on performance at high write rates, but mainlyrefactoring to more easily allow changes for power reduction bysimplifying previously tortuous code around required to allow pagecleaning and checkpointing to time slice in the same process.Patch by me, Review by Dickson Guedes

1 parent589adb8 commit806a2aeCopy full SHA for 806a2ae

File tree

11 files changed

+124

-987

lines changed

src
- backend
  - bootstrap
    - bootstrap.c
  - postmaster
  - storage
    - buffer
      - bufmgr.c
    - smgr
      - md.c
- include
  - access
    - xlog_internal.h
  - bootstrap
    - bootstrap.h
  - postmaster
    - bgwriter.h
  - storage
    - proc.h
    - procsignal.h

11 files changed

+124

-987

lines changed

`‎src/backend/bootstrap/bootstrap.c`

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -315,6 +315,9 @@ AuxiliaryProcessMain(int argc, char *argv[])`
`315`	`315`	`caseBgWriterProcess:`
`316`	`316`	`statmsg="writer process";`
`317`	`317`	`break;`
	`318`	`+caseCheckpointerProcess:`
	`319`	`+statmsg="checkpointer process";`
	`320`	`+break;`
`318`	`321`	`caseWalWriterProcess:`
`319`	`322`	`statmsg="wal writer process";`
`320`	`323`	`break;`
`@@ -415,6 +418,11 @@ AuxiliaryProcessMain(int argc, char *argv[])`
`415`	`418`	`BackgroundWriterMain();`
`416`	`419`	`proc_exit(1);/* should never return */`
`417`	`420`
	`421`	`+caseCheckpointerProcess:`
	`422`	`+/* don't set signals, checkpointer has its own agenda */`
	`423`	`+CheckpointerMain();`
	`424`	`+proc_exit(1);/* should never return */`
	`425`	`+`
`418`	`426`	`caseWalWriterProcess:`
`419`	`427`	`/* don't set signals, walwriter has its own agenda */`
`420`	`428`	`InitXLOGAccess();`

`‎src/backend/postmaster/Makefile`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,6 @@ top_builddir = ../../..`
`13`	`13`	`include$(top_builddir)/src/Makefile.global`
`14`	`14`
`15`	`15`	`OBJS = autovacuum.o bgwriter.o fork_process.o pgarch.o pgstat.o postmaster.o\`
`16`		`-syslogger.o walwriter.o`
	`16`	`+syslogger.o walwriter.o checkpointer.o`
`17`	`17`
`18`	`18`	`include$(top_srcdir)/src/backend/common.mk`

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!

Commit806a2ae

File tree

11 files changed

11 files changed

`‎src/backend/bootstrap/bootstrap.c`

`‎src/backend/postmaster/Makefile`

0 commit comments