Fix comment of StrategySyncStart()

michaelpq · michaelpq · commit 1e380fa7d82d · 2025-01-31T11:05:57.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
@@ -380,10 +380,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
`@@ -380,10 +380,10 @@ StrategyFreeBuffer(BufferDesc *buf)`
`380`	`380`	`}`
`381`	`381`
`382`	`382`	`/*`
`383`		`- * StrategySyncStart -- tell BufferSync where to start syncing`
	`383`	`+ * StrategySyncStart -- tell BgBufferSync where to start syncing`
`384`	`384`	`*`
`385`	`385`	`* The result is the buffer index of the best buffer to sync first.`
`386`		`- * BufferSync() will proceed circularly around the buffer array from there.`
	`386`	`+ * BgBufferSync() will proceed circularly around the buffer array from there.`
`387`	`387`	`*`
`388`	`388`	`* In addition, we return the completed-pass count (which is effectively`
`389`	`389`	`* the higher-order bits of nextVictimBuffer) and the count of recent buffer`