diff options
| author | Michael Paquier <michael@paquier.xyz> | 2025-05-07 14:55:57 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2025-05-07 14:55:57 +0900 |
| commit | c4c236ab5c416131baa7109cdc25ccaadd4d7158 (patch) | |
| tree | 23c7b7e12743f71036753733d999ffdfb01ef5fb | |
| parent | 09a47c68e2fce446849ce0262eff1bfbf921fa18 (diff) | |
| download | postgresql-c4c236ab5c416131baa7109cdc25ccaadd4d7158.tar.gz postgresql-c4c236ab5c416131baa7109cdc25ccaadd4d7158.zip | |
Fix some comments related to IO workers
IO workers are treated as auxiliary processes. The comments fixed in
this commit stated that there could be only one auxiliary process of
each BackendType at the same time. This is not true for IO workers, as
up to MAX_IO_WORKERS of them can co-exist at the same time.
Author: Cédric Villemain <Cedric.Villemain@data-bene.io>
Co-authored-by: Michael Paquier <michael@paquier.xyz>
Discussion: https://postgr.es/m/e4a3ac45-abce-4b58-a043-b4a31cd11113@Data-Bene.io
| -rw-r--r-- | src/backend/storage/ipc/procsignal.c | 3 | ||||
| -rw-r--r-- | src/backend/storage/ipc/sinvaladt.c | 3 | ||||
| -rw-r--r-- | src/include/miscadmin.h | 2 |
3 files changed, 5 insertions, 3 deletions
diff --git a/src/backend/storage/ipc/procsignal.c b/src/backend/storage/ipc/procsignal.c index a3c2cd12277..1a4cc5e5d97 100644 --- a/src/backend/storage/ipc/procsignal.c +++ b/src/backend/storage/ipc/procsignal.c @@ -89,7 +89,8 @@ struct ProcSignalHeader /* * We reserve a slot for each possible ProcNumber, plus one for each * possible auxiliary process type. (This scheme assumes there is not - * more than one of any auxiliary process type at a time.) + * more than one of any auxiliary process type at a time, except for + * IO workers.) */ #define NumProcSignalSlots (MaxBackends + NUM_AUXILIARY_PROCS) diff --git a/src/backend/storage/ipc/sinvaladt.c b/src/backend/storage/ipc/sinvaladt.c index 2da91738c32..c5748b690f4 100644 --- a/src/backend/storage/ipc/sinvaladt.c +++ b/src/backend/storage/ipc/sinvaladt.c @@ -198,7 +198,8 @@ typedef struct SISeg /* * We reserve a slot for each possible ProcNumber, plus one for each * possible auxiliary process type. (This scheme assumes there is not - * more than one of any auxiliary process type at a time.) + * more than one of any auxiliary process type at a time, except for + * IO workers.) */ #define NumProcStateSlots (MaxBackends + NUM_AUXILIARY_PROCS) diff --git a/src/include/miscadmin.h b/src/include/miscadmin.h index 72f5655fb34..629772b0381 100644 --- a/src/include/miscadmin.h +++ b/src/include/miscadmin.h @@ -353,7 +353,7 @@ typedef enum BackendType * Auxiliary processes. These have PGPROC entries, but they are not * attached to any particular database, and cannot run transactions or * even take heavyweight locks. There can be only one of each of these - * running at a time. + * running at a time, except for IO workers. * * If you modify these, make sure to update NUM_AUXILIARY_PROCS and the * glossary in the docs. |