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

[BACKPORT][CENNSO-2440] fix: force tcp app notification (#56) #57

Merged

Conversation

mogaika
Copy link
Contributor

@mogaika mogaika commented Nov 27, 2024

Add TCP_CONN_APP_DONT_NOTIF flag to fix
optimizationcase when FIN was received in SYN_RCVD state and caused unexpected notification during
scheduled removal when trying to avoid
notifications for session which was not notified
to application.

Add TCP_CONN_FORCE_NOTIFY flag to disable such
optimization and always notify session. This is
needed for UPG, since it "owns" state of session
and should be always notified about changes.

Add TCP_CONN_APP_DONT_NOTIF flag to fix
optimizationcase when FIN was received in SYN_RCVD
state and caused unexpected notification during
scheduled removal when trying to avoid
notifications for session which was not notified
to application.

Add TCP_CONN_FORCE_NOTIFY flag to disable such
optimization and always notify session. This is
needed for UPG, since it "owns" state of session
and should be always notified about changes.
@mogaika mogaika force-pushed the CENNSO-2440/backport-fix-tcp-notifications branch from 14b07e2 to 1d92206 Compare November 27, 2024 15:25
@mogaika mogaika merged commit 5a21205 into patches/v24.02.3 Nov 27, 2024
3 checks passed
@mogaika mogaika deleted the CENNSO-2440/backport-fix-tcp-notifications branch November 27, 2024 18:03
@korroot korroot added the fix Bugfix label Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants