Re: Build warning with meson and dtrace on Fedora 43

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andrey Rachitskiy <pl0h0yp1(at)gmail(dot)com>
Cc: Peter Eisentraut <peter(at)eisentraut(dot)org>, Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Build warning with meson and dtrace on Fedora 43
Date: 2026-07-29 04:14:38
Message-ID: 491551.1785298478@sss.pgh.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Andrey Rachitskiy <pl0h0yp1(at)gmail(dot)com> writes:
> Hi, Laurenz!
> Thanks for the report. I tried to fix the patch in the attachment.

BF animals caiman, midge, tayra, timberworm are all complaining about
this. In keeping with meson's general habit of verbose yet largely
useless configuration reporting, it's impossible to tell exactly what
OS versions those are running, though their possibly-outdated BF
metadata claims recent Red Hat versions. However, I also reproduced
the warning with an autoconf build on current Fedora 43 / x86_64.

> Commits ca326e903d and 1f8c504e308 widened the byte-count arguments of
> smgr__md__read__done and smgr__md__write__done to "long long int".
> SystemTap's dtrace(1) pyparsing grammar rejects that multi-word type and
> falls back with a warning (misreported near the previous probe). Use
> "long" instead, which matches ssize_t on our LP64 DTrace platforms and
> is already used elsewhere in probes.d.

This seems to me to be a fundamentally broken, hence short-lived
workaround. I think we need another way. As you mention upthread,
we document that system-header types such as ssize_t should not be
used in probes.d because "they cause compilation errors on macOS".
But I tried it just now and it compiles fine on current Tahoe.
Maybe that note is obsolete?

regards, tom lane

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Smith 2026-07-29 04:22:53 Re: A new C function `get_partition_root`.
Previous Message shihao zhong 2026-07-29 04:11:37 [PATCH] Add planner support function to two-argument regexp_like()