Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/4] fs/common.mk: support lz4 compression
Date: Mon, 8 Jan 2018 23:46:31 +0100	[thread overview]
Message-ID: <20180108234631.6e3db6d0@windsurf> (raw)
In-Reply-To: <20171013191655.30427-1-peter@korsgaard.com>

Hello,

On Fri, 13 Oct 2017 21:16:52 +0200, Peter Korsgaard wrote:
> Similar to the other compressors.  Notice that we use the -l (legacy format)
> for Linux kernel initrd compatibility.
> 
> Lz4 decompression is supported by the Linux kernel since 3.11.
> 
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> ---
>  fs/common.mk | 5 +++++
>  1 file changed, 5 insertions(+)

I've applied the series. I'm also not super convinced we need to
support all those compression options, but we already have plenty of
them, and Peter is just adding one more, which doesn't make the whole
thing much more complicated.

We can continue the discussion on whether we want to keep this or not,
especially in light of http://patchwork.ozlabs.org/patch/850055/ that
proposes to allow compressing the sdcard.img generated by genimage.

If we allow compression rootfs.tar, why not sdcard.img ?

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

      parent reply	other threads:[~2018-01-08 22:46 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-13 19:16 [Buildroot] [PATCH 1/4] fs/common.mk: support lz4 compression Peter Korsgaard
2017-10-13 19:16 ` [Buildroot] [PATCH 2/4] fs/cpio: add option for " Peter Korsgaard
2017-10-13 19:16 ` [Buildroot] [PATCH 3/4] fs/ext2: " Peter Korsgaard
2017-10-13 19:16 ` [Buildroot] [PATCH 4/4] fs/tar: " Peter Korsgaard
2017-10-13 22:05 ` [Buildroot] [PATCH 1/4] fs/common.mk: support " Thomas Petazzoni
2017-10-14  7:32   ` Yann E. MORIN
2017-10-15  9:03     ` Peter Korsgaard
2017-10-15 13:05       ` Yann E. MORIN
2017-10-15 20:45         ` Peter Korsgaard
2018-01-08 22:46 ` Thomas Petazzoni [this message]

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=20180108234631.6e3db6d0@windsurf \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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