-
Notifications
You must be signed in to change notification settings - Fork 94
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
use cgroup_ips map to improve mb_connect performance #227
use cgroup_ips map to improve mb_connect performance #227
Conversation
Welcome to the Merbridge OpenSource Community!👏 We're delighted to have you onboard 💘 |
Codecov Report
@@ Coverage Diff @@
## main #227 +/- ##
=======================================
Coverage 40.33% 40.33%
=======================================
Files 7 7
Lines 419 419
=======================================
Hits 169 169
Misses 236 236
Partials 14 14
Flags with carried forward coverage won't be shown. Click here to find out more. 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to optimize is_port_listen_current_ns
as well?
Great, I am trying to optimize these calls in a unified way. |
fdc994d
to
607b9ca
Compare
…bridge into improve-connect-performance
ef38107
to
ef9c205
Compare
ef9c205
to
468cc18
Compare
36c1b3f
to
a714733
Compare
a714733
to
2ee1e84
Compare
Welcome to the Merbridge OpenSource Community!👏 We're delighted to have you onboard 💘 |
@dddddai This is ready to review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For ipv6 we can follow up
98b3382
to
f2fc82d
Compare
@Xunzhuo PTAL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for working on this.
@Mergifyio refresh |
✅ Pull request refreshed |
bpf_sk_lookup_tcp may cost 50-150ns per ops, It is more expensive.
Without cache:
After cached:
The optimization results in a 20% performance improvement for connect programs.