Re: Spinlocks and semaphores in 9.2 and 9.3

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: pgsql-hackers(at)postgresql(dot)org, Robert Haas <robertmhaas(at)gmail(dot)com>
Subject: Re: Spinlocks and semaphores in 9.2 and 9.3
Date: 2016-04-18 15:24:07
Message-ID: 25476.1460993047@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Andres Freund <andres(at)anarazel(dot)de> writes:
> On 2016-04-18 11:07:08 -0400, Tom Lane wrote:
>> Did you want to actually review this patch, or should I just push it?

> No, I'm good, you should push it. I did a quick scan of the patch, and
> it looks sane. For a second I was concerned that there might be a
> situation in which this patch increases the total number of semaphore
> needed, which might make backpatching a bit problematic - but it appears
> that that'd be a very absurd configuration.

I was actually wondering whether it'd make sense to cut the number of
underlying semaphores to 128 or 512 or thereabouts. But I think we had
that discussion when the daa7527afc227443 patch went in to begin with,
and convinced ourselves that 1024 was okay. Robert, do you recall the
reasoning?

(The thing that gave me pause about this was noticing that I could not
start two such postmasters concurrently on my RHEL6 box, without changing
the default system limits on number of SysV semaphores.)

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Aleksander Alekseev 2016-04-18 15:25:23 Re: Parser extensions (maybe for 10?)
Previous Message Pavel Stehule 2016-04-18 15:19:55 Re: Parser extensions (maybe for 10?)