Page MenuHomeFreeBSD
Differential D49763

tmpfs: turn vfs.tmpfs.memory_percent into CTLFLAG_RWTUN
ClosedPublic
Actions

Authored by khng on Apr 10 2025, 6:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 25, 8:30 AM
Unknown Object (File)
Sat, May 23, 11:07 AM
Unknown Object (File)
Sat, May 23, 2:14 AM
Unknown Object (File)
Fri, May 15, 8:52 AM
Unknown Object (File)
Thu, May 14, 10:49 PM
Unknown Object (File)
May 14 2026, 4:39 PM
Unknown Object (File)
May 11 2026, 11:03 PM
Unknown Object (File)
May 6 2026, 6:06 PM
View All 89 Files
Subscribers
imp

Details

Reviewers
markj
Commits
rG7c72c0822b94: tmpfs: turn vfs.tmpfs.memory_percent into CTLFLAG_RWTUN
Summary

So sysctl is not the only way to set the value of this knob, this can be
set by loader tunables as well.

Sponsored by: Juniper Networks, Inc.
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

khng created this revision.Apr 10 2025, 6:47 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 10 2025, 6:47 PM
khng requested review of this revision.Apr 10 2025, 6:47 PM
Harbormaster completed remote builds in B63432: Diff 153469.Apr 10 2025, 6:48 PM
markj accepted this revision.Apr 10 2025, 7:02 PM
markj added inline comments.
sys/fs/tmpfs/tmpfs_subr.c
447

Does it really work? sysctl_mem_percent() calls tmpfs_set_reserve_from_percent(), which uses tmpfs_pages_avail_init, which is initialized at SI_SUB_VFS, after SI_SUB_TUNABLES. Hmm, I guess it's okay since tmpfs_init() will call tmpfs_set_reserve_from_percent() again.

This revision is now accepted and ready to land.Apr 10 2025, 7:02 PM
khng added inline comments.Apr 10 2025, 7:58 PM
sys/fs/tmpfs/tmpfs_subr.c
447

Yep. tmpfs_init() will be called again in this case.

Closed by commit rG7c72c0822b94: tmpfs: turn vfs.tmpfs.memory_percent into CTLFLAG_RWTUN (authored by khng). · Explain WhyApr 11 2025, 5:02 PM
This revision was automatically updated to reflect the committed changes.
khng added a commit: rG7c72c0822b94: tmpfs: turn vfs.tmpfs.memory_percent into CTLFLAG_RWTUN.

Revision Contents
Changeset List

PathSize
sys/
fs/
tmpfs/
2 lines

Diff 153534

View Options

sys/fs/tmpfs/tmpfs_subr.c

Loading...