public inbox for [email protected]
help / color / mirror / Atom feedFrom: Tom Lane <[email protected]>
To: Greg Burd <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Cc: Thomas Munro <[email protected]>
Subject: Re: Areas for Solaris support modernization
Date: Mon, 02 Mar 2026 17:15:59 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
<[email protected]>
<[email protected]>
"Greg Burd" <[email protected]> writes:
> On Sun, Mar 1, 2026, at 9:50 PM, Tom Lane wrote:
>>> Also, while playing with said local OpenIndiana image, I noticed
>>> that ps_status.c isn't working: "ps auxww" shows all the child
>>> processes with the same command line as the postmaster. I thought
>>> maybe we'd diked out something important in d2ea2d310, but none
>>> of the code removed there claims to apply to Solaris. So maybe
>>> it never worked on Solaris?
>>
>> Anyway, here's a slightly cleaned-up reversion of the relevant
>> bits of d2ea2d310, with PS_USE_CHANGE_ARGV now selected by
>> "defined(__sun)" not the previous logic
>> "(defined(BSD) || defined(__hurd__)) && !defined(__darwin__)".
> I've started a build/test on "icarus" with your patch applied. I'll let you know how it goes, hopefully that won't take as long now.
Oh, now THIS is interesting: I just updated to current OpenIndiana
(previously I was on Dec-2025 or thereabouts), and now ps_status
seems to be working *without* any patch! It works with the patch
too, but now I'm thinking I hit some bug in their "ps" that got
fixed and perhaps wasn't of long standing.
So maybe we don't need this patch. I'd be interested to know what
you see on whatever Solaris boxen you have. Note that the regression
tests will not reveal anything --- you need to eyeball what "ps auxww"
shows for a running postmaster and its children. With current HEAD
and all-default settings, I see something like
$ ps auxww | grep tgl
...
tgl 17855 0.1 0.6 211008 21792 pts/2 S 22:07:24 0:00 postgres -F
tgl 17860 0.0 0.1 211056 3384 ? S 22:07:24 0:00 postgres: background writer
tgl 17856 0.0 0.1 211056 3480 ? S 22:07:24 0:00 postgres: io worker 0
tgl 17857 0.0 0.1 211056 3480 ? S 22:07:24 0:00 postgres: io worker 1
tgl 17858 0.0 0.1 211040 3112 ? S 22:07:24 0:00 postgres: io worker 2
tgl 17859 0.0 0.1 211072 3288 ? S 22:07:24 0:00 postgres: checkpointer
tgl 17862 0.0 0.1 212080 4148 ? S 22:07:24 0:00 postgres: walwriter
tgl 17863 0.0 0.2 212528 6224 ? S 22:07:24 0:00 postgres: autovacuum launcher
tgl 17864 0.0 0.2 212512 5940 ? S 22:07:24 0:00 postgres: logical replication launcher
...
if it's working, and a bunch of identical command lines if not.
regards, tom lane
view thread (7+ 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]
Subject: Re: Areas for Solaris support modernization
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