Page MenuHomeFreeBSD
Differential D20277

revert 4GB workarounds for bge and aac
ClosedPublic
Actions

Authored by tychon on May 16 2019, 1:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sep 27 2024, 5:08 PM
Unknown Object (File)
Sep 27 2024, 10:40 AM
Unknown Object (File)
Sep 25 2024, 6:29 PM
Unknown Object (File)
Sep 21 2024, 6:54 AM
Unknown Object (File)
Sep 8 2024, 6:26 AM
Unknown Object (File)
Sep 6 2024, 3:13 AM
Unknown Object (File)
Aug 31 2024, 9:46 PM
Unknown Object (File)
Aug 30 2024, 7:22 PM
View All 49 Files
Subscribers
imp

Details

Reviewers
kib
jhb
Commits
rS347890: reinstate 4GB DMA boundary workarounds for bge and aac
Summary

After r346386, DMA transactions may again cross 4GB boundaries. I had intended to return the per-device workaround for devices known to have 64-bit issues. Apparently while I got the header file component I missed the salient change to bge in the shuffle. Also add the necessary workaround for aac.

This is a straightforward revert of r232766 and r232260 respectively.

Diff Detail

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

Event Timeline

tychon created this revision.May 16 2019, 1:16 PM
kib added inline comments.May 16 2019, 7:52 PM
sys/dev/aac/aac_pci.c
447 ↗(On Diff #57445)

aac uses BUS_SPACE_MAXSIZE_32BIT in other places.

tychon added inline comments.May 16 2019, 8:10 PM
sys/dev/aac/aac_pci.c
447 ↗(On Diff #57445)

Yeah, I see that but BUS_SPACE_MAXSIZE_32BIT (0xFFFFFFFF) isn't a power-of-2. Isn't that a requirement for alignment?

kib accepted this revision.May 16 2019, 8:26 PM
kib added inline comments.
sys/dev/aac/aac_pci.c
447 ↗(On Diff #57445)

I see.

This revision is now accepted and ready to land.May 16 2019, 8:26 PM
Closed by commit rS347890: reinstate 4GB DMA boundary workarounds for bge and aac (authored by tychon). · Explain WhyMay 16 2019, 8:41 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptMay 16 2019, 8:41 PM

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
aac/
3 lines
bge/
37 lines
6 lines

Diff 57461

View Options

head/sys/dev/aac/aac_pci.c

Loading...
View Options

head/sys/dev/bge/if_bge.c

Loading...
View Options

head/sys/dev/bge/if_bgereg.h

Loading...