Page MenuHomeFreeBSD
Differential D43636

vm_phys: Add a vm_phys_seg_paddr_to_vm_page() helper
ClosedPublic
Actions

Authored by markj on Jan 28 2024, 2:36 PM.
Tags
None
Referenced Files
F118749749: D43636.id133460.diff
Sun, Jun 1, 12:52 PM
Unknown Object (File)
Mon, May 26, 12:43 PM
Unknown Object (File)
Sun, May 25, 4:39 PM
Unknown Object (File)
Apr 29 2025, 6:31 AM
Unknown Object (File)
Apr 27 2025, 11:54 PM
Unknown Object (File)
Apr 22 2025, 12:06 PM
Unknown Object (File)
Apr 21 2025, 8:45 AM
Unknown Object (File)
Apr 8 2025, 3:16 AM
View All 76 Files
Subscribers
imp

Details

Reviewers
alc
dougm
kib
Commits
rG69cbb18746b6: vm_phys: Add a vm_phys_seg_paddr_to_vm_page() helper
Summary

No functional change intended.

Suggested by: alc

Diff Detail

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

Event Timeline

markj created this revision.Jan 28 2024, 2:36 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 28 2024, 2:36 PM
markj requested review of this revision.Jan 28 2024, 2:36 PM
Harbormaster completed remote builds in B55664: Diff 133460.Jan 28 2024, 2:36 PM
kib accepted this revision.Jan 28 2024, 5:05 PM
This revision is now accepted and ready to land.Jan 28 2024, 5:05 PM
alc accepted this revision.Jan 28 2024, 6:09 PM

Despite the long name, it's still two characters shorter than the original code. :-)

dougm accepted this revision.Jan 28 2024, 6:27 PM
dougm added inline comments.
sys/vm/vm_phys.c
940

This could be applied at vm_page.c:787, if it were visible there.

977

If could be used here, if the fields of vm_phys_fictitious_seg were reordered.

markj marked an inline comment as done.Jan 29 2024, 4:18 PM
markj added inline comments.
sys/vm/vm_phys.c
940

I made the function externally visible.

977

I think I'd prefer to leave this case as it is. If there were more instances of this pattern, I'd be more inclined to take the suggestion.

markj updated this revision to Diff 133501.Jan 29 2024, 4:33 PM
markj marked an inline comment as done.

Incorporate one of Doug's suggestions.

This revision now requires review to proceed.Jan 29 2024, 4:33 PM
Harbormaster completed remote builds in B55684: Diff 133501.Jan 29 2024, 4:34 PM
dougm accepted this revision.Jan 29 2024, 4:54 PM
This revision is now accepted and ready to land.Jan 29 2024, 4:54 PM
Closed by commit rG69cbb18746b6: vm_phys: Add a vm_phys_seg_paddr_to_vm_page() helper (authored by markj). · Explain WhyJun 14 2024, 1:24 AM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG69cbb18746b6: vm_phys: Add a vm_phys_seg_paddr_to_vm_page() helper.

Revision Contents
Changeset List

PathSize
sys/
vm/
2 lines
1 line
29 lines

Diff 139831

View Options

sys/vm/vm_page.c

Loading...
View Options

sys/vm/vm_phys.h

Loading...
View Options

sys/vm/vm_phys.c

Loading...