Page MenuHomeFreeBSD

x86 busdma: fix mis-use of bus_addr_t where vm_paddr_t is assumed.
ClosedPublic

Authored by kib on Jan 16 2019, 6:32 AM.
Tags
None
Referenced Files
F141022884: D18854.id52979.diff
Tue, Dec 30, 9:18 PM
Unknown Object (File)
Fri, Dec 26, 10:08 PM
Unknown Object (File)
Sat, Dec 13, 2:32 AM
Unknown Object (File)
Nov 23 2025, 11:32 PM
Unknown Object (File)
Nov 20 2025, 6:09 PM
Unknown Object (File)
Nov 20 2025, 6:09 PM
Unknown Object (File)
Nov 20 2025, 6:09 PM
Unknown Object (File)
Nov 20 2025, 6:07 PM
Subscribers

Details

Summary

Right now bus_addr_t and vm_paddr_t are always aliases for the same underlying integer type on x86, which makes the interchange hard to detect. Shortly, i386 kernel would use uint64_t for vm_paddr_t to enable automatic use of PAE paging structures if hardware allows it, while bus_addr_t would be extended to 64bit only when PAE option is specified.

Fix all places that were identified as using bus_addr_t while page address was assumed. This was performed by testing the complete PAE merging patch on machine with > 4G of RAM enabled.

Tested by: pho (as part of the larger patch)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jan 16 2019, 5:38 PM
This revision was automatically updated to reflect the committed changes.
This revision was not accepted when it landed; it landed in state Needs Review.Jan 18 2019, 1:43 PM
This revision was automatically updated to reflect the committed changes.