Page MenuHomeFreeBSD
Differential D2542

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
F169032736: D2542.id.diff
Mon, Aug 31, 4:46 AM
F168944352: D2542.id5377.diff
Sun, Aug 30, 9:31 PM
F168925790: D2542.id5377.diff
Sun, Aug 30, 7:15 PM
Unknown Object (File)
Sat, Aug 29, 4:45 AM
Unknown Object (File)
Thu, Aug 27, 4:55 AM
Unknown Object (File)
Wed, Aug 26, 3:11 PM
Unknown Object (File)
Wed, Aug 26, 1:05 AM
Unknown Object (File)
Tue, Aug 25, 11:57 PM
View All 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 Passed
Unit
No Test Coverage

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

PathSize
sys/
dev/
sfxge/
19 lines

Diff 5377

View Options

sys/dev/sfxge/sfxge_rx.c

Loading...