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
Unknown Object (File)
Wed, Aug 12, 12:16 PM
Unknown Object (File)
Wed, Aug 12, 12:16 PM
Unknown Object (File)
Tue, Aug 4, 9:31 AM
Unknown Object (File)
Mon, Aug 3, 2:54 AM
Unknown Object (File)
Mon, Jul 27, 5:01 PM
Unknown Object (File)
Sat, Jul 25, 3:50 PM
Unknown Object (File)
Wed, Jul 22, 4:37 PM
Unknown Object (File)
Jul 20 2026, 12:34 PM

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...