-
Notifications
You must be signed in to change notification settings - Fork 765
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
[15250] Fixing datarace on listener callbacks #2889
Conversation
Signed-off-by: Miguel Barro <miguelbarro@eprosima.com>
Signed-off-by: Miguel Barro <miguelbarro@eprosima.com>
Signed-off-by: Miguel Barro <miguelbarro@eprosima.com>
@richiprosima Please test this |
Signed-off-by: Miguel Barro <miguelbarro@eprosima.com>
@richiprosima Please test this |
…void linking error on ParticipantTests. The test binary links to fastrtps which doesn't export the method thus it must rely on a header definition. Signed-off-by: Miguel Barro <miguelbarro@eprosima.com>
3898974
to
01f1c21
Compare
Signed-off-by: Miguel Barro <miguelbarro@eprosima.com>
01f1c21
to
f4a38a3
Compare
@richiprosima Please test this |
…f having the listener alive (as RTPS layer does) Signed-off-by: Miguel Barro <miguelbarro@eprosima.com>
@richiprosima Please test this |
@richiprosima please test windows |
Locally checked that dataraces
are solved. |
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
@richiprosima Please test this |
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
@richiprosima Please test this |
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
Please check if dataraces 1140, 1142 and 1152 (from this sanitation-report) can be solved with this changes. |
Those will be fixed in a follow-up PR |
Signed-off-by: Miguel Barro miguelbarro@eprosima.com
Description
Avoid data races on listener callbacks by:
Contributor Checklist
versions.md
file (if applicable).Reviewer Checklist