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

	Unknown Object (File)
	Fri, Apr 12, 8:56 AM

	Unknown Object (File)
	Thu, Apr 11, 7:20 PM

	Unknown Object (File)
	Thu, Apr 11, 7:20 PM

	Unknown Object (File)
	Thu, Apr 11, 5:42 PM

	Unknown Object (File)
	Dec 22 2023, 11:07 PM

	Unknown Object (File)
	Dec 20 2023, 9:23 PM

	Unknown Object (File)
	Oct 24 2023, 10:14 PM

	Unknown Object (File)
	Oct 20 2023, 11:46 AM

View All 19 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...