Re: [PATCH alsa-utils] alsactl: Make sure the kernel driver loaded before alsa-restore.service

[Takashi Iwai <tiwai@suse.de>]

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
>