Paths

Table of Contentst

Differential D51992

virtio/mmio: Remove the unused virtio_mmio_if.m
ClosedPublic
Actions

Authored by andrew on Aug 18 2025, 9:57 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Jan 12, 3:45 PM

	Unknown Object (File)
	Mon, Jan 12, 3:45 PM

	Unknown Object (File)
	Wed, Dec 17, 7:32 PM

	Unknown Object (File)
	Wed, Dec 17, 3:12 AM

	Unknown Object (File)
	Dec 5 2025, 5:21 PM

	Unknown Object (File)
	Dec 2 2025, 10:27 AM

	Unknown Object (File)
	Nov 24 2025, 8:45 PM

	Unknown Object (File)
	Nov 20 2025, 9:18 PM

View All 50 Files

Subscribers

Details

Reviewers

br
imp

Commits

rG4740d47238cf: virtio/mmio: Remove the unused virtio_mmio_if.m

Summary

The only driver I know that implemented this interface was removed 7
months ago. Remove the interface and platform from the softc.

Sponsored by: Arm Ltd

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

andrew created this revision.Aug 18 2025, 9:57 AM

Herald added a subscriber: imp. · View Herald TranscriptAug 18 2025, 9:57 AM

andrew requested review of this revision.Aug 18 2025, 9:57 AM

Harbormaster completed remote builds in B66315: Diff 160532.Aug 18 2025, 9:57 AM

andrew added a child revision: D51993: virtio: Remove the unused poll method.Aug 18 2025, 9:57 AM

What was the last driver?

This revision is now accepted and ready to land.Aug 18 2025, 2:28 PM

It was a beri driver in sys/dev/beri/virtio/virtio_mmio_platform.c

jrtc27 added a subscriber: jrtc27.Aug 18 2025, 5:14 PM

jrtc27 added inline comments.

sys/dev/virtio/mmio/virtio_mmio.c
104–109
160	I believe this method is now never implemented? Presumably it was for cache coherence or similar in the dodgy BERI VirtIO world; virtqueue_poll calls it just before virtqueue_dequeue.

andrew added inline comments.Aug 20 2025, 9:16 AM

sys/dev/virtio/mmio/virtio_mmio.c
160	I already removed it in D51993

Closed by commit rG4740d47238cf: virtio/mmio: Remove the unused virtio_mmio_if.m (authored by andrew). · Explain WhyAug 21 2025, 8:44 AM

This revision was automatically updated to reflect the committed changes.

andrew added a commit: rG4740d47238cf: virtio/mmio: Remove the unused virtio_mmio_if.m.

Revision Contents
Changeset List

Path

Size

sys/

conf/

1 line

dev/

virtio/

mmio/

1 line

48 lines

virtio_mmio_fdt.c

47 lines

virtio_mmio_if.m

Diff 160714

sys/conf/files

Loading...

sys/dev/virtio/mmio/virtio_mmio.h

Loading...

sys/dev/virtio/mmio/virtio_mmio.c

Loading...

sys/dev/virtio/mmio/virtio_mmio_fdt.c

Loading...

sys/dev/virtio/mmio/virtio_mmio_if.m

Loading...