* [Buildroot] [PATCH 1/1 v4] Add kernel compression selection.
@ 2015-04-02 23:31 Sagaert Johan
2015-12-13 15:00 ` Yann E. MORIN
0 siblings, 1 reply; 2+ messages in thread
From: Sagaert Johan @ 2015-04-02 23:31 UTC (permalink / raw)
To: buildroot
This selection will ensure that the correct host tools
will be build used for the kernel compression method used.
Signed-off-by: Sagaert Johan <sagaert.johan@proximus.be>
---
V3->V4
Add selection menu in the kernel menu section.
This ensures only things needed are build.
V2->V3
Unconditionally build lz4.
The building overhead is negligible.
V1->V2
Only add the host-lz4 dependency when ARM is selected.
Don't touch host-lz4 package.
---
linux/Config.in | 28 ++++++++++++++++++++++++++++
linux/linux.mk | 13 ++++++++++++-
2 files changed, 40 insertions(+), 1 deletion(-)
diff --git a/linux/Config.in b/linux/Config.in
index a1113dd..e5372cb 100644
--- a/linux/Config.in
+++ b/linux/Config.in
@@ -247,6 +247,34 @@ config BR2_LINUX_KERNEL_IMAGE_TARGET_CUSTOM
endchoice
+#
+# Kernel compression format
+#
+
+choice
+ prompt "Kernel compression format"
+ help
+ This selection will just ensure that the correct host tools are build.
+ The actual compression for the kernel should be selected in the
+ kernel configuration menu.
+
+config BR2_LINUX_KERNEL_GZIP
+ bool "gzip compression"
+
+config BR2_LINUX_KERNEL_LZ4
+ bool "lz4 compression"
+
+config BR2_LINUX_KERNEL_LZMA
+ bool "lzma compression"
+
+config BR2_LINUX_KERNEL_LZO
+ bool "lzo compression"
+
+config BR2_LINUX_KERNEL_XZ
+ bool "xz compression"
+
+endchoice
+
config BR2_LINUX_KERNEL_IMAGE_TARGET_NAME
string "Kernel image target name"
depends on BR2_LINUX_KERNEL_IMAGE_TARGET_CUSTOM
diff --git a/linux/linux.mk b/linux/linux.mk
index 22fce35..67ea544 100644
--- a/linux/linux.mk
+++ b/linux/linux.mk
@@ -49,7 +49,18 @@ LINUX_PATCHES = $(call qstrip,$(BR2_LINUX_KERNEL_PATCH))
LINUX_PATCH = $(filter ftp://% http://% https://%,$(LINUX_PATCHES))
LINUX_INSTALL_IMAGES = YES
-LINUX_DEPENDENCIES += host-kmod host-lzop
+LINUX_DEPENDENCIES += host-kmod
+
+# host tools needed for kernel compression
+ifeq ($(BR2_LINUX_KERNEL_LZ4),y)
+LINUX_DEPENDENCIES += host-lz4
+else ifeq ($(BR2_LINUX_KERNEL_LZMA),y)
+LINUX_DEPENDENCIES += host-lzma
+else ifeq ($(BR2_LINUX_KERNEL_LZO),y)
+LINUX_DEPENDENCIES += host-lzop
+else ifeq ($(BR2_LINUX_KERNEL_XZ),y)
+LINUX_DEPENDENCIES += host-xz
+endif
ifeq ($(BR2_LINUX_KERNEL_UBOOT_IMAGE),y)
LINUX_DEPENDENCIES += host-uboot-tools
--
2.3.3
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Buildroot] [PATCH 1/1 v4] Add kernel compression selection.
2015-04-02 23:31 [Buildroot] [PATCH 1/1 v4] Add kernel compression selection Sagaert Johan
@ 2015-12-13 15:00 ` Yann E. MORIN
0 siblings, 0 replies; 2+ messages in thread
From: Yann E. MORIN @ 2015-12-13 15:00 UTC (permalink / raw)
To: buildroot
Johan, All,
Sorry for the slow response... :-/
On 2015-04-03 01:31 +0200, Sagaert Johan spake thusly:
> This selection will ensure that the correct host tools
> will be build used for the kernel compression method used.
Yes, that is much better. Thanks! :-)
However, just a comment below...
> Signed-off-by: Sagaert Johan <sagaert.johan@proximus.be>
> ---
> V3->V4
> Add selection menu in the kernel menu section.
> This ensures only things needed are build.
> V2->V3
> Unconditionally build lz4.
> The building overhead is negligible.
> V1->V2
> Only add the host-lz4 dependency when ARM is selected.
> Don't touch host-lz4 package.
> ---
> linux/Config.in | 28 ++++++++++++++++++++++++++++
> linux/linux.mk | 13 ++++++++++++-
> 2 files changed, 40 insertions(+), 1 deletion(-)
>
> diff --git a/linux/Config.in b/linux/Config.in
> index a1113dd..e5372cb 100644
> --- a/linux/Config.in
> +++ b/linux/Config.in
> @@ -247,6 +247,34 @@ config BR2_LINUX_KERNEL_IMAGE_TARGET_CUSTOM
>
> endchoice
>
> +#
> +# Kernel compression format
> +#
> +
> +choice
> + prompt "Kernel compression format"
> + help
> + This selection will just ensure that the correct host tools are build.
> + The actual compression for the kernel should be selected in the
> + kernel configuration menu.
> +
> +config BR2_LINUX_KERNEL_GZIP
> + bool "gzip compression"
> +
> +config BR2_LINUX_KERNEL_LZ4
> + bool "lz4 compression"
> +
> +config BR2_LINUX_KERNEL_LZMA
> + bool "lzma compression"
> +
> +config BR2_LINUX_KERNEL_LZO
> + bool "lzo compression"
> +
> +config BR2_LINUX_KERNEL_XZ
> + bool "xz compression"
> +
> +endchoice
> +
> config BR2_LINUX_KERNEL_IMAGE_TARGET_NAME
> string "Kernel image target name"
> depends on BR2_LINUX_KERNEL_IMAGE_TARGET_CUSTOM
> diff --git a/linux/linux.mk b/linux/linux.mk
> index 22fce35..67ea544 100644
> --- a/linux/linux.mk
> +++ b/linux/linux.mk
> @@ -49,7 +49,18 @@ LINUX_PATCHES = $(call qstrip,$(BR2_LINUX_KERNEL_PATCH))
> LINUX_PATCH = $(filter ftp://% http://% https://%,$(LINUX_PATCHES))
>
> LINUX_INSTALL_IMAGES = YES
> -LINUX_DEPENDENCIES += host-kmod host-lzop
> +LINUX_DEPENDENCIES += host-kmod
> +
> +# host tools needed for kernel compression
> +ifeq ($(BR2_LINUX_KERNEL_LZ4),y)
> +LINUX_DEPENDENCIES += host-lz4
> +else ifeq ($(BR2_LINUX_KERNEL_LZMA),y)
> +LINUX_DEPENDENCIES += host-lzma
> +else ifeq ($(BR2_LINUX_KERNEL_LZO),y)
> +LINUX_DEPENDENCIES += host-lzop
> +else ifeq ($(BR2_LINUX_KERNEL_XZ),y)
> +LINUX_DEPENDENCIES += host-xz
> +endif
Shouldn't we also call something like:
$(call KCONFIG_DISABLE_OPT,CONFIG_KERNEL_GZIP,$(@D)/.config))
$(call KCONFIG_DISABLE_OPT,CONFIG_KERNEL_LZ4,$(@D)/.config))
... nad so on ...
ifeq ($(BR2_LINUX_KERNEL_GZIP),y)
LINUX_COMP_CONFIG = $(call KCONFIG_ENABLE_OPT,CONFIG_KERNEL_GZIP,$(@D)/.config))
else ifeq ($(BR2_LINUX_KERNEL_LZ4),y)
LINUX_DEPENDENCIES += host-lz4
LINUX_COMP_CONFIG = $(call KCONFIG_ENABLE_OPT,CONFIG_KERNEL_LZ4,$(@D)/.config))
else ifeq ($(BR2_LINUX_KERNEL_LZO),y)
LINUX_DEPENDENCIES += host-lzop
LINUX_COMP_CONFIG = $(call KCONFIG_ENABLE_OPT,CONFIG_KERNEL_LZO,$(@D)/.config))
... and so on ...
So that what the user jas selected in the Buildroot menuconfig gets
actually applied to the Linux configuration, to avoid having to manually
handle the coherence between the two?
But anmyway, your patch is an improvement against the current situation,
so:
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> ifeq ($(BR2_LINUX_KERNEL_UBOOT_IMAGE),y)
> LINUX_DEPENDENCIES += host-uboot-tools
> --
> 2.3.3
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-12-13 15:00 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-02 23:31 [Buildroot] [PATCH 1/1 v4] Add kernel compression selection Sagaert Johan
2015-12-13 15:00 ` Yann E. MORIN
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.