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

rx thread activate #1037

Merged
merged 1 commit into from
Jan 3, 2024
Merged

rx thread activate #1037

merged 1 commit into from
Jan 3, 2024

Conversation

cspiel1
Copy link
Collaborator

@cspiel1 cspiel1 commented Jan 2, 2024

This PR makes the use of struct rtp_source thread safe.

The RX thread in baresip calls this functions from main thread

  • rtcp_debug()
  • rtcp_stats()

@cspiel1 cspiel1 changed the title rtp: a lock for rtp_source rx thread activate Jan 2, 2024
@cspiel1 cspiel1 added the RX thread RX real-time thread label Jan 2, 2024
@sreimers sreimers merged commit e9bca3f into baresip:main Jan 3, 2024
35 checks passed
@cspiel1 cspiel1 deleted the rtp_lock_rtp_source branch January 3, 2024 17:01
@cspiel1
Copy link
Collaborator Author

cspiel1 commented Jan 3, 2024

Thanks! In the meanwhile I saw that we need more locks for RX thread. I'll look for a cleaner solution.
And then I also found: baresip/baresip#2862

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RX thread RX real-time thread
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants