[Buildroot] [PATCH v2] pkg-kernel-module: Die when kernel module are disabled

Yann E. MORIN yann.morin.1998 at free.fr
Tue Aug 18 09:47:47 UTC 2015


Peter, All,

On 2015-08-18 07:41 +0200, Peter Korsgaard spake thusly:
> >>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni at free-electrons.com> writes:
> 
>  > Dear Noé Rubinstein,
>  > On Mon, 17 Aug 2015 09:10:44 +0200, Noé Rubinstein wrote:
>  >> Test the config of the kernel to see if loadable module support is
>  >> enabled, and error out otherwise. This makes the build failure less
>  >> confusing.
>  >> 
>  >> Signed-off-by: Noé Rubinstein <nrubinstein at aldebaran.com>
>  >> Acked-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
>  >> ---
>  >> package/pkg-kernel-module.mk | 4 ++++
>  >> 1 file changed, 4 insertions(+)
> 
>  > Applied after tweaking the commit log, thanks.
> 
> Couldn't we instead just force modules support on in the kernel if the
> pkg-kernel-module infrastructure is used?

And how do you suggest we test whether the kernel-module infra is used?

    1) it *is* used already: we do have packages that do call
        $(eval $(kernel-module)) , but I guess that's not what
        you meant. ;-)

    2) so you probably meant: if we actually need to build a kernel
       module.

But that's not so easy:

  - we need to know whether to enable moduls when parsing linux/linux.mk

  - external packages may call kernel-module

  - external packages are parsed after linux.mk

So, too late...

We can't enable kernel modules just from the .mk, we can only check for
them.

Unless we add an hidden kconfig knob BR2)LINUX_NEEDS_MODULES that
packages may select if they want to build modules. Of a visible knob
(user-selectable) that packages can depend on if they need to build
kernel modules...

Needless to say, for 2015.08, Noé's patch is good to go in, while the
alternatives are not; they can however be done post-2015.08.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'


More information about the buildroot mailing list