http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c2 --- Comment #2 from Karl Mistelberger --- (In reply to Takashi Iwai from comment #1)

This is likely no real bug, but the intended behavior. There are two paths calling alsactl-restore: one is directly via udev, another is via systemd. The latter case was "fixed" through your bug report, and I guess this will make both calls more concurrent. So one of them is skipped due to another being running at the same time.

I don't think so. alsactl fails to restore settings and uses defaults as indicated by the following line: alsactl[799]: Hardware is initialized using a generic method This is definitely not user intention. Users want to have restored their settings. Why should there exist two concurrent initialisations of the sound system? In my opinion initialization should be done exactly once. And it should work reliably. -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c4 --- Comment #4 from Takashi Iwai --- The fixed alsa-utils package is being built in OBS home:tiwai:branches:openSUSE:Leap:15.2:Update/alsa-utils repo for Leap 15.2 and OBS home:tiwai:branches:multimedia:libs/alsa-utils repo for TW. The x86_64 build is still ongoing as of now. Please give it a try later and let me know if the spurious init is gone (while the lock error should remain). -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c6 --- Comment #6 from Takashi Iwai --- Through the upstream discussions, we fixed the issue differently: the error must be a race at a thin timing between two opens of the lock file, and this is fixed by retrying the normal open again after EEXIST error. Also, I noticed the harmful unit condition in alsa-restore.service, and sound-extra.service is also fixed. The fixed packages are being built in the same OBS repo. Please give this a try again later. -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c8 --- Comment #8 from Karl Mistelberger --- More trouble: 3400G:~ # journalctl -b -u alsa-restore.service -o short-monotonic -- Logs begin at Tue 2020-12-01 05:11:44 CET, end at Sat 2020-12-12 07:28:38 CET. -- [ 5.423312] 3400G systemd[1]: Condition check resulted in Save/Restore Sound Card State being skipped. 3400G:~ # systemctl status alsa-restore.service ��� alsa-restore.service - Save/Restore Sound Card State Loaded: loaded (/usr/lib/systemd/system/alsa-restore.service; static) Active: inactive (dead) Condition: start condition failed at Sat 2020-12-12 07:28:03 CET; 1min 20s ago ������ ConditionPathExistsGlob=/dev/snd/control* was not met Dec 12 07:28:03 3400G systemd[1]: Condition check resulted in Save/Restore Sound Card State being skipped. 3400G:~ # However: 3400G:~ # ll /dev/snd/control* crw-rw----+ 1 root audio 116, 6 Dec 12 07:28 /dev/snd/controlC1 crw-rw----+ 1 root audio 116, 12 Dec 12 07:28 /dev/snd/controlC2 3400G:~ # :-( -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c10 --- Comment #10 from Karl Mistelberger --- Double checked: 3400G:~ # cat /etc/udev/rules.d/sound.rules SUBSYSTEM=="sound", KERNEL=="controlC*", TAG+="systemd", ENV{SYSTEMD_WANTS}+="sound.target", ENV{SYSTEMD_USER_WANTS}+="sound.target" 3400G:~ # systemctl cat alsa-restore.service # /usr/lib/systemd/system/alsa-restore.service # # Note that two different ALSA card state management schemes exist and they # can be switched using a file exist check - /etc/alsa/state-daemon.conf . # [Unit] Description=Save/Restore Sound Card State ConditionPathExists=!/etc/alsa/state-daemon.conf ConditionPathExistsGlob=/dev/snd/control* [Service] Type=oneshot RemainAfterExit=true ExecStart=-/usr/sbin/alsactl restore ExecStop=-/usr/sbin/alsactl store 3400G:~ # Found ConditionPathExistsGlob=/dev/snd/control* to be shaky when testing some services. -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c12 --- Comment #12 from Karl Mistelberger --- (In reply to Takashi Iwai from comment #11)

(In reply to Karl Mistelberger from comment #10)

...

3400G:~ # cat /etc/udev/rules.d/sound.rules

Where did you have this file..? In anyway, please remove this and retest. This extra one might confuse systemd dependencies. Instead, make sure that you corrected /usr/lib/udev/rules.d/99-systemd.conf to replace "card*" with "controlC*".

According to man udev /etc/udev/rules.d/sound.rules should work. And it does work, but not reliably! I am now testing with modified vendor file: 3400G:~ # grep sound /usr/lib/udev/rules.d/99-systemd.rules SUBSYSTEM=="sound", KERNEL=="controlC*", TAG+="systemd", ENV{SYSTEMD_WANTS}+="sound.target", ENV{SYSTEMD_USER_WANTS}+="sound.target" 3400G:~ # -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c14 --- Comment #14 from Karl Mistelberger --- (In reply to Takashi Iwai from comment #13)

The point is that this rule is utterly superfluous with the recent alsa-utils package. alsa-utils package already provides the own alsactl invocation in /usr/lib/udev/90-alsa-restore.rules, and the systemd with sound.target symlinks in /usr/lib/systemd/system/sound.target.wants/*. No need (or even harmful) to add the extra rules.

Deleted line: SUBSYSTEM=="sound", KERNEL=="controlC*", TAG+="systemd", ENV{SYSTEMD_WANTS}+="sound.target", ENV{SYSTEMD_USER_WANTS}+="sound.target" from /usr/lib/udev/rules.d/99-systemd.rules and rebooted: No sound! -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c16 --- Comment #16 from Karl Mistelberger --- (In reply to Takashi Iwai from comment #15)

Of course it would break. I asked to delete an extra file you've added, /etc/udev/rules.d/sound.rules. This is the superfluous thing.

Being reluctant to modify a file owned by systemd I added the correct rule to /etc/udev/rules.d/sound.rules. That worked well on host erlangen. When making the same change on host 3400G I observed shaky behavior. That was caused by the original rule in file /usr/lib/udev/rules.d/99-systemd.rules. I commented the line out and everything was fine. No more double initialization as in comment #2: erlangen:~ # systemctl list-units '*sound*' UNIT LOAD ACTIVE SUB DESCRIPTION sys-devices-pci0000:00-0000:00:01.0-0000:01:00.1-sound-card1-controlC1.device loaded active plugged /sys/devices/pci0000:00/0000:00:01.0/0000:01:00.1/sound/card1/controlC1 sys-devices-pci0000:00-0000:00:1f.3-sound-card0-controlC0.device loaded active plugged /sys/devices/pci0000:00/0000:00:1f.3/sound/card0/controlC0 sound.target loaded active active Sound Card erlangen:~ # alsa-utils-1.2.4-220.1.x86_64 now works reliably on both hosts! -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c18 --- Comment #18 from Karl Mistelberger --- Tried Leap 15.2: Leap:~ # zypper in http://download.opensuse.org/repositories/home:/tiwai:/branches:/multimedia:... Loading repository data... Reading installed packages... Resolving package dependencies... Problem: nothing provides libatopology.so.2()(64bit) needed by alsa-utils-1.2.4-220.1.x86_64 Solution 1: do not install alsa-utils-1.2.4-220.1.x86_64 Solution 2: break alsa-utils-1.2.4-220.1.x86_64 by ignoring some of its dependencies Choose from above solutions by number or cancel [1/2/c/d/?] (c): Leap:~ # zypper in http://download.opensuse.org/repositories/home:/tiwai:/branches:/multimedia:... Loading repository data... Reading installed packages... Resolving package dependencies... Problem: nothing provides libatopology.so.2()(64bit) needed by alsa-utils-1.2.4-lp152.220.1.x86_64 Solution 1: do not install alsa-utils-1.2.4-lp152.220.1.x86_64 Solution 2: break alsa-utils-1.2.4-lp152.220.1.x86_64 by ignoring some of its dependencies Choose from above solutions by number or cancel [1/2/c/d/?] (c): Leap:~ # -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1179904 http://bugzilla.opensuse.org/show_bug.cgi?id=1179904#c20 --- Comment #20 from Karl Mistelberger --- (In reply to Takashi Iwai from comment #19)

As mentioned in comment 4, the Leap 15.2 package is found in another OBS repo, home:tiwai:branches:openSUSE:Leap:15.2:Update/alsa-utils

Installed http://download.opensuse.org/repositories/home:/tiwai:/branches:/openSUSE:/L... fixed /usr/lib/udev/rules.d/99-systemd.rules, rebooted and got what I expected: Leap:~ # systemctl list-units "*sound*" UNIT LOAD ACTIVE SUB DESCRIPTION sys-devices-pci0000:00-0000:00:08.1-0000:08:00.1-sound-card0-controlC0.device loaded active plugged /sys/devices/pci0000:00/0000:00:08.1/0000:08:00.1/sound/card0/controlC0 sys-devices-pci0000:00-0000:00:08.1-0000:08:00.6-sound-card1-controlC1.device loaded active plugged /sys/devices/pci0000:00/0000:00:08.1/0000:08:00.6/sound/card1/controlC1 sound.target loaded active active Sound Card Both Tumbleweed and Leap 15.2 are fine now! -- You are receiving this mail because: You are on the CC list for the bug.