Differential D10430

Frames that are not considered for LRO should not be counted in LRO stats.
ClosedPublic
Actions

Authored by np on Apr 19 2017, 11:28 PM.

Tags

None

Referenced Files

	F151294950: D10430.id27701.diff
	Tue, Apr 7, 9:41 AM

	Unknown Object (File)
	Sun, Apr 5, 8:19 PM

	Unknown Object (File)
	Thu, Mar 26, 2:03 AM

	Unknown Object (File)
	Tue, Mar 10, 1:57 PM

	Unknown Object (File)
	Tue, Mar 10, 8:06 AM

	Unknown Object (File)
	Mar 7 2026, 12:06 PM

	Unknown Object (File)
	Mar 7 2026, 6:23 AM

	Unknown Object (File)
	Mar 4 2026, 5:34 AM

View All 99 Files

Subscribers

Details

Reviewers

gnn
• hselasky
gallatin

Group Reviewers

network
transport

Commits

rS317389: Frames that are not considered for LRO should not be counted in LRO statistics.

Summary

The code will currently count any (even non-TCP) frame as queued/flushed
in the LRO stats if IFCAP_LRO is disabled or csum_flags is 0. It would
have been cleaner for tcp_lro_queue_mbuf to be able to return
TCP_LRO_CANNOT and let the caller figure out what to do. But I don't
want to touch the KPI and all the drivers so the proposed change fixes
the stats only.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

np created this revision.Apr 19 2017, 11:28 PM

Herald added a reviewer: transport. · View Herald TranscriptApr 19 2017, 11:28 PM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B8799: Diff 27561.Apr 20 2017, 12:14 AM

gnn accepted this revision.Apr 22 2017, 4:30 PM

This revision is now accepted and ready to land.Apr 22 2017, 4:30 PM

Might want to ask @gallatin for a review aswell.

gallatin accepted this revision.Apr 24 2017, 10:22 PM

Closed by commit rS317389: Frames that are not considered for LRO should not be counted in LRO statistics. (authored by np). · Explain WhyApr 24 2017, 10:32 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

netinet/

2 lines

Diff 27701

head/sys/netinet/tcp_lro.c

Loading...