Page MenuHomeFreeBSD
Differential D8072

Remove an extraneous call to soisconnected() in syncache_socket(), introduced with r261242.
ClosedPublic
Actions

Authored by jch on Sep 29 2016, 11:43 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 24, 9:10 PM
Unknown Object (File)
Nov 13 2024, 10:23 AM
Unknown Object (File)
Nov 11 2024, 8:14 PM
Unknown Object (File)
Nov 11 2024, 10:16 AM
Unknown Object (File)
Nov 10 2024, 1:19 PM
Unknown Object (File)
Nov 10 2024, 3:31 AM
Unknown Object (File)
Nov 9 2024, 9:43 PM
Unknown Object (File)
Nov 9 2024, 5:24 PM
View All 63 Files
Subscribers
kbowling

Details

Reviewers
hiren
gnn
adrian
Group Reviewers
transport
Commits
rS308426: MFC r307966:
rS308263: MFC r307966:
rS307966: Remove an extraneous call to soisconnected() in syncache_socket(),
Summary

Remove an extraneous call to soisconnected() in syncache_socket(), introduced with rS261242
The useful and expected soisconnected() call is done in tcp_do_segment().

Has been found as part of PR:212920 investigation.

Improve slightly (~2%) the maximum number of TCP accept per second.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jch updated this revision to Diff 20806.Sep 29 2016, 11:43 AM
jch retitled this revision from to Remove an extraneous call to soisconnected() in syncache_socket(), introduced with r261242..
jch updated this object.
jch edited the test plan for this revision. (Show Details)
jch added reviewers: gnn, hiren, adrian.
Herald added a reviewer: transport. · View Herald TranscriptSep 29 2016, 11:43 AM
jch updated this object.Sep 30 2016, 6:52 AM
jch edited edge metadata.
kbowling added a subscriber: kbowling.Oct 3 2016, 7:22 AM

As far as I can tell this is good. I have a box serving large volume of internet facing traffic with a data accept filter.

gnn accepted this revision.Oct 3 2016, 9:59 AM
gnn edited edge metadata.

LGTM

This revision is now accepted and ready to land.Oct 3 2016, 9:59 AM
hiren accepted this revision.Oct 12 2016, 1:03 AM
hiren edited edge metadata.

As Kevin mentioned, this looks good on prod traffic at llnw.

jch added a comment.Oct 12 2016, 6:52 AM

Thanks for your time on this review, will push this change as soon as D8211: Fix a double-free when an inp transitions to INP_TIMEWAIT state after having been dropped. is complete.

Closed by commit rS307966: Remove an extraneous call to soisconnected() in syncache_socket(), (authored by jch). · Explain WhyOct 26 2016, 3:19 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
netinet/
4 lines

Diff 21703

View Options

head/sys/netinet/tcp_syncache.c

Loading...