From: "Antonin Godard" <antonin.godard@bootlin.com>
To: <dixitparmar19@gmail.com>, <docs@lists.yoctoproject.org>
Subject: Re: [docs] [PATCH v2] ref-manual: document KERNEL_SPLIT_MODULES variable
Date: Wed, 18 Jun 2025 15:42:55 +0200 [thread overview]
Message-ID: <DAPPBO0JXU6C.38BKO55VAYW0G@bootlin.com> (raw)
In-Reply-To: <26140.1750252773043231023@lists.yoctoproject.org>
On Wed Jun 18, 2025 at 3:19 PM CEST, Dixit Parmar via lists.yoctoproject.org wrote:
> On Wed, Jun 18, 2025 at 06:30 PM, Antonin Godard wrote:
>
>>
>> On Wed Jun 18, 2025 at 2:14 PM CEST, Dixit Parmar via
>> lists.yoctoproject.org wrote:
>>
>>> Will fix the ref->term typo and the take your suggestion on rewording.
>>>
>>> There seem to be confusion related to the kernel package name, however I
>>> feel
>>> it was additional information. I shall remove the mention of the kernel
>>> package name (keep the variable reference only) for simplicity in next
>>> version.
>>
>> Looking forward to it!
>>
>>
>>> Yes, you would be able to install "all" the kernel-modules without
>>> bothering
>>> about KERNEL_SPLIT_MODULES. The issue is having autoload .conf file being
>>> installed or not when KERNEL_MODULE_AUTOLOAD is having your modules listed
>>> to
>>> be autoloader. When KERNEL_SPLIT_MODULES = 0 the .conf files for driver
>>> autoload and probeconf is not getting generated.
>>
>> This sounds like a bug to me, no? Why would someone expect setting
>> KERNEL_SPLIT_MODULES to 0 have the autoload files missing?
>>
>> I ran a some tests. So when I set KERNEL_SPLIT_MODULES to 1, and
>> KERNEL_MODULE_AUTOLOAD to "xt_limit", a kernel-module-xt-limit.ipk package
>> is
>> generated like so:
>>
>> ├── etc
>> │ └── modules-load.d
>> │ └── xt_limit.conf
>> └── lib
>> └── modules
>> └── 6.12.30-yocto-standard
>> └── kernel
>> └── net
>> └── netfilter
>> └── xt_limit.ko
>>
>> When KERNEL_SPLIT_MODULES is set to 0, only one kernel-modules.ipk package
>> is
>> generated, and does _not_ contain xt_limit.conf, but only xt_limit.ko. To
>> me
>> this sounds like a bug, what do you think?
>
> Exactly, that's the issue reported https://bugzilla.yoctoproject.org/show_bug.cgi?id=15145 ( https://bugzilla.yoctoproject.org/show_bug.cgi?id=15145 ) and I am fixing it here with this solution https://lists.openembedded.org/g/openembedded-core/message/218290.
>
>>
>> I would _not_ describe that behavior in the documentation for the
>> KERNEL_SPLIT_MODULES variables.
>
> True, that's why the documentation does not mention anything about the conf files.
>
>>
>> The fact that "kernel-modules" is still available when
>> KERNEL_SPLIT_MODULES = 1
>> is simply because a "fake" kernel-modules.ipk package is generated, and
>> depends
>> on the other kernel-module-* packages - so you can ignore my remark in my
>> previous message.
>
> Which remark exactly??
About the possibility to install "kernel-modules" both when KERNEL_SPLIT_MODULES
is 0 or 1. I guess it doesn't hurt to add a note about that at the end of the
definition:
"""
.. note::
If :term:`KERNEL_SPLIT_MODULES` is set to 0, it is still possible to install
all kernel modules at once by adding ``kernel-modules`` to
:term:`IMAGE_INSTALL`. The way it works is that a placeholder
"kernel-modules" package will be created and will depend on every other
individual kernel module packages.
"""
So there's no confusion. What do you think?
Antonin
--
Antonin Godard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2025-06-18 13:43 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-08 10:44 [PATCH v2] ref-manual: document KERNEL_SPLIT_MODULES variable Dixit Parmar
2025-06-10 8:55 ` [docs] " Antonin Godard
2025-06-10 9:20 ` Antonin Godard
2025-06-18 12:14 ` Dixit Parmar
2025-06-18 12:59 ` [docs] " Antonin Godard
2025-06-18 13:19 ` Dixit Parmar
2025-06-18 13:42 ` Antonin Godard [this message]
2025-06-21 6:48 ` Dixit Parmar
2025-07-01 8:37 ` Antonin Godard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DAPPBO0JXU6C.38BKO55VAYW0G@bootlin.com \
--to=antonin.godard@bootlin.com \
--cc=dixitparmar19@gmail.com \
--cc=docs@lists.yoctoproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox