Commit1be2035

committed

Tune BufferGetLSNAtomic() when checksums !enabled

From performance analysis by Heikki Linnakangas

1 parentcf8dc9e commit1be2035Copy full SHA for 1be2035

File tree

-2

lines changed

-2

lines changed

Lines changed: 4 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -2079,8 +2079,10 @@ BufferGetLSNAtomic(Buffer buffer)`
`2079`	`2079`	`char*page=BufferGetPage(buffer);`
`2080`	`2080`	`XLogRecPtrlsn;`
`2081`	`2081`
`2082`		`-/* Local buffers don't need a lock. */`
`2083`		`-if (BufferIsLocal(buffer))`
	`2082`	`+/*`
	`2083`	`+ * If we don't need locking for correctness, fastpath out.`
	`2084`	`+ */`
	`2085`	`+if (!DataChecksumsEnabled()\|\|BufferIsLocal(buffer))`
`2084`	`2086`	`returnPageGetLSN(page);`
`2085`	`2087`
`2086`	`2088`	`/* Make sure we've got a real buffer, and that we hold a pin on it. */`

Comments

(0)