sfxge: LRO may be done only if checksums are OK
ClosedPublic
Actions

Authored by arybchik on May 14 2015, 1:02 PM.

Tags

None

Referenced Files

	F132463593: D2542.id5377.diff
	Fri, Oct 17, 3:46 AM

	Unknown Object (File)
	Sep 2 2025, 9:06 AM

	Unknown Object (File)
	Aug 23 2025, 10:45 PM

	Unknown Object (File)
	Aug 18 2025, 4:19 AM

	Unknown Object (File)
	Jul 1 2025, 6:56 AM

	Unknown Object (File)
	Jul 1 2025, 4:55 AM

	Unknown Object (File)
	Jun 30 2025, 7:05 AM

	Unknown Object (File)
	Jun 25 2025, 10:08 PM

View All 63 Files

Subscribers

imp

network

Details

Reviewers

gnn

Commits

rS282940: sfxge: LRO may be done only if checksums are OK

Summary

Also it is cheaper to check Rx descriptor flags than TCP protocol in IP
header.

Sponsored by: Solarflare Communications, Inc.
MFC after: 2 days

Test Plan

Build tested. Driver loaded. LRO works fine.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

arybchik updated this revision to Diff 5377.May 14 2015, 1:02 PM

arybchik retitled this revision from to sfxge: LRO may be done only if checksums are OK.

arybchik updated this object.

arybchik edited the test plan for this revision. (Show Details)

arybchik added a reviewer: gnn.

Herald added a subscriber: imp. · View Herald TranscriptMay 14 2015, 1:02 PM

arybchik added a subscriber: network.May 14 2015, 1:03 PM

gnn accepted this revision.May 14 2015, 1:06 PM

gnn edited edge metadata.

This revision is now accepted and ready to land.May 14 2015, 1:06 PM

Closed by commit rS282940: sfxge: LRO may be done only if checksums are OK (authored by arybchik). · Explain WhyMay 15 2015, 6:48 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

sfxge/

sfxge_rx.c

19 lines

Diff 5396

View Options

head/sys/dev/sfxge/sfxge_rx.c

sfxge: LRO may be done only if checksums are OKClosedPublicActions