Re: initdb failure on Debian sid/mips64el in EventTriggerEndCompleteQuery

From: Christoph Berg <myon(at)debian(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: initdb failure on Debian sid/mips64el in EventTriggerEndCompleteQuery
Date: 2017-08-22 19:40:33
Message-ID: 20170822194033.k2qsa2v4xautfk46@msg.df7cb.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Re: Tom Lane 2017-08-13 <14517(dot)1502638417(at)sss(dot)pgh(dot)pa(dot)us>
> I suspect you could work around this with
>
> bool isCompleteQuery = (context <= PROCESS_UTILITY_QUERY);
> - bool needCleanup;
> + volatile bool needCleanup;
> bool commandCollected = false;
>
> If that fixes it, it's definitely a compiler bug. That function does
> not change needCleanup after the sigsetjmp call, so per POSIX it
> should not have to label the variable volatile. This is far from
> being the first such bug we've seen though.

In the meantime, gcc-7 is at version 7.2.0-1, so I gave 9.6 on
mips64el a new try. It's still failing at initdb time, and indeed
adding "volatile" makes initdb proceed, but then the rest of the
testsuite fails in various ways:

DETAIL: Failed process was running: CREATE TABLE enumtest_child (parent rainbow REFERENCES enumtest_parent);
DETAIL: Failed process was running: create table trigtest2 (i int references trigtest(i) on delete cascade);
DETAIL: Failed process was running: CREATE TABLE trunc_b (a int REFERENCES truncate_a);
DETAIL: Failed process was running: CREATE SCHEMA evttrig
CREATE TABLE one (col_a SERIAL PRIMARY KEY, col_b text DEFAULT 'forty two')
CREATE INDEX one_idx ON one (col_b)
CREATE TABLE two (col_c INTEGER CHECK (col_c > 0) REFERENCES one DEFAULT 42);

Hopefully the compiler gets fixed soonish on mips64el...

Thanks for the analysis,
Christoph

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Alik Khilazhev 2017-08-22 20:09:38 Re: [WIP] Zipfian distribution in pgbench
Previous Message Peter Eisentraut 2017-08-22 18:47:18 Re: coverage analysis improvements