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.

Sign up for GitHub

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

Jump to bottom

Hang after duration expires #5

Closed
gavv opened this issue Jul 10, 2020 · 2 comments
Closed

Hang after duration expires #5

gavv opened this issue Jul 10, 2020 · 2 comments
Assignees
@gavv
Labels
bug Something isn't working

Comments

@gavv
Copy link
Owner

gavv commented Jul 10, 2020

After duration expires, signal-estimator sometimes hangs (sleeps indefinitely) instead of exiting.

Good invocation example:

$ ./bin/signal-estimator -o pulse -i pulse -d 3
opening alsa writer for device pulse
suggested_latency: 8000 us
suggested_buffer_size: 384 samples
selected_buffer_time: 12000 us
selected_buffer_size: 576 samples
selected_period_time: 4000 us
selected_period_size: 192 samples
opening alsa reader for device pulse
suggested_latency: 8000 us
suggested_buffer_size: 384 samples
selected_buffer_time: 12000 us
selected_buffer_size: 576 samples
selected_period_time: 4000 us
selected_period_size: 192 samples
can't enable real-time scheduling policy
can't enable real-time scheduling policy
latency:  sw+hw  23.839ms  hw  15.845ms  hw_avg5  15.845ms
latency:  sw+hw  50.011ms  hw  35.244ms  hw_avg5  25.545ms
latency:  sw+hw  46.396ms  hw  39.304ms  hw_avg5  30.131ms
$

Bad invocation example:

$ ./bin/signal-estimator -o pulse -i pulse -d 2
opening alsa writer for device pulse
suggested_latency: 8000 us
suggested_buffer_size: 384 samples
selected_buffer_time: 12000 us
selected_buffer_size: 576 samples
selected_period_time: 4000 us
selected_period_size: 192 samples
opening alsa reader for device pulse
suggested_latency: 8000 us
suggested_buffer_size: 384 samples
selected_buffer_time: 12000 us
selected_buffer_size: 576 samples
selected_period_time: 4000 us
selected_period_size: 192 samples
can't enable real-time scheduling policy
can't enable real-time scheduling policy
latency:  sw+hw  19.113ms  hw  12.970ms  hw_avg5  12.970ms
latency:  sw+hw  43.722ms  hw  36.153ms  hw_avg5  24.562ms
<---------------- HANG
@gavv gavv added bug Something isn't working help wanted Contributions are welcome labels Jul 10, 2020
@gavv gavv mentioned this issue Jul 10, 2020
Merged
@gavv gavv added the good first issue Good for newcomers label Jul 10, 2020
@gavv gavv removed the hacktoberfest label Dec 3, 2020
@gavv gavv mentioned this issue Jul 20, 2021
Merged
gavv added a commit that referenced this issue Mar 30, 2023
@gavv
Fix hang in alsa_close (#5)
df44a5b
@gavv
Copy link
Owner Author

gavv commented Mar 30, 2023

Fixed by df44a5b

@gavv gavv removed help wanted Contributions are welcome good first issue Good for newcomers labels Mar 30, 2023
@gavv gavv self-assigned this Mar 30, 2023
@gavv gavv closed this as completed Mar 30, 2023
@gavv gavv mentioned this issue Mar 30, 2023
Closed
@gavv
Copy link
Owner Author

gavv commented May 14, 2023

Included into v0.0.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gavv gavv

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@gavv