Page MenuHomeFreeBSD
Differential D15539

Don't implement break(2) at all on aarch64 and riscv.
ClosedPublic
Actions

Authored by brooks on May 23 2018, 9:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 4, 4:23 PM
Unknown Object (File)
Feb 18 2025, 1:40 PM
Unknown Object (File)
Feb 17 2025, 3:49 AM
Unknown Object (File)
Jan 21 2025, 12:46 AM
Unknown Object (File)
Jan 19 2025, 9:17 PM
Unknown Object (File)
Nov 16 2024, 9:31 AM
Unknown Object (File)
Oct 3 2024, 5:48 AM
Unknown Object (File)
Oct 1 2024, 10:40 PM
View All 38 Files
Subscribers
imp

Details

Reviewers
emaste
br
andrew
Commits
rS334168: Don't implement break(2) at all on aarch64 and riscv.
Summary

This should have been done when they were removed from libc, but was
overlooked in the runup to 11.0.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
head/
sys/
vm/
9 lines

Diff 42937

View Options

head/sys/vm/vm_unix.c

Show First 20 Lines • Show All 47 Lines • ▼ Show 20 Lines
.Fa fd .Fa fd
is a descriptor to be operated on by is a descriptor to be operated on by
.Fa cmd .Fa cmd
as described below. as described below.
Depending on the value of Depending on the value of
.Fa cmd , .Fa cmd ,
.Fn fcntl .Fn fcntl
can take an additional third argument can take an additional third argument
.Fa "long arg" . .Fa arg .
jhbUnsubmitted
Done
Inline Actions

Maybe just say the type of the 3rd arg depends on the operation (and document the type for each operation that takes the third arg) instead of trying to assume it's a single type? In CheriBSD we have a wrapper in libc to deal with this and have fcntl() as a truly var-args function. On existing platforms var-args is close enough to work for passing syscall arguments, but not all ABIs require this. (var-args are passed as a block in a bounded pointer on CHERI platforms and not just a continuation of register arguments).

jhb: Maybe just say the type of the 3rd arg depends on the operation (and document the type for each…
kibAuthorUnsubmitted
Done
Inline Actions

I pushed everything but this change, the review is updated with the man page changes.

kib: I pushed everything but this change, the review is updated with the man page changes.
Unless otherwise noted below for a specific operation,
emasteUnsubmitted
Done
Inline Actions

maybe "with type dependent on the specific operation"

or maybe

can take an additional third argument
.Fa arg.
Unless otherwise noted below for a specific operation,
.Fa arg
has type
.Vt int .
emaste: maybe "with type dependent on the specific operation" or maybe ``` can take an additional…
.Fa arg
has type
.Vt int .
.Bl -tag -width F_DUP2FD_CLOEXEC .Bl -tag -width F_DUP2FD_CLOEXEC
.It Dv F_DUPFD .It Dv F_DUPFD
Return a new descriptor as follows: Return a new descriptor as follows:
.Pp .Pp
.Bl -bullet -compact -offset 4n .Bl -bullet -compact -offset 4n
.It .It
Lowest numbered available descriptor greater than or equal to Lowest numbered available descriptor greater than or equal to
.Fa arg . .Fa arg .
▲ Show 20 Lines • Show All 121 Lines • ▼ Show 20 Lines
.It Dv F_GET_SEALS .It Dv F_GET_SEALS
Get seals associated with the file, if the underlying filesystem supports seals. Get seals associated with the file, if the underlying filesystem supports seals.
.It Dv F_ISUNIONSTACK .It Dv F_ISUNIONSTACK
Check if the vnode is part of a union stack (either the "union" flag from Check if the vnode is part of a union stack (either the "union" flag from
.Xr mount 2 .Xr mount 2
or unionfs). or unionfs).
This is a hack not intended to be used outside of libc. This is a hack not intended to be used outside of libc.
.It Dv F_KINFO .It Dv F_KINFO
Fills a Fills a
markjUnsubmitted
Done
Inline Actions
.It Dv F_KINFO
- Fills
+ Fills a
.Vt struct kinfo_file
markj:
.Vt struct kinfo_file .Vt struct kinfo_file
for the file referenced by the specified file descriptor. for the file referenced by the specified file descriptor.
The The
.Fa arg .Fa arg
argument should point to the storage for argument should point to the storage for
.Vt struct kinfo_file . .Vt struct kinfo_file .
The The
.Va kf_structsize .Va kf_structsize
▲ Show 20 Lines • Show All 78 Lines • ▼ Show 20 Lines
Seals are on a per-inode basis and require support by the underlying filesystem. Seals are on a per-inode basis and require support by the underlying filesystem.
If the underlying filesystem does not support seals, If the underlying filesystem does not support seals,
.Dv F_ADD_SEALS .Dv F_ADD_SEALS
and and
.Dv F_GET_SEALS .Dv F_GET_SEALS
will fail and return will fail and return
.Dv EINVAL . .Dv EINVAL .
.Pp .Pp
Several commands are available for doing advisory file locking; Several operations are available for doing advisory file locking;
emasteUnsubmitted
Done
Inline Actions

Is "command" or "operation" the better term, in general? I think we should be consistent between this sentence and the added text above/below

emaste: Is "command" or "operation" the better term, in general? I think we should be consistent…
they all operate on the following structure: they all operate on the following structure:
.Bd -literal .Bd -literal
struct flock { struct flock {
off_t l_start; /* starting offset */ off_t l_start; /* starting offset */
off_t l_len; /* len = 0 means until end of file */ off_t l_len; /* len = 0 means until end of file */
pid_t l_pid; /* lock owner */ pid_t l_pid; /* lock owner */
short l_type; /* lock type: read/write, etc. */ short l_type; /* lock type: read/write, etc. */
short l_whence; /* type of l_start */ short l_whence; /* type of l_start */
int l_sysid; /* remote system id or zero for local */ int l_sysid; /* remote system id or zero for local */
}; };
.Ed .Ed
These advisory file locking operations take a pointer to
emasteUnsubmitted
Done
Inline Actions

These commands take a pointer to struct flock as the third argument arg.
or
These advisory file locking commands take a pointer to struct flock as the third argument arg.

emaste: These commands take a pointer to struct flock as the third argument arg. or These advisory file…
emasteUnsubmitted
Done
Inline Actions

Maybe "operations" here too for consistency?
But from the context this one is clear so I don't mind either way.

emaste: Maybe "operations" here too for consistency? But from the context this one is clear so I don't…
.Vt struct flock
as the third argument
.Fa arg .
The commands available for advisory record locking are as follows: The commands available for advisory record locking are as follows:
.Bl -tag -width F_SETLKWX .Bl -tag -width F_SETLKWX
.It Dv F_GETLK .It Dv F_GETLK
Get the first lock that blocks the lock description pointed to by the Get the first lock that blocks the lock description pointed to by the
third argument, third argument,
.Fa arg , .Fa arg ,
taken as a pointer to a taken as a pointer to a
.Fa "struct flock" .Fa "struct flock"
▲ Show 20 Lines • Show All 477 Lines • Show Last 20 Lines