Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
C comment: correct use of 64-"byte" cache line size
authorBruce Momjian <bruce@momjian.us>
Fri, 4 Sep 2020 17:27:52 +0000 (13:27 -0400)
committerBruce Momjian <bruce@momjian.us>
Fri, 4 Sep 2020 17:27:52 +0000 (13:27 -0400)
Reported-by: Kelly Min
Discussion: https://postgr.es/m/CAPSbxatOiQO90LYpSC3+svAU9-sHgDfEP4oFhcEUt_X=DqFA9g@mail.gmail.com

Backpatch-through: 9.5

src/include/storage/buf_internals.h

index 5370035f0c659847e1cacaef8243fecd0f00a67f..e139de24491bd017a304a57816c3e60405f8fbab 100644 (file)
@@ -204,7 +204,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.