Skip to content

fixes panicks with sox 14.6.1-1#822

Merged
baranovmv merged 1 commit intoroc-streaming:masterfrom
baranovmv:fix/arch_linux
Mar 10, 2026
Merged

fixes panicks with sox 14.6.1-1#822
baranovmv merged 1 commit intoroc-streaming:masterfrom
baranovmv:fix/arch_linux

Conversation

@baranovmv
Copy link
Copy Markdown
Member

The issue in arch Default capacity of driver list of 128 became too low, pump it up to 256

@baranovmv baranovmv requested a review from gavv November 17, 2025 18:25
@baranovmv baranovmv added the C-sound-io category: Audio I/O label Nov 17, 2025
@rocstreaming-bot
Copy link
Copy Markdown

🤖 Pull request description does not have a link to an issue.
If there is a related issue, please add it to the description using any of the supported formats.

@rocstreaming-bot rocstreaming-bot added the S-ready-for-review status: PR can be reviewed label Nov 17, 2025
@busybox11
Copy link
Copy Markdown

would be very helpful to have this properly fixed and packaged. have tested locally, not sure why ci fails with a timeout?

@dvzrv
Copy link
Copy Markdown

dvzrv commented Mar 8, 2026

Hi! 👋

This has been an issue for Arch Linux for several months. Can this be merged? 🙏

cc @grawlinson

@baranovmv baranovmv requested a review from busybox11 March 10, 2026 14:17
The issue: https://gitlab.archlinux.org/archlinux/packaging/packages/roc-toolkit/-/issues/1
Default capacity of driver list of 128 became too low, pump it up to 256

Increase valgrind timeout

Increasing MaxDrivers from 128 to 256 doubles the embedded buffer size
in Array<DriverInfo, MaxDrivers>. This enlarges the BackendMap singleton
and increases driver iteration overhead. Under valgrind's slowdown,
roc-test-public-api now occasionally exceeds the 300s timeout in tests.
@baranovmv baranovmv merged commit f37b5b9 into roc-streaming:master Mar 10, 2026
44 of 45 checks passed
@rocstreaming-bot rocstreaming-bot removed the S-ready-for-review status: PR can be reviewed label Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-sound-io category: Audio I/O

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants