Page MenuHomeFreeBSD
Differential D50263

vm_page: cleanup
ClosedPublic
Actions

Authored by dougm on May 9 2025, 3:29 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 6, 5:12 AM
Unknown Object (File)
Wed, Jun 4, 9:10 AM
Unknown Object (File)
Sat, May 31, 8:55 PM
Unknown Object (File)
Fri, May 30, 6:03 AM
Unknown Object (File)
Wed, May 28, 10:57 PM
Unknown Object (File)
Tue, May 27, 2:56 PM
Unknown Object (File)
Tue, May 27, 7:26 AM
Unknown Object (File)
Sun, May 25, 7:14 AM
View All 17 Files
Subscribers
alc
imp

Details

Reviewers
alc
markj
kib
Commits
rG3fe2f81f665b: vm_page: reduce iter_reset calls in alloc_domain_iter
rG349340c8cd5c: vm_page: use vm_page_alloc_fail consistently
rGefde1e2c8b76: vm_page: remove unneeded use of mpred
rG7c9fe7a7925b: vm_page: remove obsolete page alloc loops
rGca18e0441029: vm_page: update some iterator-related comments
Summary

Reaction to @alc comments on D50253.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dougm requested review of this revision.May 9 2025, 3:29 AM
dougm created this revision.
markj added inline comments.Sun, May 11, 4:16 PM
sys/vm/vm_page.c
2351

Every other block of conditionally compiled code guarded by VM_NRESERVLEVEL > 0 uses a preprocessor check, so I'd rather keep things consistent here even though it's equivalent.

2360
dougm updated this revision to Diff 155352.Mon, May 12, 8:09 PM
dougm marked 2 inline comments as done.
kib accepted this revision.Tue, May 13, 12:51 AM
kib added inline comments.
sys/vm/vm_page.c
2143
This revision is now accepted and ready to land.Tue, May 13, 12:51 AM
markj accepted this revision.Thu, May 15, 6:59 PM
alc accepted this revision.Fri, May 16, 7:32 AM
Closed by commit rGca18e0441029: vm_page: update some iterator-related comments (authored by dougm). · Explain WhyFri, May 16, 8:20 AM
This revision was automatically updated to reflect the committed changes.
dougm marked an inline comment as done.
dougm added a commit: rGca18e0441029: vm_page: update some iterator-related comments.
Herald added a subscriber: imp. · View Herald TranscriptFri, May 16, 8:20 AM
dougm added a commit: rG7c9fe7a7925b: vm_page: remove obsolete page alloc loops.Fri, May 16, 8:20 AM
dougm added a commit: rGefde1e2c8b76: vm_page: remove unneeded use of mpred.
dougm added a commit: rG349340c8cd5c: vm_page: use vm_page_alloc_fail consistently.
dougm added a commit: rG3fe2f81f665b: vm_page: reduce iter_reset calls in alloc_domain_iter.

Revision Contents
Changeset List

PathSize
sys/
vm/
53 lines

Diff 155352

View Options

sys/vm/vm_page.c

Loading...