Page MenuHomeFreeBSD
Differential D49881

vm _radix: define foreach macros for walking pages
ClosedPublic
Actions

Authored by dougm on Apr 17 2025, 10:35 PM.
Tags
None
Referenced Files
F124987428: D49881.id153868.diff
Sat, Aug 2, 2:36 AM
F124973815: D49881.id.diff
Fri, Aug 1, 11:13 PM
Unknown Object (File)
Sat, Jul 5, 7:01 PM
Unknown Object (File)
Jul 1 2025, 1:04 PM
Unknown Object (File)
Jun 27 2025, 4:06 AM
Unknown Object (File)
Jun 20 2025, 10:41 AM
Unknown Object (File)
Jun 17 2025, 6:37 PM
Unknown Object (File)
Jun 17 2025, 6:55 AM
View All 36 Files
Subscribers
imp

Details

Reviewers
kib
Commits
rGb7631876399c: vm _radix: define foreach macros for walking pages
Summary

Define VM_RADIX_FOREACH to be TAILQ free equivalent of TAIL_FOREACH, and VM_RADIX_FORALL for walking consecutive pages.

Diff Detail

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

Event Timeline

dougm requested review of this revision.Apr 17 2025, 10:35 PM
dougm created this revision.
kib added a comment.Apr 17 2025, 10:39 PM

I think a comment is needed to explain the intent of each macro. In particular, I want to know which macro iterates just over existing pages, and which requires all indexes to be populated. Without looking at the commit history.

dougm updated this revision to Diff 153818.Apr 17 2025, 11:43 PM

Add comments.

kib accepted this revision.Apr 18 2025, 11:56 AM
This revision is now accepted and ready to land.Apr 18 2025, 11:56 AM
Closed by commit rGb7631876399c: vm _radix: define foreach macros for walking pages (authored by dougm). · Explain WhyApr 18 2025, 3:44 PM
This revision was automatically updated to reflect the committed changes.
dougm added a commit: rGb7631876399c: vm _radix: define foreach macros for walking pages.
Herald added a subscriber: imp. · View Herald TranscriptApr 18 2025, 3:44 PM

Revision Contents
Changeset List

PathSize
sys/
vm/
26 lines

Diff 153868

View Options

sys/vm/vm_radix.h

Loading...