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

[mod_timerfd] Fixed - continue timer loop after receiving a SIGSTOP #2476

Merged
merged 3 commits into from
Jan 16, 2025

Conversation

ar45
Copy link
Contributor

@ar45 ar45 commented Jun 5, 2024

When taking a snapshot of a machine which pauses the process, mod_timerfd runtime thread exits, causing all channels to wait indefinitely.

Check errno == EINTR and continue the timer loop.

@signalwire-ci
Copy link

signalwire-ci bot commented Jun 5, 2024

@ar45
Copy link
Contributor Author

ar45 commented Jan 14, 2025

@seven1240 can you please review this?

@ar45
Copy link
Contributor Author

ar45 commented Jan 14, 2025

@andywolk can you review this one?

Thank you!

src/mod/timers/mod_timerfd/mod_timerfd.c Outdated Show resolved Hide resolved
src/mod/timers/mod_timerfd/mod_timerfd.c Show resolved Hide resolved
@andywolk andywolk changed the title mod_timerfd: Fixed - continue timer loop after receiving a SIGSTOP [mod_timerfd] Fixed - continue timer loop after receiving a SIGSTOP Jan 14, 2025
@andywolk andywolk added the bug Something isn't working label Jan 14, 2025
@andywolk andywolk self-requested a review January 14, 2025 22:10
ar45 added 3 commits January 16, 2025 18:47
When taking a snapshot of a machine which pauses the process, mod_timerfd exits and freeswitch, causing all channels to wait indefinitely.

Check `errno == EINTR` and continue the timer loop.
Removed trailing whitespace to improve code formatting.
switch.h to be first
@andywolk andywolk merged commit 96de8fd into signalwire:master Jan 16, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants