[PATCH alsa-utils] alsactl: Make sure the kernel driver loaded before alsa-restore.service
alsa-restore.service requires the sound devices having been set up beforehand. However, systemd tends to start this service too early before the driver setup, which leads to the unsuccessful state restore.
For assuring the kernel driver setup before this service, add the dependency on sound.target.
BugLink: https://bugzilla.suse.com/show_bug.cgi?id=1179363 Signed-off-by: Takashi Iwai tiwai@suse.de --- alsactl/alsa-restore.service.in | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/alsactl/alsa-restore.service.in b/alsactl/alsa-restore.service.in index a84c2e842444..104b97a6ef68 100644 --- a/alsactl/alsa-restore.service.in +++ b/alsactl/alsa-restore.service.in @@ -8,6 +8,8 @@ Description=Save/Restore Sound Card State ConditionPathExists=!@daemonswitch@ ConditionPathExistsGlob=/dev/snd/control* ConditionPathExists=@asoundrcfile@ +Requires=sound.target +After=sound.target
[Service] Type=oneshot
On Sat, 28 Nov 2020 21:31:01 +0100, Takashi Iwai wrote:
alsa-restore.service requires the sound devices having been set up beforehand. However, systemd tends to start this service too early before the driver setup, which leads to the unsuccessful state restore.
For assuring the kernel driver setup before this service, add the dependency on sound.target.
BugLink: https://bugzilla.suse.com/show_bug.cgi?id=1179363 Signed-off-by: Takashi Iwai tiwai@suse.de
Scratch this. There is already certain dependencies between this service and sound.target. It needs more investigation why it doesn't work as expected.
Takashi
alsactl/alsa-restore.service.in | 2 ++
1 file changed, 2 insertions(+)
diff --git a/alsactl/alsa-restore.service.in b/alsactl/alsa-restore.service.in index a84c2e842444..104b97a6ef68 100644 --- a/alsactl/alsa-restore.service.in +++ b/alsactl/alsa-restore.service.in @@ -8,6 +8,8 @@ Description=Save/Restore Sound Card State ConditionPathExists=!@daemonswitch@ ConditionPathExistsGlob=/dev/snd/control* ConditionPathExists=@asoundrcfile@ +Requires=sound.target +After=sound.target
[Service] Type=oneshot -- 2.26.2
participants (1)
-
Takashi Iwai