mbox series

[v1,0/3] riscv: make image compression configurable

Message ID 20240502111613.1380453-1-emil.renner.berthing@canonical.com (mailing list archive)
Headers show
Series riscv: make image compression configurable | expand

Message

Emil Renner Berthing May 2, 2024, 11:16 a.m. UTC 
Masahiro's patch[1] made me wonder why we're not just using KBUILD_IMAGE
to determine which (possibly compressed) kernel image to use in 'make
tar-pkg' like other architectures do. It turns out we're always setting
KBUILD_IMAGE to the uncompressed Image file and then compressing it into
the Image.gz file afterwards.

This series fixes that so the compression method is configurable and
KBUILD_IMAGE is set to the chosen (possibly uncompressed) kernel image
which is then used by targets like 'make install' and 'make bindeb-pkg' and
'make tar-pkg'.

Patch 3/3 depends on the previously mentioned patch below.

[1]: https://lore.kernel.org/r/20240414174139.3001175-1-masahiroy@kernel.org

Emil Renner Berthing (3):
  riscv: make image compression configurable
  riscv: show help string for riscv-specific targets
  kbuild: buildtar: install riscv compressed images as vmlinuz

 arch/riscv/Kconfig         |  7 +++++
 arch/riscv/Makefile        | 60 +++++++++++++++++++++++++-------------
 arch/riscv/boot/install.sh |  9 ++++--
 scripts/package/buildtar   | 18 +++++-------
 4 files changed, 61 insertions(+), 33 deletions(-)

Comments

Björn Töpel May 2, 2024, 1:03 p.m. UTC | #1 
Emil Renner Berthing <emil.renner.berthing@canonical.com> writes:

> Masahiro's patch[1] made me wonder why we're not just using KBUILD_IMAGE
> to determine which (possibly compressed) kernel image to use in 'make
> tar-pkg' like other architectures do. It turns out we're always setting
> KBUILD_IMAGE to the uncompressed Image file and then compressing it into
> the Image.gz file afterwards.
>
> This series fixes that so the compression method is configurable and
> KBUILD_IMAGE is set to the chosen (possibly uncompressed) kernel image
> which is then used by targets like 'make install' and 'make bindeb-pkg' and
> 'make tar-pkg'.
>
> Patch 3/3 depends on the previously mentioned patch below.

Nice!

Tested-by: Björn Töpel <bjorn@rivosinc.com>