From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Tomas Vondra <tv(at)fuzzy(dot)cz>, Simon Riggs <simon(at)2ndquadrant(dot)com>, pgsql-committers <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Generational memory allocator |
Date: | 2017-11-25 20:00:15 |
Message-ID: | 20171125200014.qbewtip5oydqsklt@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Hi,
On 2017-11-25 14:50:41 -0500, Tom Lane wrote:
> I wrote:
> > Tomas Vondra <tv(at)fuzzy(dot)cz> writes:
> >> BTW I also see these failures in hstore:
>
> >> ==15168== Source and destination overlap in memcpy(0x5d0fed0, 0x5d0fed0, 40)
> >> ==15168== at 0x4C2E00C: memcpy@@GLIBC_2.14 (vg_replace_strmem.c:1018)
> >> ==15168== by 0x15419A06: hstoreUniquePairs (hstore_io.c:343)
> >> ==15168== by 0x15419EE4: hstore_in (hstore_io.c:416)
>
> > Huh ...
>
> I tried to duplicate this on my RHEL6 workstation, and failed to,
> even though adding an assertion easily proves that the hstore
> regression test does exercise the case. So apparently the answer
> as to why skink isn't reporting this is just "not all versions of
> valgrind check it".
I suspect that the issue rather is that the compiler will sometimes
replace the memcpy() with an in-line member-by-member version. That'll
not be visible as a memcpy to valgrind.
> Meanwhile, skink has come back with a success as of 0f2458f, rendering
> the other mystery even deeper --- although at least this confirms the
> idea that the crashes we are seeing predate the generation.c patch.
Hm, wonder whether that could be optimization dependent too.
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-11-25 20:13:56 | Re: pgsql: Generational memory allocator |
Previous Message | Tom Lane | 2017-11-25 19:50:41 | Re: pgsql: Generational memory allocator |