C comment: correct use of 64-"byte" cache line size

bmomjian · bmomjian · commit 226fd0c518bb · 2020-09-04T13:27:52.000-04:00
Reported-by: Kelly Min Discussion: https://postgr.es/m/CAPSbxatOiQO90LYpSC3+svAU9-sHgDfEP4oFhcEUt_X=DqFA9g@mail.gmail.com Backpatch-through: 9.5
diff --git a/src/include/storage/buf_internals.h b/src/include/storage/buf_internals.h
@@ -166,7 +166,7 @@ typedef struct BufferDesc
  * Note that local buffer descriptors aren't forced to be aligned - as there's
  * no concurrent access to those it's unlikely to be beneficial.
  *
- * We use 64bit as the cache line size here, because that's the most common
+ * We use a 64-byte cache line size here, because that's the most common
  * size. Making it bigger would be a waste of memory. Even if running on a
  * platform with either 32 or 128 byte line sizes, it's good to align to
  * boundaries and avoid false sharing.

Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,7 @@ typedef struct BufferDesc`
`166`	`166`	`* Note that local buffer descriptors aren't forced to be aligned - as there's`
`167`	`167`	`* no concurrent access to those it's unlikely to be beneficial.`
`168`	`168`	`*`
`169`		`- * We use 64bit as the cache line size here, because that's the most common`
	`169`	`+ * We use a 64-byte cache line size here, because that's the most common`
`170`	`170`	`* size. Making it bigger would be a waste of memory. Even if running on a`
`171`	`171`	`* platform with either 32 or 128 byte line sizes, it's good to align to`
`172`	`172`	`* boundaries and avoid false sharing.`