Re: Bump soft open file limit (RLIMIT_NOFILE) to hard limit on startup

Jelte Fennema-Nio <postgres@jeltef.nl>

From: "Jelte Fennema-Nio" <postgres@jeltef.nl>

To: "Peter Eisentraut" <peter@eisentraut.org>

Cc: "Heikki Linnakangas" <hlinnaka@iki.fi>, "Andres Freund" <andres@anarazel.de>, "Tom Lane" <tgl@sss.pgh.pa.us>, "Tomas Vondra" <tomas@vondra.me>, "PostgreSQL-development" <pgsql-hackers@postgresql.org>

Date: 2025-11-03T17:04:28Z

Lists: pgsql-hackers

Attachments

v8-0001-Adds-a-helper-for-places-that-shell-out-to-system.patch (text/x-patch) patch v8-0001
v8-0002-Bump-postmaster-soft-open-file-limit-RLIMIT_NOFIL.patch (text/x-patch) patch v8-0002
v8-0003-Reflect-the-value-of-max_safe_fds-in-max_files_pe.patch (text/x-patch) patch v8-0003

On Wed, 29 Oct 2025 at 11:11, Peter Eisentraut <peter@eisentraut.org> wrote:
> This patch set is showing compiler warnings because pg_system() wasn't
> properly declared where needed.  Please provide an update that builds
> cleanly.

It still compiled fine on my local branch, but indeed after a rebase not anymore. I guess some include got removed in some header in the meantime. Attached an updated version (which added an storage/fd.h include).

> Also, it appears the patch for pgbench disappeared from the series.  Was
> that intentional?

Yes, the pgbench change got committed for PG18 already (see Andres' last message in the thread).

Commits

Redefine max_files_per_process to control additionally opened files
- adb5f85fa5a0 18.0 landed
pgbench: Increase RLIMIT_NOFILE if necessary
- d38bab5edd60 18.0 landed
Move extra code out of the Pre/PostRestoreCommand() section.
- 8fb13dd6ab5b 17.0 cited