Page MenuHomeFreeBSD
Differential D54959

virtio: Use bus_dma for ring and indirect buffer allocations
ClosedPublic
Actions

Authored by sarah.walker2_arm.com on Jan 30 2026, 11:14 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 2, 6:25 PM
Unknown Object (File)
Sat, Feb 28, 10:21 PM
Unknown Object (File)
Sat, Feb 28, 3:57 PM
Unknown Object (File)
Sat, Feb 28, 1:09 PM
Unknown Object (File)
Sat, Feb 28, 11:49 AM
Unknown Object (File)
Tue, Feb 24, 6:06 AM
Unknown Object (File)
Mon, Feb 23, 9:54 PM
Unknown Object (File)
Wed, Feb 18, 11:19 PM
View All 18 Files
Subscribers
cperciva
emaste
imp
jrtc27
virtio

Details

Reviewers
bryanv
andrew
Commits
rGc499ad6f997c: virtio: Use bus_dma for ring and indirect buffer allocations
Summary

While the majority of virtio platforms will be fully coherent, some may
require cache maintenance or other specific device memory handling (eg for
secure partitioning). Using bus_dma allows for these usecases.

The virtio buffers are marked as coherent; this should ensure that sync
calls are no-ops in the common cases.

Sponsored by: Arm Ltd

Diff Detail

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

Event Timeline

sarah.walker2_arm.com created this revision.Jan 30 2026, 11:14 AM
Herald added a subscriber: imp. · View Herald TranscriptJan 30 2026, 11:14 AM
sarah.walker2_arm.com requested review of this revision.Jan 30 2026, 11:14 AM
Harbormaster completed remote builds in B70227: Diff 170769.Jan 30 2026, 11:14 AM
sarah.walker2_arm.com added a child revision: D54960: virtio_blk: Use bus_dma for command/ack buffer allocations.Jan 30 2026, 11:14 AM
andrew added reviewers: bryanv, andrew.Feb 2 2026, 5:42 PM
emaste added a subscriber: emaste.Tue, Feb 3, 5:23 PM
jrtc27 added a subscriber: jrtc27.Tue, Feb 3, 5:30 PM

Do we still need the mb()s? I would expect bus_dmamap_sync to suffice.

jrtc27 added a comment.Tue, Feb 3, 5:31 PM

virtio_blk calls bus_dma_tag_set_iommu on PowerPC; would that not be needed for these new tags?

br mentioned this in D55220: virtio: get a correct tag for IOMMU operation.Tue, Feb 10, 2:05 PM
sarah.walker2_arm.com updated this revision to Diff 171981.Mon, Feb 16, 3:36 PM

Remove superfluous mb() calls.

Harbormaster completed remote builds in B70704: Diff 171981.Mon, Feb 16, 3:36 PM
andrew added a subscriber: cperciva.Tue, Feb 17, 12:06 PM
In D54959#1259073, @jrtc27 wrote:

virtio_blk calls bus_dma_tag_set_iommu on PowerPC; would that not be needed for these new tags?

It looks like that was added in D37891 by @cperciva. Based on the commit message we could add it here, but can't test.

sarah.walker2_arm.com updated this revision to Diff 172614.Tue, Feb 24, 5:05 PM

Add iommu tag calls for PowerPC.

Harbormaster completed remote builds in B70963: Diff 172614.Tue, Feb 24, 5:05 PM
andrew accepted this revision.Tue, Feb 24, 5:05 PM
This revision is now accepted and ready to land.Tue, Feb 24, 5:05 PM
Closed by commit rGc499ad6f997c: virtio: Use bus_dma for ring and indirect buffer allocations (authored by sarah.walker2_arm.com, committed by andrew). · Explain WhyTue, Mar 3, 5:38 PM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rGc499ad6f997c: virtio: Use bus_dma for ring and indirect buffer allocations.
emaste added a subscriber: virtio.Tue, Mar 3, 5:57 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
virtio/
27 lines
216 lines

Diff 173067

View Options

sys/dev/virtio/virtio_ring.h

Loading...
View Options

sys/dev/virtio/virtqueue.c

Loading...