NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit848b9f0

committed

Use correctly-sized buffer when zero-filling a WAL file.

I mixed up BLCKSZ and XLOG_BLCKSZ when I changed the way the buffer isallocated a couple of weeks ago. With the default settings, they are both8k, but they can be changed at compile-time.

1 parentab76d8e commit848b9f0Copy full SHA for 848b9f0

File tree

1 file changed

-2

lines changed

src/backend/access/transam
- xlog.c

1 file changed

-2

lines changed

`‎src/backend/access/transam/xlog.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -3079,7 +3079,7 @@ XLogFileInit(XLogSegNo logsegno, bool *use_existent, bool use_lock)`
`3079`	`3079`	`{`
`3080`	`3080`	`charpath[MAXPGPATH];`
`3081`	`3081`	`chartmppath[MAXPGPATH];`
`3082`		`-charzbuffer_raw[BLCKSZ+MAXIMUM_ALIGNOF];`
	`3082`	`+charzbuffer_raw[XLOG_BLCKSZ+MAXIMUM_ALIGNOF];`
`3083`	`3083`	`char*zbuffer;`
`3084`	`3084`	`XLogSegNoinstalled_segno;`
`3085`	`3085`	`intmax_advance;`
`@@ -3139,7 +3139,7 @@ XLogFileInit(XLogSegNo logsegno, bool *use_existent, bool use_lock)`
`3139`	`3139`	`* cycles transferring data to the kernel.`
`3140`	`3140`	`*/`
`3141`	`3141`	`zbuffer= (char*)MAXALIGN(zbuffer_raw);`
`3142`		`-memset(zbuffer,0,BLCKSZ);`
	`3142`	`+memset(zbuffer,0,XLOG_BLCKSZ);`
`3143`	`3143`	`for (nbytes=0;nbytes<XLogSegSize;nbytes+=XLOG_BLCKSZ)`
`3144`	`3144`	`{`
`3145`	`3145`	`errno=0;`

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!

Commit848b9f0

File tree

1 file changed

1 file changed

`‎src/backend/access/transam/xlog.c`

0 commit comments