diff options
| author | Tomas Vondra <tomas.vondra@postgresql.org> | 2025-03-04 18:33:09 +0100 |
|---|---|---|
| committer | Tomas Vondra <tomas.vondra@postgresql.org> | 2025-03-04 18:33:12 +0100 |
| commit | c878de1db438f9f4747374c286d4319cc527eec2 (patch) | |
| tree | 0444c92e7f15016d25f4ade8b815ccd0228c6f3f /src/include/storage/proc.h | |
| parent | 91ecb5e0bc7f8373ec9ff7d188bfd0d45f13a333 (diff) | |
Make FP_LOCK_SLOTS_PER_BACKEND look like a function
The FP_LOCK_SLOTS_PER_BACKEND macro looks like a constant, but it
depends on the max_locks_per_transaction GUC, and thus can change. This
is non-obvious and confusing, so make it look more like a function by
renaming it to FastPathLockSlotsPerBackend().
While at it, use the macro when initializing fast-path shared memory,
instead of using the formula.
Reported-by: Andres Freund
Discussion: https://postgr.es/m/ffiwtzc6vedo6wb4gbwelon5nefqg675t5c7an2ta7pcz646cg%40qwmkdb3l4ett
Diffstat (limited to 'src/include/storage/proc.h')
| -rw-r--r-- | src/include/storage/proc.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/include/storage/proc.h b/src/include/storage/proc.h index 20777f7d5ae..114eb1f8f76 100644 --- a/src/include/storage/proc.h +++ b/src/include/storage/proc.h @@ -88,7 +88,8 @@ extern PGDLLIMPORT int FastPathLockGroupsPerBackend; #define FP_LOCK_GROUPS_PER_BACKEND_MAX 1024 #define FP_LOCK_SLOTS_PER_GROUP 16 /* don't change */ -#define FP_LOCK_SLOTS_PER_BACKEND (FP_LOCK_SLOTS_PER_GROUP * FastPathLockGroupsPerBackend) +#define FastPathLockSlotsPerBackend() \ + (FP_LOCK_SLOTS_PER_GROUP * FastPathLockGroupsPerBackend) /* * Flags for PGPROC.delayChkptFlags |