Invent a variant of getopt(3) that is thread-safe
Heikki Linnakangas <heikki.linnakangas@iki.fi>
Author:
Heikki Linnakangas <heikki.linnakangas@iki.fi>
Date: 2026-03-30T17:47:13Z
Releases:
19 (unreleased)
Invent a variant of getopt(3) that is thread-safe The standard getopt(3) function is not re-entrant nor thread-safe. That's OK for current usage, but it's one more little thing we need to change in order to make the server multi-threaded. There's no standard getopt_r() function on any platform, I presume because command line arguments are usually parsed early when you start a program, before launching any threads, so there isn't much need for it. However, we call it at backend startup to parse options from the startup packet. Because there's no standard, we're free to define our own. The pg_getopt_start/next() implementation is based on the old getopt implementation, I just gathered all the state variables to a struct. The non-re-entrant getopt() function is now a wrapper around the re-entrant variant, on platforms that don't have getopt(3). getopt_long() is not used in the server, so we don't need to provide a re-entrant variant of that. Reviewed-by: Peter Eisentraut <peter@eisentraut.org> Discussion: https://www.postgresql.org/message-id/d1da5f0e-0d68-47c9-a882-eb22f462752f@iki.fi
Files
| Path | Change | +/− |
|---|---|---|
| src/include/port/pg_getopt_ctx.h | added | +39 −0 |
| src/port/getopt.c | modified | +21 −70 |
| src/port/Makefile | modified | +1 −0 |
| src/port/meson.build | modified | +1 −0 |
| src/port/pg_getopt_ctx.c | added | +136 −0 |
| src/tools/pgindent/typedefs.list | modified | +1 −0 |
Discussion
- report on not thread-safe functions 9 messages · 2024-06-06 → 2026-03-30