Re: pgsql: aio: Infrastructure for io_method=worker

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: aio: Infrastructure for io_method=worker
Date: 2025-03-18 22:53:48
Message-ID: 593800.1742338428@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Andres Freund <andres(at)anarazel(dot)de> writes:
> To allow the number of IO workers to be increased without a restart, we need
> to reserve PGPROC entries for the workers unconditionally. This has been
> judged to be worth the cost. If it turns out to be problematic, we can
> introduce a PGC_POSTMASTER GUC to control the maximum number.

So I see this patch added 32 PGPROCs and hence 32 semaphores to the
system's requirements. Unsurprisingly, this broke OpenBSD/NetBSD
again:

https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=sawshark&dt=2025-03-18%2016%3A20%3A05

It's probably time to just abandon the idea of being able to run with
only 60 semaphores. I wonder though if we ought to revert 38da05346
and/or 6d0154196 in view of that.

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andres Freund 2025-03-18 23:07:53 Re: pgsql: aio: Infrastructure for io_method=worker
Previous Message Nathan Bossart 2025-03-18 22:00:40 pgsql: Add commit 796bdda484 to .git-blame-ignore-revs.

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2025-03-18 23:07:53 Re: pgsql: aio: Infrastructure for io_method=worker
Previous Message Aidar Imamov 2025-03-18 22:02:52 Re: Add pg_buffercache_evict_all() and pg_buffercache_mark_dirty[_all]() functions