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.

Sign up for GitHub

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

Jump to bottom

elminate tcp_client_endpoint restart on SubscribeNack #691

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

elminate tcp_client_endpoint restart on SubscribeNack #691

wants to merge 1 commit into from

Commits on May 4, 2024

  1. elminate tce restart on SubscribeNak 

    When resuming from suspend-to-ram it's likely that UDP SOME-IP works but
the TCP socket got broken. Until a new TCP socket gets established, the
remote end responds SubscribeNak. The handler will call restart()

restart() will early terminate 5 times total. Thereafter it will call
shutdown_and_close_socket_unlocked. This is really expensive if the node
sends thousands of total requests before a TCP socket gets established.

Options include:
1. do not call restart() at all on SubscribeNak. seems safe?
2. fix restart() to early-terminate better in this state, more than 5x
3. ensure that SOMEIP-SD is inhibited in this state (suspend routing?)

This commit implements COVESA#1 but the others might be alternatives
    @joeyoravec
    joeyoravec committed May 4, 2024
    Configuration menu
    Copy the full SHA
    4f2ed15 View commit details
    Browse the repository at this point in the history