From cb9d1faa4a7dca9b28f4a21cc9b2c25287eb74d5 Mon Sep 17 00:00:00 2001 From: Bruce Momjian Date: Fri, 4 Sep 2020 13:27:52 -0400 Subject: [PATCH] C comment: correct use of 64-"byte" cache line size 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 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/include/storage/buf_internals.h b/src/include/storage/buf_internals.h index 5370035f0c6..e139de24491 100644 --- a/src/include/storage/buf_internals.h +++ b/src/include/storage/buf_internals.h @@ -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. -- 2.47.2