Page MenuHomeFreeBSD

amd64 pmap: Tidy up pmap_promote_pde() calls
ClosedPublic

Authored by alc on Jun 24 2023, 6:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 11 2024, 7:12 AM
Unknown Object (File)
Oct 11 2024, 7:11 AM
Unknown Object (File)
Oct 5 2024, 2:55 PM
Unknown Object (File)
Oct 2 2024, 4:21 PM
Unknown Object (File)
Oct 2 2024, 12:04 AM
Unknown Object (File)
Oct 1 2024, 2:12 AM
Unknown Object (File)
Sep 30 2024, 8:19 PM
Unknown Object (File)
Sep 27 2024, 10:15 AM
Subscribers

Details

Summary

Since pmap_ps_enabled() is true by default, check it inside of pmap_promote_pde() instead of at every call site.

Modify pmap_promote_pde() to return true if the promotion succeeded and false otherwise. Use this return value in a couple places.

Diff Detail

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

Event Timeline

alc requested review of this revision.Jun 24 2023, 6:47 AM
alc created this revision.
This revision is now accepted and ready to land.Jun 24 2023, 7:41 AM

Is there any reason not to make the same change on arm64? (Or riscv for that matter.)

Is there any reason not to make the same change on arm64? (Or riscv for that matter.)

In principle, no. I will look at arm64 soon. The pmap->pm_stage == PG_STAGE1 checks on arm64 complicate things.

This revision was automatically updated to reflect the committed changes.