Page MenuHomeFreeBSD

Port x86 busdma to ARM64
ClosedPublic

Authored by zbb on Apr 23 2015, 6:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 29, 12:04 AM
Unknown Object (File)
Jan 11 2024, 11:13 PM
Unknown Object (File)
Jan 8 2024, 2:50 AM
Unknown Object (File)
Dec 23 2023, 10:34 AM
Unknown Object (File)
Dec 22 2023, 9:19 PM
Unknown Object (File)
Dec 17 2023, 10:57 AM
Unknown Object (File)
Nov 28 2023, 1:07 PM
Unknown Object (File)
Nov 23 2023, 3:33 PM

Details

Summary

The x86 busdma subsystem allows using multiple implementations.
By default the classic bounce buffer approach is used, however
on systems with IOMMU it could be in runtime switched to more
efficient hardware accelerated implementation.

This commit adds ARM64 port of the x86 busdma framework and bounce
buffer backend. It is ready to use on IO coherent systems. If the
IO coherency cannot be guaranteed, the cache management operations have
to be added to this code in places marked by /* XXX ARM64TODO (...) */
comments. Also IOMMU support might be added by registering another
busdma implementation like it is already done on the x86.

Obtained from: Semihalf
Sponsored by: The FreeBSD Foundation

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

zbb retitled this revision from to Port x86 busdma to ARM64.
zbb updated this object.
zbb edited the test plan for this revision. (Show Details)
zbb added reviewers: emaste, ian, imp, andrew.
zbb added a subscriber: Unknown Object (MLST).

The lack of cache handling makes this unusable for a large amount of hardware that doesn't have cache-coherent DMA.

skra added inline comments.
sys/arm64/arm64/busdma_bounce.c
1006

Just a note that vaddr should be replaced by addr. See r282120.

sys/arm64/arm64/busdma_bounce.c
1006

Oh. Great thanks!

sys/arm64/arm64/busdma_bounce.c
1006

As an aside, if you use Phabricator style rev numbers (rS282120) it will automatically link to the revision.
rS = src
rP = ports
rD = docs

sys/arm64/arm64/busdma_bounce.c
405

Can you add an ARM64TODO for easy grepping later on

sys/arm64/include/bus_dma.h
1–34

This (entire file) change should be reverted

sys/arm64/include/bus_dma_impl.h
32

s/ARM64/MACHINE/ in the include guards

sys/arm64/arm64/busdma_bounce.c
405

This one is actually a copy-paste from x86, present in all copies like sparc, powerpc, etc.
Still change it to ARM64TODO?

zbb edited edge metadata.
In D2357#43295, @andrew wrote:

The lack of cache handling makes this unusable for a large amount of hardware that doesn't have cache-coherent DMA.

The commit description has already said that and we have ARM64TODOs in the file. This commit is still less unusable than the previous implementation.

sys/arm64/arm64/busdma_bounce.c
135

Style?

937

Extra indentation

sys/arm64/arm64/busdma_bounce.c
135

Right,

Long parameter lists are wrapped with a normal four space indent.

406

No, I'd leave this.

andrew edited edge metadata.

Only a few minor issues.

sys/arm64/arm64/busdma_bounce.c
203

Wouldn't error already be zero in this case?

324

Extra braces

sys/arm64/arm64/busdma_machdep.c
127

Could you provide a better error string here?

133

We only return a NULL tag on some failures? There is a case on the previous line where it could fail without the tag set.

sys/arm64/include/bus_dma_impl.h
33

There should be no tab after #ifndef

This revision is now accepted and ready to land.May 7 2015, 4:17 PM
emaste edited edge metadata.
emaste added inline comments.
sys/arm64/arm64/busdma_bounce.c
324

I'd argue that if one side of an else has braces the other should be allowed to as well, even if technically contrary to style(9).