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

bmomjian · bmomjian · commit 3b5af0e95ad5 · 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
@@ -203,7 +203,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
`@@ -203,7 +203,7 @@ typedef struct BufferDesc`
`203`	`203`	`* Note that local buffer descriptors aren't forced to be aligned - as there's`
`204`	`204`	`* no concurrent access to those it's unlikely to be beneficial.`
`205`	`205`	`*`
`206`		`- * We use 64bit as the cache line size here, because that's the most common`
	`206`	`+ * We use a 64-byte cache line size here, because that's the most common`
`207`	`207`	`* size. Making it bigger would be a waste of memory. Even if running on a`
`208`	`208`	`* platform with either 32 or 128 byte line sizes, it's good to align to`
`209`	`209`	`* boundaries and avoid false sharing.`