Re: [RFC PATCH] alsactl: Do not run udev rule before datadir is mounted

[Jaroslav Kysela <perex@perex.cz>]

Date 13.6.2014 11:05, David Henningsson wrote:
> 
> 
> On 2014-06-13 10:26, Jaroslav Kysela wrote:
>> Date 13.6.2014 10:15, David Henningsson wrote:
>>> The 90-alsa-restore.rules udev rule is primarily meant to handle hotplugging,
>>> but sometimes it is also run before /usr is mounted,
>>> and alsactl restore depends on /usr/share/alsa being present.
>>>
>>> If /usr/share/alsa is not present, we're so early in the boot
>>> process that alsa-restore.service (or upstart equivalent) will
>>> run later on.
>>>
>>> BugLink: https://bugs.launchpad.net/bugs/1289730
>>> Signed-off-by: David Henningsson <david.henningsson@canonical.com>
>>> ---
>>>
>>> I'm not really sure of all quirks in the boot process, and I haven't tested the
>>> patch either, hence the RFC above. Do you think it makes sense?
>>
>> Yes, I think that's ok. Takashi?
>>
>>> Should we also add @sbindir@ to the things to test for?
>>
>> It shouldn't harm anything. Could you prepare a new patch?
> 
> Sure, please find it attached. I was also considering adding something 
> under /var as well, but I didn't do that, because I think it will fall 
> back to an "alsactl init" in that case (right?), so it still makes some 
> sense to run alsactl without /var.

Yes, /var contents is not mandatory. Thanks, I applied it to the
alsa-utils repository.

					Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Kernel Sound Maintainer
ALSA Project; Red Hat, Inc.