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

PCM::direct_mmap_capture errors with ENOTTY on ioctl #118

Closed
tranzystorekk opened this issue Mar 2, 2024 · 5 comments · Fixed by #119
Closed

PCM::direct_mmap_capture errors with ENOTTY on ioctl #118

tranzystorekk opened this issue Mar 2, 2024 · 5 comments · Fixed by #119

Comments

@tranzystorekk
Copy link

After updating to version 0.9.0 I get an ENOTTY error when attempting to call PCM::direct_mmap_capture():

Error: ALSA function 'SNDRV_PCM_IOCTL_CHANNEL_INFO' failed with error 'Not a typewriter (25)'

The environment is:

Void Linux (vmware guest)
Linux 6.6.18
Vmware Workstation Player 17.5.0

@diwic
Copy link
Owner

diwic commented Mar 4, 2024

Interesting. Are you sure the difference is between two different alsa-rs versions and not something else in the environment (e g, not all drivers/devices support mmap)?

@tranzystorekk
Copy link
Author

I'm pretty sure, this reproduces when I update this project from 0.8.1 to 0.9.0: https://codeberg.org/tranzystorekk/spectroscope, here's where I call the API: https://codeberg.org/tranzystorekk/spectroscope/src/commit/376cfbf1151a3d77615b20822965fb728ea5ffa9/src/capture.rs#L32

For an overview, the program attempts to open an alsa loopback device to read audio for visualizing.

@diwic
Copy link
Owner

diwic commented Mar 4, 2024

@Jake-Shadle Hi, it's probably a regression from your "nuking nix" PR, would you mind having a look at this bug? Thanks!

@Jake-Shadle
Copy link
Contributor

Sorry about that, fixed in #119

@tranzystorekk
Copy link
Author

Just tested with the PR commit and the issue is fixed, thank you @Jake-Shadle ❤️

@diwic diwic closed this as completed in #119 Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants