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)
Thu, Nov 27, 1:37 PM
Unknown Object (File)
Wed, Nov 26, 11:34 AM
Unknown Object (File)
Fri, Nov 21, 2:55 AM
Unknown Object (File)
Fri, Nov 21, 2:52 AM
Unknown Object (File)
Fri, Nov 21, 2:50 AM
Unknown Object (File)
Sun, Nov 16, 10:17 PM
Unknown Object (File)
Sat, Nov 15, 10:29 PM
Unknown Object (File)
Sat, Nov 15, 6:03 PM

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