Re: [PATCH 1/4] kernel/module: Add AUTOLOAD option

[Jan Kiszka <jan.kiszka@siemens.com>]

On 2018-03-07 08:36, Alexander Smirnov wrote:
> 
> 
> On 03/07/2018 10:19 AM, Jan Kiszka wrote:
>> On 2018-03-06 13:40, Alexander Smirnov wrote:
>>> Add possibility to set kernel module to be autoloaded.
>>> Also add run dependency from 'systemd' because this package owns the
>>> content of '/etc/modules-load.d'.
>>>
>>> Signed-off-by: Alexander Smirnov <asmirnov@ilbers.de>
>>> ---
>>>   meta/recipes-kernel/linux-module/files/debian/control | 2 +-
>>>   meta/recipes-kernel/linux-module/module.inc           | 7 +++++++
>>>   2 files changed, 8 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/meta/recipes-kernel/linux-module/files/debian/control
>>> b/meta/recipes-kernel/linux-module/files/debian/control
>>> index 707f857..0eae9a4 100644
>>> --- a/meta/recipes-kernel/linux-module/files/debian/control
>>> +++ b/meta/recipes-kernel/linux-module/files/debian/control
>>> @@ -7,5 +7,5 @@ Maintainer: ISAR project <isar-users@googlegroups.com>
>>>     Package: @PN@
>>>   Architecture: any
>>> -Depends: linux-image-@KERNEL_NAME@
>>> +Depends: linux-image-@KERNEL_NAME@, systemd
>>
>> Why this, BTW?
> 
> See comment to the patch:
> 
> 8<--
> Also add run dependency from 'systemd' because this package owns the
> content of '/etc/modules-load.d'.
> 8<--
> 
> So if this dependency is not set, the module.postinst script could be
> run *before* '/etc/modules*' infrastructure is created.

...and that means?

Also, is systemd now the only provider of this infrastructure in Debian?

Jan