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)
Thu, May 2, 7:00 PM
Unknown Object (File)
Thu, May 2, 3:35 PM
Unknown Object (File)
Thu, May 2, 3:35 PM
Unknown Object (File)
Thu, May 2, 2:59 PM
Unknown Object (File)
Dec 20 2023, 7:28 AM
Unknown Object (File)
Dec 12 2023, 7:30 AM
Unknown Object (File)
Aug 7 2023, 11:12 PM
Unknown Object (File)
Jul 22 2023, 8:36 PM
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.