The helper saved some code churn initially, but having multiple
interfaces is sub-optimal.
Remove the old kern_mmap.
Rename kern_mmap_req to kern_mmap
There is no need for the suffix now that there is only one interface.
Paths
| Differential D28292 Authored by brooks on Jan 23 2021, 12:25 AM.
Details
Summary The helper saved some code churn initially, but having multiple Remove the old kern_mmap. Rename kern_mmap_req to kern_mmap There is no need for the suffix now that there is only one interface.
Diff Detail
Event Timelinebrooks created this revision. Harbormaster completed remote builds in B36418: Diff 82747.Jan 23 2021, 12:25 AM2021-01-23 00:25:08 (UTC+0) Harbormaster completed remote builds in B36419: Diff 82748.Jan 23 2021, 12:33 AM2021-01-23 00:33:27 (UTC+0) Comment Actions I've done this cleanup in CheriBSD on the path to some local cleanups. I've chosen to use compound literals because they are IMO tidier than single use temporary variables when the initialization values aren't available at the top of the file. They aren't widely used in the tree so now is probably the time to discuss style. The one aspect of the current style I debated the most was if the &(struct foo){ should be on it's own line or on the line above. I felt splitting it out worked well with the other indentation. This revision is now accepted and ready to land.Jan 23 2021, 11:43 AM2021-01-23 11:43:13 (UTC+0) This revision now requires review to proceed.Jan 25 2021, 9:24 PM2021-01-25 21:24:38 (UTC+0) Harbormaster completed remote builds in B36481: Diff 82877.Jan 25 2021, 9:24 PM2021-01-25 21:24:43 (UTC+0) This revision is now accepted and ready to land.Jan 25 2021, 9:49 PM2021-01-25 21:49:58 (UTC+0) Closed by commit rG7a1591c1b625: Rename kern_mmap_req to kern_mmap (authored by brooks). · Explain WhyJan 25 2021, 9:59 PM2021-01-25 21:59:08 (UTC+0) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 82877 sys/compat/cloudabi/cloudabi_mem.c
sys/compat/freebsd32/freebsd32_misc.c
sys/compat/linux/linux_mmap.c
sys/sys/syscallsubr.h
|