Re: Shmem allocated wrong for custom cumulative stats

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Shmem allocated wrong for custom cumulative stats
Date: 2026-04-06 03:14:06
Message-ID: adMk_lWbnz3HDOA8@paquier.xyz
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Apr 06, 2026 at 11:36:21AM +0900, Michael Paquier wrote:
> On Mon, Apr 06, 2026 at 09:55:14AM +0900, Michael Paquier wrote:
>> Right, down to v18 where this has been introduced. That's my bug, so
>> I'd be OK to take care of it myself, if you are OK with that of
>> course.
>
> Note: something is wrong with -m32, for both patches. Digging into
> that..

And I have been puzzled for a few minutes here, trying to figure out
if this was something in v18 or something with the new shmem routines.
It is nothing of the kind: test_custom_stats has been underestimating
its shared_size, using PgStat_StatCustomFixedEntry instead of
PgStatShared_CustomFixedEntry. Interesting copy-pasto, HEAD-only,
second bug.

The attached is working correctly. The v18 flavor is slightly
simpler.
--
Michael

Attachment Content-Type Size
v3-0001-Use-the-allocated-space-properly-for-custom--HEAD.patch text/plain 2.1 KB
v3-0001-Use-the-allocated-space-properly-for-custom-s-v18.patch text/plain 1.1 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Xuneng Zhou 2026-04-06 03:26:59 Re: Implement waiting for wal lsn replay: reloaded
Previous Message Richard Guo 2026-04-06 03:06:44 Re: Eager aggregation, take 3