Differential D25132

virtio: Support non-legacy network device and queue
ClosedPublic
Actions

Authored by jrtc27 on Jun 4 2020, 7:02 PM.

Tags

None

Referenced Files

	F151354056: D25132.id.diff
	Tue, Apr 7, 8:28 PM

	Unknown Object (File)
	Tue, Apr 7, 4:33 AM

	Unknown Object (File)
	Tue, Apr 7, 1:45 AM

	Unknown Object (File)
	Mon, Apr 6, 8:38 AM

	Unknown Object (File)
	Sun, Apr 5, 2:27 AM

	Unknown Object (File)
	Sun, Mar 22, 3:08 PM

	Unknown Object (File)
	Sun, Mar 22, 8:39 AM

	Unknown Object (File)
	Tue, Mar 17, 3:36 AM

Subscribers

Details

Reviewers

br
andrew
kp
brooks
jhb

Commits

rS361944: virtio: Support non-legacy network device and queue

Summary

The non-legacy interface always defines num_buffers in the header,
regardless of whether VIRTIO_NET_F_MRG_RXBUF, just leaving it unused. We
also need to ensure our virtqueue doesn't filter out VIRTIO_F_VERSION_1
during negotiation, as it supports non-legacy transports just fine. This
fixes network packet transmission on TinyEMU.

Test Plan

Network interface still works in QEMU (which provides a legacy device by
default), but now also works on a TinyEMU-based system.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 31491
Build 29098: arc lint + arc unit

Event Timeline

jrtc27 requested review of this revision.Jun 4 2020, 7:02 PM

jrtc27 created this revision.

Harbormaster completed remote builds in B31491: Diff 72679.Jun 4 2020, 7:02 PM

jrtc27 added a parent revision: D25131: virtio_mmio: Negotiate the upper half of the feature bits too.Jun 4 2020, 7:02 PM

brooks accepted this revision.Jun 5 2020, 4:53 PM

This revision is now accepted and ready to land.Jun 5 2020, 4:53 PM

br accepted this revision.Jun 8 2020, 4:27 PM

jhb accepted this revision.Jun 8 2020, 9:38 PM

Closed by commit rS361944: virtio: Support non-legacy network device and queue (authored by jrtc27). · Explain WhyJun 8 2020, 9:51 PM

This revision was automatically updated to reflect the committed changes.

jrtc27 added a commit: rS361944: virtio: Support non-legacy network device and queue.

Herald added a subscriber: imp. · View Herald TranscriptJun 8 2020, 9:51 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

virtio/

network/

12 lines

14 lines

1 line

1 line

Diff 72679

sys/dev/virtio/network/if_vtnet.c

Loading...

sys/dev/virtio/network/if_vtnetvar.h

Loading...

sys/dev/virtio/virtio.c

Loading...

sys/dev/virtio/virtqueue.c

Loading...