Re: pgsql: Fix crash when ALTER TABLE recreates indexes on partitions

From: Andres Freund <andres(at)anarazel(dot)de>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Fix crash when ALTER TABLE recreates indexes on partitions
Date: 2018-07-09 20:39:15
Message-ID: 20180709203915.fz44v4odlzmd27v5@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On 2018-07-04 20:33:05 -0700, Andres Freund wrote:
> On 2018-07-04 13:15:19 -0400, Alvaro Herrera wrote:
> > On 2018-Jun-30, Tom Lane wrote:
> >
> > > Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:
> > > > Fix crash when ALTER TABLE recreates indexes on partitions
> > >
> > > So ... buildfarm member skink has been reporting a valgrind failure
> > > during initdb since this patch went in. However, I'm unable to reproduce
> > > such a failure here, and it's less than obvious how this particular patch
> > > could have caused a problem, and skink's report is pretty useless because
> > > it's providing only a numeric stack trace. Thoughts?
> >
> > Actually, older branches are failing in the same way, so it's not my
> > patch that caused it to fail. Something must have changed in the system ...
> > Andres?
>
> I spent a fair amount of time trying to dig into this. It appears to be
> related to a binutils update. Everything built with a binutils version <
> 2.30.90.20180627-1 (in my case 2.30-22) works correctly with
> valgrind. If rebuilt after an upgrade it doesn't. Still trying to
> narrow down further.

FWIW, I've reported this as a bug to debian. And figured out in the
course of collecting the necessary information that forcing gcc to use
the gold linker works around the issue. Started a run on skink with
that, hopefully that resolves the issue.

Greetings,

Andres Freund

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2018-07-09 21:23:55 pgsql: Prevent accidental linking of system-supplied copies of libpq.so
Previous Message Peter Geoghegan 2018-07-09 19:12:42 Re: pgsql: Add wait event for fsync of WAL segments

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2018-07-09 20:53:14 Re: no partition pruning when partitioning using array type
Previous Message Markus Wanner 2018-07-09 20:38:12 Re: How can we submit code patches that implement our (pending) patents?