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

[NOMERGE] alsa-lib: update to 1.2.5. #31286

Merged
merged 1 commit into from
Aug 18, 2021
Merged

Conversation

ericonr
Copy link
Member

@ericonr ericonr commented Jun 4, 2021

Doesn't even build on musl yet, opening PR to prevent others from updating. See alsa-project/alsa-lib#143 and alsa-project/alsa-lib#142

General

Have the results of the proposed changes been tested?

  • I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
  • I generally don't use the affected packages but briefly tested this PR

@biopsin
Copy link
Contributor

biopsin commented Jun 11, 2021

today I noticed, for the first time, running this for a while on x86_64_glibc

ALSA lib conf.c:5560:(snd_config_expand) Parse arguments error: Invalid argument
ALSA lib confmisc.c:1369:(snd_func_refer) Unable to find definition 'cards.USB-A
udio.pcm.default_playback_dmix_yes:CARD=1cards.USB-Audio.pcm.default_playback_dm
ix_yes:CARD=1'
ALSA lib conf.c:5088:(_snd_config_evaluate) function snd_func_refer returned err
or: No such file or directory
ALSA lib conf.c:5088:(_snd_config_evaluate) function snd_func_refer returned err
or: No such file or directory
ALSA lib conf.c:5576:(snd_config_expand) Evaluate error: No such file or directo

and dmix broke, think restart will resolve it for now..
EDIT: # alsactl restore yealds

No state is present for card VMini
alsa-lib parser.c:2372:(load_toplevel_config) Unable to find the top-level configuration file
'/usr/share/alsa/ucm2/ucm.conf'.
alsa-lib main.c:1402:(snd_use_case_mgr_open) error: failed to import hw:0 use case configuration -2
Found hardware: "USB-Audio" "USB Mixer" "USB13b2:00a2" "" ""
Hardware is initialized using a generic method
No state is present for card VMini

nope, dmix suddently broke out of nowhere, and I thought alsa-ucm-conf is optional

@cinerea0
Copy link
Contributor

@biopsin Do you think that if you installed alsa-ucm-conf 1.2.5 (my PR for that: #31262) it would fix the issue?

@biopsin
Copy link
Contributor

biopsin commented Jun 13, 2021

@cinerea0 I assume so, but I don't think its related to the dmix dropout.
I downgraded to resolve the problem for now, as reloading alsa did not fix it.

@biopsin
Copy link
Contributor

biopsin commented Jun 15, 2021

@ericonr try bump to 1.2.5.1 as of alsa-project/alsa-lib@abe805e
it's ready to go and local x86_64-musl build is successfull :)
Hopefully dmix is resolved too

@ericonr
Copy link
Member Author

ericonr commented Jun 15, 2021

Will import alsa-project/alsa-lib@f4c061f as well. Thanks.

@ericonr ericonr force-pushed the alsa branch 2 times, most recently from 1b8066d to af13747 Compare June 18, 2021 03:01
@cinerea0
Copy link
Contributor

cinerea0 commented Jul 2, 2021

@ericonr Have you tested this since updating to 1.2.5.1?

@ericonr
Copy link
Member Author

ericonr commented Jul 2, 2021

@cinerea0 I'm running it locally, yes. Do you think it's good to merge? I haven't looked for further bug reports.

@cinerea0
Copy link
Contributor

cinerea0 commented Jul 2, 2021

@ericonr I was going to test this PR today with my alsa-ucm-conf PR, but I can't seem to get the tarball to download. If you haven't had any more bug reports it should be fine. Actually, this has been a persistent problem for me with packages whose source tarballs come from alsa-project.org. Should we consider switching them to their GitHub sources?

@paper42
Copy link
Member

paper42 commented Jul 2, 2021

This update breaks pipewire until it's updated to 0.3.31, so we should wait for #31549

@biopsin
Copy link
Contributor

biopsin commented Jul 14, 2021

Also I'm experiencing more and frequent underruns in Reaper with this 1.2.5.1.
Downgraded again and issue solved.
Edit: well not quite, the underruns count is much more present now then before,
might be tied to kernel updates..
Edit2: currently building 5.4.132, as there are some alsa:usb-audio fixes..
Edit3: unable to track this down, my only assumtion is fixes in ext4, as I mostly diskstream from a
sata3 10,000 RPM disk. Temporary solution changing buffersize from 128ms to 512ms which is utterly crap for recording, and I will look somewhere else for help.. Im fine for merge.

@cinerea0
Copy link
Contributor

@ericonr Now that the pipewire PR has been merged, should this and the alsa-ucm-conf PRs be merged as well?

@ericonr
Copy link
Member Author

ericonr commented Jul 15, 2021

Probably should, yes...

@cinerea0
Copy link
Contributor

Are there any bug reports remaining for this?

@ericonr ericonr merged commit 8cabe7b into void-linux:master Aug 18, 2021
@ericonr
Copy link
Member Author

ericonr commented Aug 18, 2021

Shouldn't be, thanks for the persistence :)

@ericonr ericonr deleted the alsa branch August 18, 2021 22:12
@dkwo
Copy link
Contributor

dkwo commented Aug 19, 2021

This update breaks audio for me: pipewire gives the warning
[W][02053.586348][ policy-node.c: 910 rescan_node()] no node found for 38
while for Card: PipeWire, alsamixer says This sound device does not have any controls.
The issue is solved with xdowngrade /var/cache/xbps/alsa-lib-1.2.4_2.x86_64-musl.xbps

@biopsin
Copy link
Contributor

biopsin commented Aug 19, 2021

@dkwo if its relevant, try maybe debug with

Can you killall pipwire-media-session then run:
PIPEWIRE_DEBUG=4 pipewire-media-session 2>log

@dkwo
Copy link
Contributor

dkwo commented Aug 20, 2021

@biopsin Your other alsa pr solved my issue, thanks.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants