Differential D27922

if_vtnet: Schedule Rx task if pending items when enabling interrupt
ClosedPublic
Actions

Authored by bryanv on Jan 2 2021, 9:17 PM.

Tags

None

Referenced Files

	F83307018: D27922.id.diff
	Wed, May 8, 4:45 PM

	Unknown Object (File)
	Dec 20 2023, 6:56 PM

	Unknown Object (File)
	Dec 20 2023, 4:12 AM

	Unknown Object (File)
	Nov 22 2023, 3:01 AM

	Unknown Object (File)
	Nov 10 2023, 5:38 AM

	Unknown Object (File)
	Oct 10 2023, 4:35 PM

	Unknown Object (File)
	Oct 9 2023, 4:29 AM

	Unknown Object (File)
	Oct 7 2023, 5:10 AM

View All 15 Files

Subscribers

Details

Reviewers

Commits

rG4f18e23f8432: if_vtnet: Schedule Rx task if pending items when enabling interrupt

Summary

Prior to V1, the driver would enable interupts and then notify the
host that DRIVER_OK. Since for V1, DRIVER_OK needs to be set before
notifying the virtqueues, there may be items in the queues waiting
to be processed by the time interrupts are enabled.

This fixes a bug where the first Rx queue would appear stuck, only
being fixable by an interface down/up.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 35887
Build 32776: arc lint + arc unit

Event Timeline

bryanv created this revision.Jan 2 2021, 9:17 PM

bryanv held this revision as a draft.

Herald added a subscriber: imp. · View Herald TranscriptJan 2 2021, 9:17 PM

Harbormaster completed remote builds in B35887: Diff 81545.Jan 2 2021, 9:17 PM

bryanv added a reviewer: grehan.Jan 2 2021, 9:17 PM

bryanv published this revision for review.Jan 2 2021, 10:05 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jan 19 2021, 5:12 AM

Closed by commit rG4f18e23f8432: if_vtnet: Schedule Rx task if pending items when enabling interrupt (authored by bryanv). · Explain Why

This revision was automatically updated to reflect the committed changes.

bryanv added a commit: rG4f18e23f8432: if_vtnet: Schedule Rx task if pending items when enabling interrupt.

Revision Contents
Changeset List

Path

Size

sys/

dev/

virtio/

network/

14 lines

Diff 81545

sys/dev/virtio/network/if_vtnet.c

Loading...