Page MenuHomeFreeBSD

pmap: Degrade pmap_page_set_attr*() into a no-op on same attribute
ClosedPublic

Authored by olce on Jul 11 2025, 7:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jul 11, 3:59 PM
Unknown Object (File)
Fri, Jul 10, 9:52 PM
Unknown Object (File)
Tue, Jul 7, 1:01 PM
Unknown Object (File)
Fri, Jul 3, 11:32 PM
Unknown Object (File)
Thu, Jun 18, 9:21 PM
Unknown Object (File)
Jun 12 2026, 7:46 PM
Unknown Object (File)
May 29 2026, 8:52 PM
Unknown Object (File)
May 29 2026, 12:28 AM

Details

Summary

For 32-bit arm, move the no-op test that was already in place at start
of the function so that it stays first even if the '#if 0' block around
the call to sf_buf_invalidate_cache() is uncommented at some point (if
ever).

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 65346
Build 62229: arc lint + arc unit

Event Timeline

olce requested review of this revision.Jul 11 2025, 7:27 AM

As a non-expert on the code, LGTM (assuming there aren't some special rules about read-write accesses to the saved attributes).

sys/powerpc/aim/mmu_radix.c
5942

Trivial nit: other files got an extra empty line after the if-return.

This revision is now accepted and ready to land.Jul 12 2025, 6:03 PM