[Buildroot] [git commit] kmod: add option to install module utilities

[Buildroot] [git commit] kmod: add option to install module utilities

[Peter Korsgaard]

commit: http://git.buildroot.net/buildroot/commit/?id=78f5ac2908b30aa6f67583a355e058d6703755ac
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master

And ensure optional zlib+xz dependencies gets picked up.

Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
 package/kmod/Config.in |    6 ++++++
 package/kmod/kmod.mk   |   26 ++++++++++++++++++++++++++
 2 files changed, 32 insertions(+), 0 deletions(-)

diff --git a/package/kmod/Config.in b/package/kmod/Config.in
index c5e8ca8..6fdd92c 100644
--- a/package/kmod/Config.in
+++ b/package/kmod/Config.in
@@ -4,3 +4,9 @@ config BR2_PACKAGE_KMOD
 	  handle kernel modules
 
 	  http://git.kernel.org/?p=utils/kernel/kmod/kmod.git
+
+config BR2_PACKAGE_KMOD_TOOLS
+	bool "kmod utilities"
+	help
+	  Install kmod module utilities (depmod, insmod, lsmod,
+	  modinfo, modprobe, rmmod).
diff --git a/package/kmod/kmod.mk b/package/kmod/kmod.mk
index f71087c..55cbdbb 100644
--- a/package/kmod/kmod.mk
+++ b/package/kmod/kmod.mk
@@ -4,4 +4,30 @@ KMOD_SITE = $(BR2_KERNEL_MIRROR)/linux/utils/kernel/kmod/
 KMOD_INSTALL_STAGING = YES
 KMOD_DEPENDENCIES = host-pkg-config
 
+ifeq ($(BR2_PACKAGE_ZLIB),y)
+KMOD_DEPENDENCIES += zlib
+KMOD_CONF_OPT += --with-zlib
+endif
+
+ifeq ($(BR2_PACKAGE_XZ),y)
+KMOD_DEPENDENCIES += xz
+KMOD_CONF_OPT += --with-xz
+endif
+
+ifeq ($(BR2_PACKAGE_KMOD_TOOLS),y)
+# take precedence over busybox / module-init-tools implementations
+KMOD_DEPENDENCIES += \
+	$(if $(BR2_PACKAGE_BUSYBOX),busybox) \
+	$(if $(BR2_PACKAGE_MODULE_INIT_TOOLS),module-init-tools)
+
+define KMOD_INSTALL_TOOLS
+	for i in depmod insmod lsmod modinfo modprobe rmmod; \
+	do ln -sf ../usr/bin/kmod $(TARGET_DIR)/sbin/$$i; done
+endef
+
+KMOD_POST_INSTALL_TARGET_HOOKS += KMOD_INSTALL_TOOLS
+else
+KMOD_CONF_OPT += --disable-tools
+endif
+
 $(eval $(call AUTOTARGETS))

[Buildroot] [git commit] kmod: add option to install module utilities

[Maxime Ripard]

Hi Peter,

Maybe we should also deprecate modules-init-tools, since Jon Masters
announced some time ago that it was no longer maintained and people
should go to kmod utils ?

Le 22/03/2012 16:03, Peter Korsgaard a ?crit :
> commit: http://git.buildroot.net/buildroot/commit/?id=78f5ac2908b30aa6f67583a355e058d6703755ac
> branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
> 
> And ensure optional zlib+xz dependencies gets picked up.
> 
> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
> ---
>  package/kmod/Config.in |    6 ++++++
>  package/kmod/kmod.mk   |   26 ++++++++++++++++++++++++++
>  2 files changed, 32 insertions(+), 0 deletions(-)
> 
> diff --git a/package/kmod/Config.in b/package/kmod/Config.in
> index c5e8ca8..6fdd92c 100644
> --- a/package/kmod/Config.in
> +++ b/package/kmod/Config.in
> @@ -4,3 +4,9 @@ config BR2_PACKAGE_KMOD
>  	  handle kernel modules
>  
>  	  http://git.kernel.org/?p=utils/kernel/kmod/kmod.git
> +
> +config BR2_PACKAGE_KMOD_TOOLS
> +	bool "kmod utilities"
> +	help
> +	  Install kmod module utilities (depmod, insmod, lsmod,
> +	  modinfo, modprobe, rmmod).
> diff --git a/package/kmod/kmod.mk b/package/kmod/kmod.mk
> index f71087c..55cbdbb 100644
> --- a/package/kmod/kmod.mk
> +++ b/package/kmod/kmod.mk
> @@ -4,4 +4,30 @@ KMOD_SITE = $(BR2_KERNEL_MIRROR)/linux/utils/kernel/kmod/
>  KMOD_INSTALL_STAGING = YES
>  KMOD_DEPENDENCIES = host-pkg-config
>  
> +ifeq ($(BR2_PACKAGE_ZLIB),y)
> +KMOD_DEPENDENCIES += zlib
> +KMOD_CONF_OPT += --with-zlib
> +endif
> +
> +ifeq ($(BR2_PACKAGE_XZ),y)
> +KMOD_DEPENDENCIES += xz
> +KMOD_CONF_OPT += --with-xz
> +endif
> +
> +ifeq ($(BR2_PACKAGE_KMOD_TOOLS),y)
> +# take precedence over busybox / module-init-tools implementations
> +KMOD_DEPENDENCIES += \
> +	$(if $(BR2_PACKAGE_BUSYBOX),busybox) \
> +	$(if $(BR2_PACKAGE_MODULE_INIT_TOOLS),module-init-tools)
> +
> +define KMOD_INSTALL_TOOLS
> +	for i in depmod insmod lsmod modinfo modprobe rmmod; \
> +	do ln -sf ../usr/bin/kmod $(TARGET_DIR)/sbin/$$i; done
> +endef
> +
> +KMOD_POST_INSTALL_TARGET_HOOKS += KMOD_INSTALL_TOOLS
> +else
> +KMOD_CONF_OPT += --disable-tools
> +endif
> +
>  $(eval $(call AUTOTARGETS))
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot


-- 
Maxime Ripard, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [git commit] kmod: add option to install module utilities

[Peter Korsgaard]

>>>>> "Maxime" == Maxime Ripard <maxime.ripard@free-electrons.com> writes:

 Maxime> Hi Peter,
 Maxime> Maybe we should also deprecate modules-init-tools, since Jon Masters
 Maxime> announced some time ago that it was no longer maintained and people
 Maxime> should go to kmod utils ?

Sooner or later we should, but I'm not sure if kmod is mature enough to
do it yet. I actually very rarely use kernel modules, perhaps someone
else knows more about it?

-- 
Bye, Peter Korsgaard

[Buildroot] [git commit] kmod: add option to install module utilities

[Thomas Petazzoni]

Le Thu, 22 Mar 2012 16:39:53 +0100,
Peter Korsgaard <jacmet@sunsite.dk> a ?crit :

> Sooner or later we should, but I'm not sure if kmod is mature enough to
> do it yet. I actually very rarely use kernel modules, perhaps someone
> else knows more about it?

According to the kmod talk at ELC, kmod is now a complete replacement
for module-init-tools, and module-init-tools are deprecated, as Maxime
said. So all distributions are likely to switch to kmod rather soon.

Regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [git commit] kmod: add option to install module utilities

[Arnout Vandecappelle]

On Thursday 22 March 2012 16:03:12 Peter Korsgaard wrote:
> +config BR2_PACKAGE_KMOD_TOOLS
> +       bool "kmod utilities"
> +       help
> +         Install kmod module utilities (depmod, insmod, lsmod,
> +         modinfo, modprobe, rmmod).

 Shouldn't this depend on BR2_PACKAGE_BUSYBOX_SHOW_OTHERS?

 Regards,
 Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20120322/f1d7221f/attachment.html>

[Buildroot] [git commit] kmod: add option to install module utilities

[Peter Korsgaard]

>>>>> "Arnout" == Arnout Vandecappelle <arnout@mind.be> writes:

 Arnout> On Thursday 22 March 2012 16:03:12 Peter Korsgaard wrote:
 >> +config BR2_PACKAGE_KMOD_TOOLS

 >> +???????bool "kmod utilities"

 >> +???????help

 >> +?????????Install kmod module utilities (depmod, insmod, lsmod,

 >> +?????????modinfo, modprobe, rmmod).

 Arnout> Shouldn't this depend on BR2_PACKAGE_BUSYBOX_SHOW_OTHERS?

Arguably it should (even though I would like us to consider getting rid
of that option).

-- 
Bye, Peter Korsgaard

[Buildroot] [git commit] kmod: add option to install module utilities

[Arnout Vandecappelle]

On Thursday 22 March 2012 22:48:36 Peter Korsgaard wrote:
>  Arnout> On Thursday 22 March 2012 16:03:12 Peter Korsgaard wrote:
>  >> +config BR2_PACKAGE_KMOD_TOOLS
>  >> +       bool "kmod utilities"
>  >> +       help
>  >> +         Install kmod module utilities (depmod, insmod, lsmod,
>  >> +         modinfo, modprobe, rmmod).
>  Arnout> Shouldn't this depend on BR2_PACKAGE_BUSYBOX_SHOW_OTHERS?
> 
> Arguably it should (even though I would like us to consider getting rid
> of that option).

 +1!

 But then I think the prompt should mention it is also in busybox.

 Regards,
 Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F