Fix comment of StrategySyncStart()

michaelpq · michaelpq · commit 71832a189b19 · 2025-01-31T11:06:12.000+09:00
The top comment of StrategySyncStart() mentions BufferSync(), but this function calls BgBufferSync(), not BufferSync(). Oversight in 9cd00c4. Author: Ashutosh Bapat Discussion: https://postgr.es/m/CAExHW5tgkjag8i-s=RFrCn5KAWDrC4zEPPkfUKczfccPOxBRQQ@mail.gmail.com Backpatch-through: 13
diff --git a/src/backend/storage/buffer/freelist.c b/src/backend/storage/buffer/freelist.c
@@ -381,10 +381,10 @@ StrategyFreeBuffer(BufferDesc *buf)
 }
 
 /*
- * StrategySyncStart -- tell BufferSync where to start syncing
+ * StrategySyncStart -- tell BgBufferSync where to start syncing
  *
  * The result is the buffer index of the best buffer to sync first.
- * BufferSync() will proceed circularly around the buffer array from there.
+ * BgBufferSync() will proceed circularly around the buffer array from there.
  *
  * In addition, we return the completed-pass count (which is effectively
  * the higher-order bits of nextVictimBuffer) and the count of recent buffer

Original file line number	Diff line number	Diff line change
`@@ -381,10 +381,10 @@ StrategyFreeBuffer(BufferDesc *buf)`
`381`	`381`	`}`
`382`	`382`
`383`	`383`	`/*`
`384`		`- * StrategySyncStart -- tell BufferSync where to start syncing`
	`384`	`+ * StrategySyncStart -- tell BgBufferSync where to start syncing`
`385`	`385`	`*`
`386`	`386`	`* The result is the buffer index of the best buffer to sync first.`
`387`		`- * BufferSync() will proceed circularly around the buffer array from there.`
	`387`	`+ * BgBufferSync() will proceed circularly around the buffer array from there.`
`388`	`388`	`*`
`389`	`389`	`* In addition, we return the completed-pass count (which is effectively`
`390`	`390`	`* the higher-order bits of nextVictimBuffer) and the count of recent buffer`