Re: kevent latch paths don't handle postmaster death well
Thomas Munro <thomas.munro@gmail.com>
From: Thomas Munro <thomas.munro@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Andres Freund <andres@anarazel.de>, Heikki Linnakangas <hlinnaka@iki.fi>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2020-10-15T05:42:01Z
Lists: pgsql-hackers
On Thu, Oct 15, 2020 at 12:55 PM Thomas Munro <thomas.munro@gmail.com> wrote: > On Thu, Oct 15, 2020 at 12:14 PM Tom Lane <tgl@sss.pgh.pa.us> wrote: > > I did not try to test it, but there's code that purports to handle that > > in latch.c, ~ line 1150, and the behavior it's expecting mostly agrees > > with what I read in the macOS kevent man page. One thing I'd suggest > > is that EACCES probably needs to be treated as "postmaster already dead", > > too, in case the PID is now owned by another user ID. > > Good point. I'll push that change later today. I tried to test this on my system but it seems like maybe FreeBSD can't really report EACCES for EVFILT_PROC. From the man page and a quick inspection of the source, you only have to be able to "see" the process, and if you can't I think you'll get ESRCH, so EACCES may be for other kinds of filters. I don't currently have any Apple gear to hand, but its man page uses the same language, but on the other hand I do see EACCES in filt_procattach() in the darwin-xnu sources on github so I guess you can reach this case and get an ugly ereport (hopefully followed swiftly by a proc_exit() from the next wait on one of the long lived WESs, or a FATAL if this was the creation of one of those). Fixed. Thanks!
Commits
-
Handle EACCES errors from kevent() better.
- 47522ee00ddb 13.1 landed
- 70516a178ad0 14.0 landed
-
Make WL_POSTMASTER_DEATH level-triggered on kqueue builds.
- e0950135ae5d 13.1 landed
- b94109ce375b 14.0 landed