Message ID | 20201128203101.7333-1-tiwai@suse.de (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [alsa-utils] alsactl: Make sure the kernel driver loaded before alsa-restore.service | expand |
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 >
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
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(+)