public inbox for [email protected]
help / color / mirror / Atom feedFrom: Peter Eisentraut <[email protected]>
To: Srirama Kucherlapati <[email protected]>
To: Andres Freund <[email protected]>
To: AIX PG user <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: AIX support
Date: Wed, 15 Oct 2025 20:15:50 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <SJ4PPFB8177832600E5AAE4214587DC3F7BDBE8A@SJ4PPFB81778326.namprd15.prod.outlook.com>
References: <CY5PR11MB63928CC05906F27FB10D74D0FD322@CY5PR11MB6392.namprd11.prod.outlook.com>
<ko5nio3iyt7fonvwqqgs6gddzyzlad7qm5v6zo72h753jmitwy@llz3bghejzig>
<175863756865.1038.1316404745203068811.pgcf@coridan.postgresql.org>
<3xsqvatpnbb7wd3vdxdygzh6s2bbxn3ulmarli67x5u7hekvei@3it2vxprjglu>
<SJ4PPFB817783263D506AAE4C480F5B2866DB1DA@SJ4PPFB81778326.namprd15.prod.outlook.com>
<SJ4PPFB8177832600E5AAE4214587DC3F7BDBE8A@SJ4PPFB81778326.namprd15.prod.outlook.com>
On 15.10.25 17:58, Srirama Kucherlapati wrote:
> Main AIX changes patch - 0001-Changes-to-add-support-for-
> AIX.v9.pg19.patch (previously submitted one)
>
> Meson AIX changes patch - 0001-AIX-changes-for-meson-build.patch
>
> Looking forward to hearing your thoughts on these changes.
>
> Your feedback on these updates is highly valued.
The largest problem I see in this patch set is that you are making
unexplained changes in src/include/storage/s_lock.h. The code that you
are changing does not only affect AIX but also PPC with other operating
systems. AFAICT, the change you are proposing does not correspond to
the commit that removed AIX support.
Maybe the proposed changes are beneficial for other operating systems as
well. But in that case, propose this as a separate patch in a separate
thread, with supporting information (and probably benchmarks in this case).
If not, then you need to find a way to make this work on AIX without
materially changing the code that exists for already-supported operating
systems.
view thread (73+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: AIX support
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox