-
Notifications
You must be signed in to change notification settings - Fork 679
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
Reapply netlink retries #591
Conversation
The stack sends three requests to the kernel using the netlink socket. If the kernel is e.g. busy at this point, netlink will respond with a EBUSY error value. The current code does not handle these errors gracefully, but instead silently ignores them. This can lead to vsomeip stack being stalled, and not starting certain services, e.g. service discovery. This patch helps fix these issues by sending a retry of the messages that fail.
6342a57
to
e4d1a01
Compare
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.
Please apply the following patch
0001-creted-method.zip
Only log error after retries exhausted and fix error message output, as strerror takes positive errno.
e4d1a01
to
8e7896e
Compare
Applied ✔️ |
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
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.
Thank you for the input.
* Retry failed netlink operations The stack sends three requests to the kernel using the netlink socket. If the kernel is e.g. busy at this point, netlink will respond with a EBUSY error value. The current code does not handle these errors gracefully, but instead silently ignores them. This can lead to vsomeip stack being stalled, and not starting certain services, e.g. service discovery. This patch helps fix these issues by sending a retry of the messages that fail. * Limit netlink retries to 3, improve error logging Only log error after retries exhausted and fix error message output, as strerror takes positive errno. --------- Co-authored-by: Philip Werner <phiwer@gmail.com>
Cherry-pick: Reapply netlink retries (COVESA#591)
Reapplying commit a268e2d from @phiwer with #283 which was removed by commit 826ebb8 from @DiogoPedrozza.
Additionally:
strerror
takes a positiveerrno
.