Page MenuHomeFreeBSD

Don't panic in vm_radix_remove(); instead return a value
ClosedPublic

Authored by alc on Dec 4 2016, 6:46 AM.
Tags
None
Referenced Files
F131932801: D8708.diff
Sun, Oct 12, 7:45 AM
Unknown Object (File)
Thu, Oct 9, 1:47 AM
Unknown Object (File)
Wed, Sep 24, 6:45 AM
Unknown Object (File)
Sep 11 2025, 4:11 PM
Unknown Object (File)
Aug 30 2025, 7:34 AM
Unknown Object (File)
Aug 14 2025, 10:49 PM
Unknown Object (File)
Jul 18 2025, 2:48 AM
Unknown Object (File)
Jul 6 2025, 4:38 PM
Subscribers

Details

Summary

Change vm_radix_remove() to return a value, specifically, a vm_page_t or NULL, rather than panic()ing when the specified index isn't found in the trie. This change simplifies the pmap's use of tries for page table pages, specifically, we no longer have to do a lookup before every remove.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

alc retitled this revision from to Don't panic in vm_radix_remove(); instead return a value.
alc updated this object.
alc edited the test plan for this revision. (Show Details)
alc added reviewers: kib, markj, andrew.
kib edited edge metadata.
This revision is now accepted and ready to land.Dec 4 2016, 2:01 PM
markj edited edge metadata.
This revision was automatically updated to reflect the committed changes.