diff options
| author | Thomas Munro <tmunro@postgresql.org> | 2020-01-31 16:58:38 +1300 |
|---|---|---|
| committer | Thomas Munro <tmunro@postgresql.org> | 2020-01-31 17:29:38 +1300 |
| commit | d061ea21fc1cc1c657bb5c742f5c4a1564e82ee2 (patch) | |
| tree | 812300662f52ce3edaefdbe98c717bc1de1864a9 | |
| parent | a069218163704c44a8996e7e98e765c56e2b9c8e (diff) | |
| download | postgresql-d061ea21fc1cc1c657bb5c742f5c4a1564e82ee2.tar.gz postgresql-d061ea21fc1cc1c657bb5c742f5c4a1564e82ee2.zip | |
Adjust DSM and DSA slot usage constants.
When running a lot of large parallel queries concurrently, or a plan with
a lot of separate Gather nodes, it is possible to run out of DSM slots.
There are better solutions to these problems requiring architectural
redesign work, but for now, let's adjust the constants so that it's more
difficult to hit the limit.
1. Previously, a DSA area would create up to four segments at each size
before doubling the size. After this commit, it will create only two at
each size, so it ramps up faster and therefore needs fewer slots.
2. Previously, the total limit on DSM slots allowed for 2 per connection.
Switch to 5 per connection.
Also remove an obsolete nearby comment.
Author: Thomas Munro
Reviewed-by: Robert Haas, Andres Freund
Discussion: https://postre.es/m/CA%2BhUKGL6H2BpGbiF7Lj6QiTjTGyTLW_vLR%3DSn2tEBeTcYXiMKw%40mail.gmail.com
| -rw-r--r-- | src/backend/storage/ipc/dsm.c | 7 | ||||
| -rw-r--r-- | src/backend/utils/mmgr/dsa.c | 2 |
2 files changed, 2 insertions, 7 deletions
diff --git a/src/backend/storage/ipc/dsm.c b/src/backend/storage/ipc/dsm.c index 90e0d739f80..1ed6ed2b1d1 100644 --- a/src/backend/storage/ipc/dsm.c +++ b/src/backend/storage/ipc/dsm.c @@ -45,13 +45,8 @@ #define PG_DYNSHMEM_CONTROL_MAGIC 0x9a503d32 -/* - * There's no point in getting too cheap here, because the minimum allocation - * is one OS page, which is probably at least 4KB and could easily be as high - * as 64KB. Each currently sizeof(dsm_control_item), currently 8 bytes. - */ #define PG_DYNSHMEM_FIXED_SLOTS 64 -#define PG_DYNSHMEM_SLOTS_PER_BACKEND 2 +#define PG_DYNSHMEM_SLOTS_PER_BACKEND 5 #define INVALID_CONTROL_SLOT ((uint32) -1) diff --git a/src/backend/utils/mmgr/dsa.c b/src/backend/utils/mmgr/dsa.c index adbb06fb523..b7ad8e62ef3 100644 --- a/src/backend/utils/mmgr/dsa.c +++ b/src/backend/utils/mmgr/dsa.c @@ -74,7 +74,7 @@ * dsm.c's limits on total number of segments), or limiting the total size * an area can manage when using small pointers. */ -#define DSA_NUM_SEGMENTS_AT_EACH_SIZE 4 +#define DSA_NUM_SEGMENTS_AT_EACH_SIZE 2 /* * The number of bits used to represent the offset part of a dsa_pointer. |