Reserve gaps in syscall numbers for local use

[brooks]

Summary

It is best for auditing of syscalls.master if we only append to the
file. Reserving unimplemented system call numbers for local use makes
this policy and provides a large set of syscall numbers FreeBSD
derivatives can use without risk of conflict.

While here GC some obsolete and irrelevent comments.

[brooks]

• Reclaim never-implemented or foreign syscalls

[kib]

Can we split the cleanup of unimplemented syscalls entries and policy statement? I have no objections against former, but IMO the new policy is somewhat wide open and strange. Lets discuss it separately, not blocking the good cleanup.

[brooks]

In D27988#625257, @kib wrote:

Can we split the cleanup of unimplemented syscalls entries and policy statement? I have no objections against former, but IMO the new policy is somewhat wide open and strange. Lets discuss it separately, not blocking the good cleanup.

All the UNIMPL pdwait4 -> UNIMPL nosys changes are already in a separate commit so I could easy do that separately. It wouldn't be hard to pull the nonsensical comment removal into that commit if that makes sense.

[kib]

In D27988#626098, @brooks wrote:

In D27988#625257, @kib wrote:

Can we split the cleanup of unimplemented syscalls entries and policy statement? I have no objections against former, but IMO the new policy is somewhat wide open and strange. Lets discuss it separately, not blocking the good cleanup.

All the UNIMPL pdwait4 -> UNIMPL nosys changes are already in a separate commit so I could easy do that separately. It wouldn't be hard to pull the nonsensical comment removal into that commit if that makes sense.

You mean comments from inside the table? Ok.

[jhb]

sys/kern/syscalls.master

88

I wonder if we want to be slightly more explicit? For example, I assume that OBSOL syscalls are not free for local use, but only UNIMPL noses? It might be nice if we adopted a consistent way of naming the gaps that we could use here to describe them (e.g. the loadable syscalls range all use lkmnosys which is a clear marker). It could be as simple as a new localnosys even that we could use in place of nosys and then this comment could refer to that?

[jrtc27]

sys/kern/syscalls.master

2830

AUE_NULL?

[brooks]

• Rebase and split into two commits
  • Remove documentation of unimplemented syscalls
  • Reserve gaps in syscall numbers for local use

[brooks]

sys/kern/syscalls.master

88–89

I think UNIMPL nosys is sufficiently documentary if we call it out explicitly. In practice OBSOL are likely fine for local use where people are building their own code, but I don't think we'd want to call that out. How about this text instead?

[kib]

Again, why not push cleanup separately.

I prefer that all reserved entries we provide for third-party use were explicitly marked, either by local comment, or by some variant of UNIMPL type, for instance RSRVD.

[brooks]

In D27988#633155, @kib wrote:

Again, why not push cleanup separately.

It is. See https://github.com/brooksdavis/freebsd/tree/D27988-reserve-syscalls

I prefer that all reserved entries we provide for third-party use were explicitly marked, either by local comment, or by some variant of UNIMPL type, for instance RSRVD.

Alright, I'm convinced. I'll go with RSRVD and rework the second commit.

[brooks]

• syscalls.master: Add a new syscall type: RESERVED
• Reserve gaps in syscall numbers for local use

[brooks]

• Update comment saying to add syscalls at the bottom

[brooks]

I've updated the diff (it's three commits). I ended up using RESERVED since the only thing a long name causes issues with is alignment of the comment at the top of syscalls.master.

[kevans]

sys/tools/makesyscalls.lua

1123 ↗

(On Diff #82864)

I have a slight preference for handle_reserved to match tolower(flag_name) in case we eventually try to streamline this section and avoid the (admittedly really minor) additional formality for adding a flag with a new handler. Ideally, a new flag would just have to be defined in known_flags and either: added to one of ncompatflags/compatflags OR just write the handler, along with any less-trivial semantics above.

[jhb]

I'll defer to @kevans on the lua bits, but I like the RESERVED annotation much better, thanks!

[brooks]

sys/tools/makesyscalls.lua

1123 ↗

(On Diff #82864)

oops, that's a leftover from implementing RSRVD before editing syscalls.master and thinking the result was silly, will fixed.

[brooks]

• Rebase
• s/rsrvd/reserved in makesyscalls.lua

[kevans]

Lua changes LGTM

[kib]

sys/kern/syscalls.master

35

Might be add that use of that entries should not be committed into FreeBSD repo?