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

ipvs: sctp implementation #949

Merged
merged 1 commit into from
May 29, 2024
Merged

ipvs: sctp implementation #949

merged 1 commit into from
May 29, 2024

Conversation

Vladimare
Copy link
Contributor

Added support for SCTP protocol which widely used in telco area.
IPVS implementation was used as a reference.

@ywc689 ywc689 self-assigned this Apr 16, 2024
@ywc689 ywc689 added the pr/to-review-codes review codes line by line and check if problem exists. label Apr 28, 2024
@ywc689
Copy link
Collaborator

ywc689 commented May 20, 2024

Thanks for the contribution!
I tested the sctp fullnat forwarding, it works well with a single forwarding lcore configured, or multiple forwarding lcores with connection redirection also enabled. Fullnat forwarding with multiple lcores does not work for rte_flows of sctp have not been programmed. The pr is accepted, and if interested, you may add support for sctp rte_flow later.

@ywc689 ywc689 added pr/codes-reviewed-ok code review passed and no problem found pr/accepted the pr passed all review stages and await to be merged and removed pr/to-review-codes review codes line by line and check if problem exists. labels May 20, 2024
@Vladimare
Copy link
Contributor Author

Thanks @ywc689 for the review and testing. I'll check the rte_flow on multiple cores later but can't promise any date at the moment.

@ywc689 ywc689 merged commit e1d2929 into iqiyi:devel May 29, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/accepted the pr passed all review stages and await to be merged pr/codes-reviewed-ok code review passed and no problem found
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants