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

Fix SCTP fragment handling #8837

Open
wants to merge 5 commits into
base: maint-26
Choose a base branch
from

Conversation

RaimoNiskanen
Copy link
Contributor

Getting ERRNO_BLOCK immediately could cause a buffer with message head to be wasted since it was forgotten that there was valid data in it.

@RaimoNiskanen RaimoNiskanen added this to the OTP-26.2.5.4 milestone Sep 20, 2024
@RaimoNiskanen RaimoNiskanen self-assigned this Sep 20, 2024
Copy link
Contributor

github-actions bot commented Sep 20, 2024

CT Test Results

    3 files    143 suites   46m 40s ⏱️
1 572 tests 1 523 ✅ 49 💤 0 ❌
2 115 runs  2 042 ✅ 73 💤 0 ❌

Results for commit 73e1aa1.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

Getting ERRNO_BLOCK immediately could cause a buffer with message head
to be wasted since it was forgotten that there was valid data in it.
Don't used the async return value functions except for TCP
that would block.  All other cases are synchronous so return
the result (ok or {ok,AssocId}) directly from port control.
@RaimoNiskanen RaimoNiskanen force-pushed the raimo/sctp/block-wastes-buffer/ERIERL-1133/OTP-19235 branch from 73e1aa1 to 620fab3 Compare September 27, 2024 13:08
@RaimoNiskanen
Copy link
Contributor Author

Now rebased onto OTP-26.2.5, current OTP 26 patch base.

@RaimoNiskanen RaimoNiskanen force-pushed the raimo/sctp/block-wastes-buffer/ERIERL-1133/OTP-19235 branch from 0427f24 to ea4be99 Compare October 1, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant