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

StreamWorker deadlock issue in LLHLS #1633

Open
getroot opened this issue Jun 3, 2024 Discussed in #1627 · 2 comments
Open

StreamWorker deadlock issue in LLHLS #1633

getroot opened this issue Jun 3, 2024 Discussed in #1627 · 2 comments
Assignees
Labels
bug Confirmed as bug patched Patch applied

Comments

@getroot
Copy link
Sponsor Member

getroot commented Jun 3, 2024

Discussed in #1627

Originally posted by mbakumenkov May 29, 2024
Recently I have the message in server logs:
[2024-05-28 17:23:31.225] W [AW-LLHLS0:19] ManagedQueue | managed_queue.h:444 | [4051] mngq:v=#default#live:s=stream:p=pub:n=streamworker_llhls size has exceeded the threshold: queue: 937, threshold: 500, peak: 937

All outgoing traffic from server just got stopped (dropped to zero), and queue size continue rising until I restarted ome docker container.
Please help me understand what could be wrong and how this can be avoided.

@getroot getroot self-assigned this Jun 3, 2024
@getroot getroot added the in progress Being actively worked on but may take some time to complete label Jun 3, 2024
@getroot getroot changed the title Need help with log message (LLHLS managed queue size has exceeded the threshold) StreamWorker deadlock issue in LLHLS Jun 3, 2024
@getroot getroot added the bug Confirmed as bug label Jun 3, 2024
@dimiden
Copy link
Member

dimiden commented Jun 11, 2024

@mbakumenkov
I discovered a scenario where a deadlock could occur at an extremely low probability and have fixed and committed it.
You can use the airensoft/ovenmediaengine:dev image before the next version is released.
Thank you!

@dimiden dimiden added patched Patch applied and removed in progress Being actively worked on but may take some time to complete labels Jun 11, 2024
@mbakumenkov
Copy link

Thank you, @dimiden for your work!
I'll try out the new version as soon as I can

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed as bug patched Patch applied
Projects
None yet
Development

No branches or pull requests

3 participants