Page MenuHomeFreeBSD
Differential D15557

Make vadvise compat freebsd11.
ClosedPublic
Actions

Authored by brooks on May 24 2018, 5:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 5, 12:34 AM
Unknown Object (File)
Thu, May 2, 12:10 AM
Unknown Object (File)
Mon, Apr 29, 9:36 PM
Unknown Object (File)
Mon, Apr 29, 1:57 AM
Unknown Object (File)
Mar 31 2024, 2:44 AM
Unknown Object (File)
Mar 9 2024, 7:17 AM
Unknown Object (File)
Feb 29 2024, 10:43 PM
Unknown Object (File)
Jan 29 2024, 3:35 AM
View All 46 Files
Subscribers
imp

Details

Reviewers
kib
emaste
Commits
rS334223: Make vadvise compat freebsd11.
Summary

The vadvise syscall (aka ovadvise) is undocumented and has always been
implmented as returning EINVAL. Put the syscall under COMPAT11 and
provide a userspace implementation.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16814
Build 16697: arc lint + arc unit

Event Timeline

brooks created this revision.May 24 2018, 5:16 PM
Harbormaster completed remote builds in B16814: Diff 42939.May 24 2018, 5:16 PM
kib added a comment.May 24 2018, 5:35 PM

Could you, please, remove the generated files from the review and from the commit ?

brooks updated this revision to Diff 42943.May 24 2018, 5:52 PM
  • Remove generated files from review.
Harbormaster completed remote builds in B16818: Diff 42943.May 24 2018, 5:52 PM
emaste added a comment.May 24 2018, 5:55 PM

Add a dependency cleanup in Makefile.inc1 after commit also?

lib/libc/sys/vadvise.c
5

holder?

brooks updated this revision to Diff 42945.May 24 2018, 6:10 PM
  • Actually name copyright holder.
Harbormaster completed remote builds in B16820: Diff 42945.May 24 2018, 6:10 PM
brooks added a comment.May 24 2018, 6:11 PM
In D15557#328428, @emaste wrote:

Add a dependency cleanup in Makefile.inc1 after commit also?

That's the plan.

brooks marked an inline comment as done.May 24 2018, 6:11 PM
kib added inline comments.May 24 2018, 6:18 PM
lib/libc/sys/vadvise.c
39

Are __sys_vadvise and _vadvise symbols used by anything in the base system ? If no, then they can be removed, since we do not provide stable ABI in FreeBSD private namespace.

sys/vm/vm_unix.c
240–241

I believe this structure definition is useless.

brooks updated this revision to Diff 42949.May 24 2018, 6:35 PM
  • Nothing uses _vadvise() or __sys_vadvise so don't provide them.
  • Don't document ignored struct freebsd11_vadvise_args.
Harbormaster completed remote builds in B16823: Diff 42949.May 24 2018, 6:35 PM
kib accepted this revision.May 24 2018, 7:03 PM
This revision is now accepted and ready to land.May 24 2018, 7:03 PM
Closed by commit rS334223: Make vadvise compat freebsd11. (authored by brooks). · Explain WhyMay 25 2018, 8:40 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptMay 25 2018, 8:40 PM

Revision Contents
Changeset List

PathSize
lib/
libc/
sys/
1 line
47 lines
sys/
compat/
freebsd32/
2 lines
2 lines
2 lines
22 lines
3 lines
kern/
2 lines
2 lines
3 lines
22 lines
sys/
2 lines
2 lines
10 lines
vm/
14 lines

Diff 42939

View Options

lib/libc/sys/Makefile.inc

Loading...
View Options

lib/libc/sys/vadvise.c

Loading...
View Options

sys/compat/freebsd32/freebsd32_syscall.h

Loading...
View Options

sys/compat/freebsd32/freebsd32_syscalls.c

Loading...
View Options

sys/compat/freebsd32/freebsd32_sysent.c

Loading...
View Options

sys/compat/freebsd32/freebsd32_systrace_args.c

Loading...
View Options

sys/compat/freebsd32/syscalls.master

Loading...
View Options

sys/kern/init_sysent.c

Loading...
View Options

sys/kern/syscalls.c

Loading...
View Options

sys/kern/syscalls.master

Loading...
View Options

sys/kern/systrace_args.c

Loading...
View Options

sys/sys/syscall.h

Loading...
View Options

sys/sys/syscall.mk

Loading...
View Options

sys/sys/sysproto.h

Loading...
View Options

sys/vm/vm_unix.c

Loading...