Differential D15261

Add netdump hooks for vtnet(4).
ClosedPublic
Actions

Authored by markj on May 1 2018, 5:24 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Apr 27, 8:24 AM

	Unknown Object (File)
	Fri, Apr 26, 6:13 PM

	Unknown Object (File)
	Mar 21 2024, 6:13 PM

	Unknown Object (File)
	Dec 20 2023, 2:52 AM

	Unknown Object (File)
	Oct 15 2023, 1:22 AM

	Unknown Object (File)
	Oct 1 2023, 9:31 PM

	Unknown Object (File)
	Aug 23 2023, 2:36 PM

	Unknown Object (File)
	Jul 18 2023, 12:44 PM

View All 18 Files

Subscribers

Details

Reviewers

bryanv
julian

Commits

rS333290: Add netdump support to vtnet(4).

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.May 1 2018, 5:24 PM

Harbormaster completed remote builds in B16384: Diff 42048.May 1 2018, 5:24 PM

markj added a reviewer: bryanv.May 1 2018, 6:00 PM

julian accepted this revision.May 2 2018, 6:24 AM

This revision is now accepted and ready to land.May 2 2018, 6:24 AM

bryanv added inline comments.May 3 2018, 4:39 AM

sys/dev/virtio/network/if_vtnet.c
4029 ↗	(On Diff #42048)	vtnet_txq_notify can return non-zero but I don't think you consider that an error: if it was able to drain enough completed mbufs it returns non-zero so that the caller can continue transmitting. Since you're transmitting one mbuf here and are polling I think you can just ignore the return value.

markj marked an inline comment as done.May 4 2018, 7:39 PM

bryanv accepted this revision.May 4 2018, 7:42 PM

Closed by commit rS333290: Add netdump support to vtnet(4). (authored by markj). · Explain WhyMay 6 2018, 12:54 AM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptMay 6 2018, 12:54 AM

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

virtio/

network/

81 lines

Diff 42186

head/sys/dev/virtio/network/if_vtnet.c

Loading...