pcm_meter.c issue at s16_update
Hi,
It would be great to have a fix/patch from Pavel in the main codebase. I've tested his fix and it worked fine.
Just to remind what the issue is - while switching from one web radio station to another there is a high CPU usage for about 20 seconds. Because of that high CPU consumption almost all other processes are blocked. The fix from Pavel helps to avoid this situation.
Without the ALSA patch I need to make a custom ALSA build every time I release my software. This is very annoying.
Thanks in advance!
Hi,
Dne 15. 09. 20 v 5:40 Go Peppy napsal(a):
Just to remind what the issue is - while switching from one web radio station to another there is a high CPU usage for about 20 seconds. Because of that high CPU consumption almost all other processes are blocked.
20 seconds is only due to 32bit kernel on RPi. On 64bit machines the boundary value is usually many orders of magnitude larger, basically making the method never finish.
With regards,
Pavel.
Dne 17. 09. 20 v 21:13 Pavel Hofman napsal(a):
Hi,
Dne 15. 09. 20 v 5:40 Go Peppy napsal(a):
Just to remind what the issue is - while switching from one web radio station to another there is a high CPU usage for about 20 seconds. Because of that high CPU consumption almost all other processes are blocked.
20 seconds is only due to 32bit kernel on RPi. On 64bit machines the boundary value is usually many orders of magnitude larger, basically making the method never finish.
Could you try this patch?
https://github.com/alsa-project/alsa-lib/commit/a6c8ac0c85ca1b16684a687c7000...
It should prevent the big copies which are definitely nonsense and this change does not hide the real problem.
Jaroslav
With regards,
Pavel.
Hi Jaroslav,
I've tested the patch. It works fine. I didn't experience the original issue with high CPU utilization after applying the patch. When is the next planned release?
Thanks! PPP
On Tue, Oct 13, 2020 at 10:35 AM Jaroslav Kysela perex@perex.cz wrote:
Dne 17. 09. 20 v 21:13 Pavel Hofman napsal(a):
Hi,
Dne 15. 09. 20 v 5:40 Go Peppy napsal(a):
Just to remind what the issue is - while switching from one web radio station to another there is a high CPU usage for about 20 seconds.
Because
of that high CPU consumption almost all other processes are blocked.
20 seconds is only due to 32bit kernel on RPi. On 64bit machines the boundary value is usually many orders of magnitude larger, basically making the method never finish.
Could you try this patch?
https://github.com/alsa-project/alsa-lib/commit/a6c8ac0c85ca1b16684a687c7000...
It should prevent the big copies which are definitely nonsense and this change does not hide the real problem.
JaroslavWith regards,
Pavel.
-- Jaroslav Kysela perex@perex.cz Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
participants (3)
-
Go Peppy -
Jaroslav Kysela -
Pavel Hofman