Page MenuHomeFreeBSD
Differential D57935

pdopenpid(2)/pddupfd(2)/execblock report
ClosedPublic
Actions

Authored by kib on Mon, Jun 29, 12:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jul 9, 2:02 PM
Unknown Object (File)
Thu, Jul 9, 9:45 AM
Unknown Object (File)
Thu, Jul 9, 2:02 AM
Unknown Object (File)
Wed, Jul 8, 11:02 PM
Unknown Object (File)
Tue, Jul 7, 10:41 PM
Unknown Object (File)
Mon, Jul 6, 8:18 PM
Unknown Object (File)
Mon, Jul 6, 5:10 AM
Unknown Object (File)
Sun, Jul 5, 11:06 PM
View All Files
Subscribers
gperciva_tarsnap.com

Details

Reviewers
jrm
salvadore
Group Reviewers
status
Commits
R9:2ac80023518f: pdopenpid(2)/pddupfd(2)/execblock report

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib created this revision.Mon, Jun 29, 12:23 AM
Herald added a reviewer: salvadore. · View Herald TranscriptMon, Jun 29, 12:23 AM
kib requested review of this revision.Mon, Jun 29, 12:23 AM
gperciva_tarsnap.com added a subscriber: gperciva_tarsnap.com.Tue, Jun 30, 7:18 PM
gperciva_tarsnap.com added inline comments.
website/content/en/status/report-2026-04-2026-06/pdopenpid.adoc
10
11
22
23
kib marked 4 inline comments as done.Tue, Jun 30, 10:40 PM
kib added inline comments.
website/content/en/status/report-2026-04-2026-06/pdopenpid.adoc
23

I do not see why 'kernels' should be plural there.

kib updated this revision to Diff 181127.Tue, Jun 30, 10:40 PM
kib marked an inline comment as done.

Handle notes

gperciva_tarsnap.com added inline comments.Wed, Jul 1, 2:13 AM
website/content/en/status/report-2026-04-2026-06/pdopenpid.adoc
23

I went back and forth about it making it plural or not, but either way it needs some modification. Pick:

Kernels typically check...

or

The kernel typically checks...

Granted, now that I'm looking at it in more detail, I'm not certain "typically" is required. Does a kernel sometimes skip that check? If so, keep "typically"; if the check always happens, I recommend removing "typically".

kib updated this revision to Diff 181138.Wed, Jul 1, 2:16 AM
kib marked an inline comment as done.

"The kernel checks ..."

salvadore accepted this revision.Sun, Jul 12, 1:21 PM

Approved once the few small nits are fixed, thanks!

website/content/en/status/report-2026-04-2026-06/pdopenpid.adoc
7–8

+ sign misplaced

17
20
32

No need to reference the manual page again, but if you really wanted to do so please use the man macro

This revision is now accepted and ready to land.Sun, Jul 12, 1:21 PM
kib marked 4 inline comments as done.Sun, Jul 12, 2:23 PM
kib added inline comments.
website/content/en/status/report-2026-04-2026-06/pdopenpid.adoc
20

There is no fget_remote(9) man page.

Closed by commit R9:2ac80023518f: pdopenpid(2)/pddupfd(2)/execblock report (authored by kib). · Explain WhySun, Jul 12, 2:25 PM
This revision was automatically updated to reflect the committed changes.
kib marked an inline comment as done.
kib added a commit: R9:2ac80023518f: pdopenpid(2)/pddupfd(2)/execblock report.

Revision Contents
Changeset List

PathSize
website/
content/
en/
status/
report-2026-04-2026-06/
33 lines

Diff 181832

View Options

website/content/en/status/report-2026-04-2026-06/pdopenpid.adoc

Loading...