pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Peter Geoghegan <pg@bowt.ie>
Cc: Andres Freund <andres@anarazel.de>
Cc: Noah Misch <noah@leadboat.com>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: AIX support - alignment issues
Date: Sat, 02 Jul 2022 15:22:23 -0400
Message-ID: <968410.1656789743@sss.pgh.pa.us> (raw)
In-Reply-To: <CAH2-WzndKXQQGh-7sCOTKL_87Kt918r0F2ztTAraeBCAC8v3OQ@mail.gmail.com>
References: <20220702183354.a6uhja35wta7agew@alap3.anarazel.de>
	<CAH2-WzndKXQQGh-7sCOTKL_87Kt918r0F2ztTAraeBCAC8v3OQ@mail.gmail.com>

Peter Geoghegan <pg@bowt.ie> writes:
> I tend to agree about dropping AIX. But I wonder if there is an
> argument against that proposal that doesn't rely on AIX being relevant
> to at least one user. Has supporting AIX ever led to the discovery of
> a bug that didn't just affect AIX?

Searching the commit log quickly finds

591e088dd

    datetime.c's parsing logic has assumed that strtod() will accept
    a string that looks like ".", which it does in glibc, but not on
    some less-common platforms such as AIX.

glibc's behavior is clearly not meeting the letter of the POSIX spec here.

a745b9365
    
    I'm not sure how we've managed not to notice this problem, but it
    seems to explain slow execution of the 017_shm.pl test script on AIX
    since commit 4fdbf9af5, which added a speculative "pg_ctl stop" with
    the idea of making real sure that the postmaster isn't there.  In the
    test steps that kill-9 and then restart the postmaster, it's possible
    to get past the initial signal attempt before kill() stops working
    for the doomed postmaster.  If that happens, pg_ctl waited till
    PGCTLTIMEOUT before giving up ... and the buildfarm's AIX members
    have that set very high.

Admittedly, this one is more about "slow" than about "AIX".

57b5a9646
    
    Most versions of tar are willing to overlook the missing terminator, but
    the AIX buildfarm animals were not. Fix by inventing a new kind of
    bbstreamer that just blindly adds a terminator, and using it whenever we
    don't parse the tar archive.

Another place where we failed to conform to relevant standards.    

b9b610577

    Fix ancient violation of zlib's API spec.
    
And another.

Now, it's certainly possible that AIX is the only surviving platform
that hasn't adopted bug-compatible-with-glibc interpretations of
POSIX.  But I think the standard is the standard, and we ought to
stay within it.  So I find value in these fixes.

			regards, tom lane





view thread (48+ messages)  latest in thread

Message-ID: <968410.1656789743@sss.pgh.pa.us>
Permalink:  ../968410.1656789743@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/968410.1656789743@sss.pgh.pa.us

 · 

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: pgsql-hackers@postgresql.org
  Cc: tgl@sss.pgh.pa.us, pg@bowt.ie, andres@anarazel.de, noah@leadboat.com
  Subject: Re: AIX support - alignment issues
  In-Reply-To: <968410.1656789743@sss.pgh.pa.us>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox