Differential D15262

Add netdump hooks for iflib.
ClosedPublic
Actions

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

Tags

None

Referenced Files

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

	Unknown Object (File)
	Feb 27 2024, 1:26 PM

	Unknown Object (File)
	Feb 3 2024, 3:21 AM

	Unknown Object (File)
	Jan 7 2024, 10:55 PM

	Unknown Object (File)
	Dec 12 2023, 6:37 PM

	Unknown Object (File)
	Dec 9 2023, 10:40 PM

	Unknown Object (File)
	Dec 9 2023, 10:15 PM

	Unknown Object (File)
	Nov 27 2023, 12:48 AM

View All 31 Files

Subscribers

Details

Reviewers

shurd
kmacy

Commits

rS333291: Add netdump support to iflib.

Test Plan

Tested with em (me), igb (me), ixgbe (sbruno), bnxt (sbruno)

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

Herald added a reviewer: shurd. · View Herald TranscriptMay 1 2018, 5:24 PM

Herald added a subscriber: ae. · View Herald Transcript

Harbormaster completed remote builds in B16385: Diff 42049.May 1 2018, 5:24 PM

markj edited the test plan for this revision. (Show Details)May 1 2018, 5:59 PM

markj added a reviewer: kmacy.

shurd added inline comments.May 1 2018, 6:18 PM

sys/net/iflib.c
6063 ↗	(On Diff #42049)	Wouldn't we need a NETDUMP_END case that calls txd_flush() in case it wasn't called in the last iflib_txd_db_check()?
6102 ↗	(On Diff #42049)	This is the first thing that iflib_completed_tx_reclaim() does... do we need to call it here too?

markj added inline comments.May 1 2018, 6:21 PM

sys/net/iflib.c
6063 ↗	(On Diff #42049)	The netdump transmit method already forces a flush after every packet.
6102 ↗	(On Diff #42049)	Oh, indeed. Looks like we do not - I'm not sure why I included it here.

shurd added inline comments.May 1 2018, 6:50 PM

sys/net/iflib.c
6063 ↗	(On Diff #42049)	Oh... is that just to simplify state, or to lower TX latency?

markj added inline comments.May 1 2018, 6:53 PM

sys/net/iflib.c
6063 ↗	(On Diff #42049)	It's basically a requirement of the interface. In some cases the client code will wait for an ACK from the server after transmitting a packet. This could be changed, but as you suggested, it's simpler to just flush after each transmit.

Remove unneeded iflib_tx_credits_update() call

Harbormaster completed remote builds in B16394: Diff 42063.May 2 2018, 2:47 PM

markj marked 2 inline comments as done.May 2 2018, 2:47 PM

lgtm

This revision is now accepted and ready to land.May 2 2018, 5:11 PM

kmacy accepted this revision.May 3 2018, 3:44 AM

Closed by commit rS333291: Add netdump support to iflib. (authored by markj). · Explain WhyMay 6 2018, 12:58 AM

This revision was automatically updated to reflect the committed changes.

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

Revision Contents
Changeset List

Path

Size

head/

sys/

net/

93 lines

Diff 42187

head/sys/net/iflib.c

Loading...