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/backend/utils/init/postinit.c | |
| parent | 91ecb5e0bc7f8373ec9ff7d188bfd0d45f13a333 (diff) | |
| download | postgresql-c878de1db438f9f4747374c286d4319cc527eec2.tar.gz postgresql-c878de1db438f9f4747374c286d4319cc527eec2.zip | |
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/backend/utils/init/postinit.c')
| -rw-r--r-- | src/backend/utils/init/postinit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/init/postinit.c b/src/backend/utils/init/postinit.c index b428a59bdd2..ee1a9d5d98b 100644 --- a/src/backend/utils/init/postinit.c +++ b/src/backend/utils/init/postinit.c @@ -587,7 +587,7 @@ InitializeFastPathLocks(void) while (FastPathLockGroupsPerBackend < FP_LOCK_GROUPS_PER_BACKEND_MAX) { /* stop once we exceed max_locks_per_xact */ - if (FastPathLockGroupsPerBackend * FP_LOCK_SLOTS_PER_GROUP >= max_locks_per_xact) + if (FastPathLockSlotsPerBackend() >= max_locks_per_xact) break; FastPathLockGroupsPerBackend *= 2; |