Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dataplane: pass protocol number to bind() instead of socket() #169

Merged
merged 1 commit into from
Mar 19, 2015

Conversation

rlane
Copy link
Contributor

@rlane rlane commented Mar 19, 2015

Reviewer: @rlane

This makes the bind call much faster because it doesn't need to unregister the
protocol hook, which requires waiting for an RCU grace period (~10ms). Linux
3.14 and later optimize this internally, but Ubuntu 14.04 doesn't have a
recent enough kernel.

The Python socket library does the htons for us when creating the sockaddr for
bind.

This change reduces the time to run IVS automated tests by half.

This makes the bind call much faster because it doesn't need to unregister the
protocol hook, which requires waiting for an RCU grace period (~10ms). Linux
3.14 and later optimize this internally, but Ubuntu 14.04 doesn't have a recent
enough kernel.

The Python socket library does the htons for us when creating the sockaddr for
bind.

This change reduces the time to run IVS automated tests by half.
@rlane
Copy link
Contributor Author

rlane commented Mar 19, 2015

SELF-ACK

@bsn-abat
Copy link

ABAT: START_MERGE: Log file at http://vbs21/abat/2015.03.19.0810-m.oftest.master/abat.log

@bsn-abat bsn-abat merged commit 3e7d28a into floodlight:master Mar 19, 2015
bsn-abat pushed a commit that referenced this pull request Mar 19, 2015
dataplane: pass protocol number to bind() instead of socket() (#169)
@bsn-abat
Copy link

ABAT: ACCEPT: Successfully merged

In case you want to see the build log, check out :
Log file at http://vbs21/abat/2015.03.19.0810-m.oftest.master/abat.log
/cc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants