Page MenuHomeFreeBSD
Differential D45065

param.h: Add PAGE_SIZE_MAX and PAGE_SHIFT_MAX
Needs ReviewPublic
Actions

Authored by andrew on May 2 2024, 4:38 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 8, 12:56 PM
Unknown Object (File)
Sat, Oct 3, 2:58 PM
Unknown Object (File)
Thu, Oct 1, 10:35 AM
Unknown Object (File)
Thu, Sep 24, 3:46 AM
Unknown Object (File)
Thu, Sep 24, 2:26 AM
Unknown Object (File)
Mon, Sep 21, 1:01 PM
Unknown Object (File)
Sat, Sep 12, 3:30 AM
Unknown Object (File)
Fri, Sep 11, 12:19 AM
View All 86 Files
Subscribers
emaste
gallatin
imp
jhibbits
riscv

Details

Reviewers
manu
Summary

These are intended to hold the largest page size the architecture
supports.

Sponsored by: Arm Ltd

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 57548
Build 54436: arc lint + arc unit

Event Timeline

andrew created this revision.May 2 2024, 4:38 PM
Herald added a reviewer: manu. · View Herald TranscriptMay 2 2024, 4:38 PM
Herald added subscribers: riscv, jhibbits, emaste, imp. · View Herald Transcript
andrew requested review of this revision.May 2 2024, 4:38 PM
Harbormaster completed remote builds in B57548: Diff 138026.May 2 2024, 4:38 PM
andrew added a child revision: D45066: libsys: Fall back to the worst case page size.May 2 2024, 4:38 PM
andrew mentioned this in D45042: arm64: Make jemalloc safe for 16k / 4k interoperability.May 2 2024, 4:39 PM
gallatin added a subscriber: gallatin.May 2 2024, 6:11 PM
gallatin added inline comments.
sys/arm64/include/param.h
101

I thought 64K was mostly theoretical. Eg, we cannot actually boot a 64k kernel, while 16k works just fine with a few minor fixes.

andrew added inline comments.May 3 2024, 9:17 AM
sys/arm64/include/param.h
101

This creates an ABI that userspace will use in jemalloc. If we add a 64k kernel later then we would have to bump this and wouldn't be able to run old binaries.

emaste added inline comments.May 3 2024, 2:28 PM
sys/arm64/include/param.h
101

What is the cost of having this be 64K vs 16K (if we never implemented a 64K kernel)?

Revision Contents
Changeset List

PathSize
sys/
amd64/
include/
2 lines
arm/
include/
2 lines
arm64/
include/
3 lines
i386/
include/
2 lines
powerpc/
include/
2 lines
riscv/
include/
2 lines

Diff 138026

View Options

sys/amd64/include/param.h

Loading...
View Options

sys/arm/include/param.h

Loading...
View Options

sys/arm64/include/param.h

Loading...
View Options

sys/i386/include/param.h

Loading...
View Options

sys/powerpc/include/param.h

Loading...
View Options

sys/riscv/include/param.h

Loading...