pgsql: Fix the fallback memory barrier implementation to be reentrant.

From: Andres Freund <andres(at)anarazel(dot)de>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix the fallback memory barrier implementation to be reentrant.
Date: 2015-06-26 15:13:32
Message-ID: E1Z8VJs-0006nf-I3@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix the fallback memory barrier implementation to be reentrant.

This was essentially "broken" since 0c8eda62; but until more
recently (14e8803f) barriers usage in signal handlers was infrequent.

The failure to be reentrant was noticed because the test_shm_mq, which
uses memory barriers at a high frequency, occasionally got stuck on some
solaris buildfarm animals. Turns out, those machines use sun studio
12.1, which doesn't yet have efficient memory barrier support. A machine
with a newer sun studio did not fail. Forcing the barrier fallback to
be used on x86 allows to reproduce the problem.

The new fallback is to use kill(PostmasterPid, 0) based on the theory
that that'll always imply a barrier due to checking the liveliness of
PostmasterPid on systems old enough to need fallback support. It's hard
to come up with a good and performant fallback.

I'm not backpatching this for now - the problem isn't active in the back
branches, and we haven't backpatched barrier changes for
now. Additionally master looks entirely different than the back branches
due to the new atomics abstraction. It seems better to let this rest in
master, where the non-reentrancy actively causes a problem, and then
consider backpatching.

Found-By: Robert Haas
Discussion: 55626265(dot)3060800(at)dunslane(dot)net

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/1b468a131bd260c9041484f78b8580c7f232d580

Modified Files
--------------
src/backend/port/atomics.c | 21 +++++++++++++++++++--
1 file changed, 19 insertions(+), 2 deletions(-)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2015-06-26 15:28:29 pgsql: Reduce log level for background worker events from LOG to DEBUG1
Previous Message Robert Haas 2015-06-26 13:42:46 pgsql: Improve handling of CustomPath/CustomPlan(State) children.