Yocto Project Documentation
 help / color / mirror / Atom feed
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



  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