Re: Areas for Solaris support modernization
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: "Greg Burd" <greg@burd.me>
Cc: "PostgreSQL Hackers" <pgsql-hackers@lists.postgresql.org>,
"Thomas Munro" <thomas.munro@gmail.com>
Date: 2026-03-02T22:15:59Z
Lists: pgsql-hackers
"Greg Burd" <greg@burd.me> 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
Commits
-
Switch the semaphore API on Solaris to unnamed POSIX.
- 0123ce131fca 19 (unreleased) landed
-
Fix aclitemout() to work during early bootstrap.
- 2eb87345e18e 19 (unreleased) landed