Re: [COMMITTERS] pgsql: Allow Pin/UnpinBuffer to operate in a lockfree manner.

From: Andres Freund <andres(at)anarazel(dot)de>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Allow Pin/UnpinBuffer to operate in a lockfree manner.
Date: 2016-04-15 02:17:02
Message-ID: 20160415021702.rw6n725aqkfluf5w@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On 2016-04-13 23:31:21 -0700, Andres Freund wrote:
> I'm also putting up an animal with clang that uses
> CFLAGS='-std=c89 -Wc99-extensions -Werror=c99-extensions'
> which actually catches this.

Hm. Doing so I found the following in 9.3:

/home/andres/src/postgresql-9.3/src/bin/pg_dump/parallel.c:561:23: error: initializer for aggregate is not a compile-time constant [-Werror,-Wc99-extensions]
int pipefd[2] = {pipeMW[PIPE_READ], pipeWM[PIPE_WRITE]};
^~~~~~~~~~~~~~~~~

which is, afaics, the same class of issue we're hitting on master right
now. I apparently fixed that, via Robert, back in 59202fae0. I'd planned
to put up that animal (mylodon) for all branches, but given that that's
been unfixed for years I'm not sure. What do you think?

- Andres

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andres Freund 2016-04-15 02:28:41 pgsql: Make init_spin_delay() C89 compliant #2.
Previous Message Tom Lane 2016-04-14 23:42:56 pgsql: Fix core dump in ReorderBufferRestoreChange on alignment-picky p

Browse pgsql-hackers by date

  From Date Subject
Next Message David Rowley 2016-04-15 02:35:19 Re: Disallow unique index on system columns
Previous Message Tom Lane 2016-04-15 01:56:13 Re: Disallow unique index on system columns