In D41379#942527, @markj wrote:

Can you use our funlinkat() instead of keeping track of the directory descriptor?

Update after rebase

Update after rebase

Update after rebase

Update after rebase

Update after rebase

Address Mark's comments

Update after rebase

Update after rebase

Update after rebase

Update after rebase

Update after rebase

Update after rebase

Update after rebase

Address Mark's comments

Address Mark's suggestions

Update after rebase

Update after rebase

Update after rebase

Address Mark's suggestions

In D41361#942459, @markj wrote:

Did you test compiling with WITHOUT_INET_SUPPORT=or WITHOUT_INET6_SUPPORT= defined? I suspect it will fail, because the #include <netinet/in.h> at the beginning of syslogd is still conditional on INET or INET6 being defined. While the ifdefs are ugly, I'm not sure we can do much better without adding some dummy definitions to replace those in netinet/in.h.

• Remove style(9) changes
• Check for blocking fd

Fix spelling mistakes and update to-do list

In D41385#942540, @markj wrote:

I'm not a big fan of reordering functions like this, since that makes git blame quite useless for little gain. In general this kind of commit should come at the end of a patch stack if at all. If it's too painful to reorder it then ok, just something to keep in mind for next time.

Process descriptors for piped processes are registered with kevent() in this patch:

In D41362#942043, @emaste wrote:

The needdofsync global variable was initialized as true, when it should be false. Fix this.

We may want to just leave the 0/false globals without explicit initialization.

Update diff with context

Update diff with context.

Update diff with context

Update diff with context

Update diff with context.

In D41362#941850, @emaste wrote:

In future when uploading patches via the web interface please generate them with -U99999 so that Phabricator can show full context (see https://wiki.freebsd.org/Phabricator for details)

The needdofsync global variable was initialized as true, when it should be false. Fix this.

Fix formatting issue in license text

Move SIG_FOREACH() into an #ifdef _KERNEL.

Fix formatting in HISTORY section thanks to @gbe.

I would like to get this patch committed before the 14.0 freeze in August. I am personally in favor of making these syscalls, but I'd like some notice if we're not going this route.

Add brief comments explaining each of the tfd_jumped macros.

Use cap_svflags instead of cap_flags when determining kernel ABI with syscallabi().

Change NI_LCF_STRICTRELATIVE to NI_LCF_STRICTREL where applicable.