Differential D28073

virtio: Reduce boilerplate for device driver module definitions
ClosedPublic
Actions

Authored by jrtc27 on Jan 9 2021, 10:05 PM.

Tags

None

Referenced Files

	F112079202: D28073.id82667.diff
	Wed, Mar 12, 12:50 PM

	F112058883: D28073.diff
	Wed, Mar 12, 5:56 AM

	Unknown Object (File)
	Tue, Mar 4, 6:57 PM

	Unknown Object (File)
	Tue, Feb 25, 7:24 PM

	Unknown Object (File)
	Sat, Feb 15, 12:51 PM

	Unknown Object (File)
	Feb 2 2025, 11:13 PM

	Unknown Object (File)
	Jan 17 2025, 9:45 PM

	Unknown Object (File)
	Jan 15 2025, 2:05 PM

View All 58 Files

Subscribers

Details

Reviewers

Commits

rG633218ee4615: virtio: Reduce boilerplate for device driver module definitions

Summary

Rather than have every device register itself for both virtio_pci and
virtio_mmio, provide a VIRTIO_DRIVER_MODULE wrapper to declare both,
merge VIRTIO_SIMPLE_PNPTABLE with VIRTIO_SIMPLE_PNPINFO and make the
latter register for both buses. This also has the benefit of abstracting
away the available transports and their names.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 36070
Build 32959: arc lint + arc unit

Event Timeline

jrtc27 created this revision.Jan 9 2021, 10:05 PM

Herald added a subscriber: imp. · View Herald TranscriptJan 9 2021, 10:05 PM

jrtc27 requested review of this revision.Jan 9 2021, 10:05 PM

Harbormaster completed remote builds in B36070: Diff 81977.Jan 9 2021, 10:05 PM

jrtc27 mentioned this in D28031: virtio: Use a common class name between virtio_mmio and virtio_pci.Jan 9 2021, 10:13 PM

jrtc27 added a child revision: D28031: virtio: Use a common class name between virtio_mmio and virtio_pci.Jan 9 2021, 10:13 PM

jrtc27 mentioned this in D27856: virtio: Add VirtIO PCI modern (V1) support.Jan 12 2021, 7:12 AM

bryanv accepted this revision.Jan 19 2021, 5:30 AM

This revision is now accepted and ready to land.Jan 19 2021, 5:30 AM

Closed by commit rG633218ee4615: virtio: Reduce boilerplate for device driver module definitions (authored by jrtc27). · Explain WhyJan 21 2021, 1:15 AM

This revision was automatically updated to reflect the committed changes.

jrtc27 added a commit: rG633218ee4615: virtio: Reduce boilerplate for device driver module definitions.

Revision Contents
Changeset List

Path

Size

sys/

dev/

virtio/

balloon/

virtio_balloon.c

8 lines

block/

8 lines

console/

virtio_console.c

8 lines

network/

8 lines

random/

virtio_random.c

8 lines

scsi/

8 lines

13 lines

Diff 81977

sys/dev/virtio/balloon/virtio_balloon.c

Loading...

sys/dev/virtio/block/virtio_blk.c

Loading...

sys/dev/virtio/console/virtio_console.c

Loading...

sys/dev/virtio/network/if_vtnet.c

Loading...

sys/dev/virtio/random/virtio_random.c

Loading...

sys/dev/virtio/scsi/virtio_scsi.c

Loading...

sys/dev/virtio/virtio.h

Loading...