Re: max_files_per_process ignored on Windows - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: max_files_per_process ignored on Windows
Date
Msg-id CABUevEz=GPhspq9Th+nHvZPttn-j7-_nobsT05eOMmHRNu_7kg@mail.gmail.com
Whole thread Raw
In response to max_files_per_process ignored on Windows  (Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>)
Responses Re: max_files_per_process ignored on Windows  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Wed, Mar 28, 2012 at 18:12, Heikki Linnakangas
<heikki.linnakangas@enterprisedb.com> wrote:
> At postmaster startup, we determine the maximum number of open files we can
> handle by trying to open a lot of file descriptors, up to
> max_files_per_process. This is done in set_max_safe_fds(), and the
> determined max_safe_fds value is inherited by child processes at fork().
> However, with EXEC_BACKEND, ie. Windows, it's not inherited, so we always
> run with the initial conservative default of 32.
>
> An obvious fix would be to call set_max_safe_fds() in the child processes,
> although I wonder if that's too expensive. Another option is to pass down
> the value with save_restore_backend_variables().

ISTM that passing down through save_restore_backend_variables() is a
much better choice.


> Thoughts? Although this has apparently always been like this, no-one has
> complained, so I'm thinking that we shouldn't backport this.

We should absolutely *not* backport this. It needs to go through some
proper testing first, it might cause serious effects on some systems.
In particular, it might have yet another round of effects on people
who run with AV enabled...

--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/


pgsql-hackers by date:

Previous
From: Jaime Casanova
Date:
Subject: Re: Review of pg_archivecleanup -x option patch
Next
From: Robert Haas
Date:
Subject: Re: Finer Extension dependencies