Page MenuHomeFreeBSD
Differential D42707

getpagesizes(3): drop support for kernels before 9.0
ClosedPublic
Actions

Authored by brooks on Nov 22 2023, 1:08 AM.
Tags
None
Referenced Files
F102907257: D42707.diff
Mon, Nov 18, 2:34 PM
F102876387: D42707.id.diff
Mon, Nov 18, 8:00 AM
Unknown Object (File)
Wed, Nov 13, 1:25 PM
Unknown Object (File)
Oct 2 2024, 6:19 PM
Unknown Object (File)
Sep 30 2024, 4:13 PM
Unknown Object (File)
Sep 30 2024, 1:25 AM
Unknown Object (File)
Sep 22 2024, 1:21 AM
Unknown Object (File)
Sep 13 2024, 1:40 AM
View All 44 Files
Subscribers
None

Details

Reviewers
imp
emaste
kevans
Commits
rGdf65c89375e2: getpagesizes(3): drop support for kernels before 9.0
Summary

AT_PAGESIZES and elf_aux_info where added prior to FreeBSD 9.0 in commit
ee235befcb8253fab9beea27b916f1bc46b33147. It's safe to say that a
FreeBSD 15 libc won't work on a 8.x kernel so drop sysctl fallback.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 54578
Build 51467: arc lint + arc unit

Event Timeline

brooks requested review of this revision.Nov 22 2023, 1:08 AM
brooks created this revision.
Harbormaster completed remote builds in B54555: Diff 130410.Nov 22 2023, 1:08 AM
brooks added a child revision: D42708: getpagesize(3): drop support for non-ELF kernels.Nov 22 2023, 1:08 AM
emaste accepted this revision.Nov 22 2023, 1:21 AM
emaste added inline comments.
lib/libc/gen/getpagesizes.c
59–61

LGTM, but maybe drop error altogether and just if (_elf_aux_info(AT_PAGESIZES, ps, sizeof(ps) != 0)?

This revision is now accepted and ready to land.Nov 22 2023, 1:21 AM
imp accepted this revision.Nov 22 2023, 1:46 AM
kevans accepted this revision.Nov 22 2023, 5:43 AM
brooks updated this revision to Diff 130444.Nov 22 2023, 8:31 PM
  • Remove error variable
  • On error, fall back to PAGE_SIZE
This revision now requires review to proceed.Nov 22 2023, 8:31 PM
Harbormaster completed remote builds in B54578: Diff 130444.Nov 22 2023, 8:31 PM
emaste accepted this revision.Nov 22 2023, 10:15 PM
This revision is now accepted and ready to land.Nov 22 2023, 10:15 PM
Closed by commit rGdf65c89375e2: getpagesizes(3): drop support for kernels before 9.0 (authored by brooks). · Explain WhyNov 28 2023, 5:16 PM
This revision was automatically updated to reflect the committed changes.
brooks added a commit: rGdf65c89375e2: getpagesizes(3): drop support for kernels before 9.0.

Revision Contents
Changeset List

PathSize
lib/
libc/
gen/
23 lines

Diff 130444

View Options

lib/libc/gen/getpagesizes.c

Loading...