Page MenuHomeFreeBSD
Differential D49559

arm64: Reduce the use of DMAMAP_COHERENT
Needs ReviewPublic
Actions

Authored by andrew on Mar 28 2025, 5:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 7, 3:01 PM
Unknown Object (File)
Sat, Sep 5, 3:54 PM
Unknown Object (File)
Wed, Sep 2, 4:05 PM
Unknown Object (File)
Wed, Sep 2, 9:42 AM
Unknown Object (File)
Thu, Aug 27, 2:39 PM
Unknown Object (File)
Thu, Aug 27, 2:24 PM
Unknown Object (File)
Thu, Aug 20, 8:50 AM
Unknown Object (File)
Aug 8 2026, 4:08 PM
View All 84 Files
Subscribers
ehem_freebsd_m5p.com
emaste
imp

Details

Reviewers
manu
Group Reviewers
arm64
Summary

The coherent busdma implementation doesn't need to set DMAMAP_COHERENT.
It is only used in cacheline_bounce which is unused there.

Sponsored by: Arm Ltd

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 63184
Build 60068: arc lint + arc unit

Event Timeline

andrew created this revision.Mar 28 2025, 5:16 PM
Herald added a reviewer: manu. · View Herald TranscriptMar 28 2025, 5:16 PM
Herald added subscribers: emaste, imp. · View Herald Transcript
andrew requested review of this revision.Mar 28 2025, 5:16 PM
Harbormaster completed remote builds in B63184: Diff 152802.Mar 28 2025, 5:16 PM
andrew added a parent revision: D49558: arm64: Move cacheline_bounce later.Mar 28 2025, 5:16 PM
ehem_freebsd_m5p.com added a subscriber: ehem_freebsd_m5p.com.Mar 28 2025, 11:20 PM
ehem_freebsd_m5p.com added inline comments.
sys/arm64/arm64/busdma_bounce.c
105

Why is DMAMAP_COHERENT being renamed here? This seems an okay idea, but if you're going to rename all the other uses should also be modified. Yet instead you have the new #define below so this doesn't seem useful as you've got a halfway done job.

364–368

Since both of these are now single-line, I would remove the brackets while you're at it.

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
5 lines
13 lines

Diff 152802

View Options

sys/arm64/arm64/busdma_bounce.h

Loading...
View Options

sys/arm64/arm64/busdma_bounce.c

Loading...