Page MenuHomeFreeBSD

Update comment describing vm_map
ClosedPublic

Authored by kib on Aug 10 2023, 5:05 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sep 11 2024, 7:58 PM
Unknown Object (File)
Sep 8 2024, 10:20 AM
Unknown Object (File)
Sep 7 2024, 7:02 PM
Unknown Object (File)
Sep 7 2024, 5:24 PM
Unknown Object (File)
Sep 5 2024, 6:54 PM
Unknown Object (File)
Sep 5 2024, 5:41 AM
Unknown Object (File)
Sep 4 2024, 12:10 AM
Unknown Object (File)
Sep 3 2024, 12:32 PM
Subscribers

Details

Summary

There is no list connecting all entries any more, and correspondingly no order on the list entries.

Diff Detail

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

Event Timeline

kib requested review of this revision.Aug 10 2023, 5:05 AM
sys/vm/vm_map.h
186

I agree that "The tree is" should replace "Both structures are", since one of the two structures mentioned in the original comment, the linked-list, disappeared when the tree became threaded.

But I don't understand how the tree is no longer threaded.

A threaded binary search tree is one where a node with no left child points with its left pointer to its leftmost ancestor, instead of NULL, and similarly for right children and ancestors. I don't think that has changed.

kib marked an inline comment as done.Aug 10 2023, 5:57 AM
kib added inline comments.
sys/vm/vm_map.h
186

I thought 'threaded' referred to the list linked all entries.

kib marked an inline comment as done.
kib edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Aug 10 2023, 5:59 AM
This revision was automatically updated to reflect the committed changes.