Page MenuHomeFreeBSD

pmap_mapdev: Consistently use vm_paddr_t for the first argument.
ClosedPublic

Authored by jhb on Jul 26 2022, 10:37 PM.
Tags
None
Referenced Files
F111563683: D35961.diff
Wed, Mar 5, 7:55 AM
Unknown Object (File)
Tue, Feb 25, 6:24 AM
Unknown Object (File)
Jan 12 2025, 11:42 PM
Unknown Object (File)
Dec 9 2024, 5:33 PM
Unknown Object (File)
Nov 15 2024, 1:18 AM
Unknown Object (File)
Oct 1 2024, 1:18 PM
Unknown Object (File)
Sep 23 2024, 12:40 AM
Unknown Object (File)
Sep 23 2024, 12:14 AM

Details

Summary

The devmap variants used vm_offset_t for some reason, and a few places
explicitly cast bus addresses to vm_offset_t. (Probably those casts
along with similar casts for vm_size_t should just be removed and
instead permit the compiler to DTRT.)

Sponsored by: DARPA

Diff Detail

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

Event Timeline

jhb requested review of this revision.Jul 26 2022, 10:37 PM
This revision is now accepted and ready to land.Jul 27 2022, 3:19 PM
sys/kern/subr_devmap.c
273

This can probably be reverted back to trunc_page? I think I was just overly cautious.

293

It might be a nice followup if pmap_mapdev() on aarch64 and risc-v just called pmap_mapdev_attr() so we could avoid code duplication.

sys/kern/subr_devmap.c
273

I can't see any reason not to use trunc_page() with a vm_paddr_t...