Page MenuHomeFreeBSD

budsma: fix boundary alignment test
ClosedPublic

Authored by dougm on Jan 1 2022, 11:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 8:22 PM
Unknown Object (File)
Tue, Nov 19, 6:27 PM
Unknown Object (File)
Thu, Nov 14, 7:28 AM
Unknown Object (File)
Wed, Nov 13, 11:57 AM
Unknown Object (File)
Mon, Nov 11, 6:32 AM
Unknown Object (File)
Mon, Nov 11, 3:53 AM
Unknown Object (File)
Mon, Nov 11, 3:45 AM
Unknown Object (File)
Mon, Nov 11, 3:24 AM

Details

Summary

c606ab59e7f9 vm_extern: use standard address checkers everywhere

broke a check in _bus_dmamap_addseg on many architectures. This change makes it clear that it is not a particular range that is being boundary-checked, but the proposed union of the two adjacent ranges.

Diff Detail

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

Event Timeline

dougm requested review of this revision.Jan 1 2022, 11:48 PM

Looks good to me:

increase current segment by sgsize bytes if

  • curraddr is contiguous to the end of the segment,
  • adding sgsize will not cause maxsegsz to be exceeded, and
  • start address plus increased segment length will still pass the boundary check.
This revision is now accepted and ready to land.Jan 2 2022, 1:23 PM
This revision was automatically updated to reflect the committed changes.