Flush the LRO ctrl as soon as lro_mbufs fills up.
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F152022145: D10432.id27562.diff
	Sun, Apr 12, 3:58 AM

	Unknown Object (File)
	Sat, Apr 11, 12:20 PM

	Unknown Object (File)
	Thu, Apr 9, 9:23 PM

	Unknown Object (File)
	Tue, Apr 7, 5:42 AM

	Unknown Object (File)
	Mon, Apr 6, 1:17 PM

	Unknown Object (File)
	Sun, Apr 5, 4:06 AM

	Unknown Object (File)
	Sat, Apr 4, 5:15 AM

	Unknown Object (File)
	Sat, Apr 4, 12:21 AM

Subscribers

Details

Reviewers

gnn
• hselasky
gallatin

Group Reviewers

network
transport

Commits

rS317390: Flush the LRO ctrl as soon as lro_mbufs fills up. There is no need to

Summary

There's no point in waiting for the driver to enqueue the next mbuf
before flushing an LRO ctrl that's already full. Flush it as soon as it
fills up instead.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 8800
Build 9168: CI src build	Jenkins
Build 9167: arc lint + arc unit

Event Timeline

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

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

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B8800: Diff 27562.Apr 20 2017, 12:49 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. Looks good to me.

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

Closed by commit rS317390: Flush the LRO ctrl as soon as lro_mbufs fills up. There is no need to (authored by np). · Explain WhyApr 24 2017, 10:35 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

netinet/

tcp_lro.c

10 lines

Diff 27562

View Options

Flush the LRO ctrl as soon as lro_mbufs fills up.ClosedPublicActions