From: Matthias Brugger <mbrugger@suse.com>
To: u-boot@lists.denx.de
Subject: [PATCH v4 1/2] Kconfig: add btrfs to distro boot
Date: Tue, 28 Jan 2020 11:26:25 +0100 [thread overview]
Message-ID: <75ba16cb-bab7-fd2d-e33e-5ec654814ebe@suse.com> (raw)
In-Reply-To: <20200127215806.GA13444@bill-the-cat>
On 27/01/2020 22:58, Tom Rini wrote:
> On Fri, Jan 17, 2020 at 08:59:02PM +0100, matthias.bgg at kernel.org wrote:
>
>> From: Matthias Brugger <mbrugger@suse.com>
>>
>> Some distributions use btrfs as the default file system.
>> Enable btrfs support by default when using distro boot for all
>> architectures but riscv, as it breaks compilation due to size problems.
>>
>> Signed-off-by: Matthias Brugger <mbrugger@suse.com>
>
> This adds around 60kb to many platforms, so I'm not going to take this
> right now. But I'm not rejecting it outright. My question however is,
> I was under the impression that the direction distributions were taking
> was that bootefi would be used to start GRUB2 and that (and its
> filesystem modules) would be responsible for doing the next steps. So
> we wouldn't need btrfs support here for example as everything would be
> picked out of the system partition, which is FAT32. Am I mistaken
> about the flow here? Thanks!
>
I think the assumption is correct for most of the distributions (if not all).
However in distro boot we support e.g. extlinux/sysboot which not necessarily
boots a EFI binary. Apart from that, think about a broken system partition. In
that case if your distribution has the kernel on a btrfs partition, you would
not be able to boot this kernel.
That's the reason why we enable ext2 and ext4 support in distro boot.
Regarding the size, I'm aware of the problem, that's why it is set as imply in
Kconfig so that individual boards can turn it off. We might think about chaning
ext2 and ext4 from select to imply as well as we are at the limit on some platforms.
Regards,
Matthias
Regards,
Matthias
next prev parent reply other threads:[~2020-01-28 10:26 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-17 19:59 [PATCH v4 1/2] Kconfig: add btrfs to distro boot matthias.bgg at kernel.org
2020-01-17 19:59 ` [PATCH v4 2/2] configs: Re-sync with CONFIG_DISTRO_DEFAULTS matthias.bgg at kernel.org
2020-01-20 9:34 ` [PATCH v4 1/2] Kconfig: add btrfs to distro boot Matthias Brugger
2020-01-27 21:58 ` Tom Rini
2020-01-27 23:26 ` Marek Behun
2020-01-27 23:28 ` Tom Rini
2020-01-28 13:06 ` Marek Behun
2020-01-28 13:31 ` Tom Rini
2020-01-28 10:26 ` Matthias Brugger [this message]
2020-01-28 13:30 ` Tom Rini
2020-02-03 14:17 ` Petr Vorel
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=75ba16cb-bab7-fd2d-e33e-5ec654814ebe@suse.com \
--to=mbrugger@suse.com \
--cc=u-boot@lists.denx.de \
/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