From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6C1DDC07E95 for ; Fri, 2 Jul 2021 20:19:14 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id D2BE4600CC for ; Fri, 2 Jul 2021 20:19:13 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D2BE4600CC Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=konsulko.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id B17CD82979; Fri, 2 Jul 2021 22:19:11 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=konsulko.com header.i=@konsulko.com header.b="BDI6IoKc"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id C7BDF82979; Fri, 2 Jul 2021 22:19:10 +0200 (CEST) Received: from mail-qk1-x72a.google.com (mail-qk1-x72a.google.com [IPv6:2607:f8b0:4864:20::72a]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id D9B3D829BE for ; Fri, 2 Jul 2021 22:19:07 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=trini@konsulko.com Received: by mail-qk1-x72a.google.com with SMTP id c3so9594818qkc.3 for ; Fri, 02 Jul 2021 13:19:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=yUlZ3+IhpqaIseKR1LUcUaN789omxR5BubNcpE36wsw=; b=BDI6IoKcGqP7r5Opz0UuaQCx8kHug+ofrFLfD9hoOiJuOBc/vRhb82VQ3dz96wDbly wI56JzN7qZlyBFHjgbyL7f+8CyZfF8RFYAxDt3BnXH9J9OX1qtgBkwLuVm/yD6aSn5Qx iKT5NYnt30aYpAAIqdqMzqGhRx0524f5haQPo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=yUlZ3+IhpqaIseKR1LUcUaN789omxR5BubNcpE36wsw=; b=g9I1MpjIiWAbXfsR+qHWgrKTeepsiTAoVXFxy8b/SdHxzWXQei26zEPvcqFsl1LoPQ wgOLMd6AuUv035gDEgmeaqAySt182F1DCIDpEy1t3+Nk6r4QH9LABO6bISpI3FU2KYhw Sywg/ZB/8EAkLLvALp8BhMtEKR76KeFfxnnG21X1AP7qjaA8oOBirGouOLwND8FGQcyM VdT5pm1uP6s6PVwdlz4BwO9B/WvEie6Zk7xHHK+sxVQ9j9KB0gH12Q2Td1ZeBw1JRDUL TviFCtfzeKnUs3MuR7vE6h4JLN1jXmMYLeOWyVhq0mntqKv5jOsvY6XJpGjv/b6fMsV3 2/iQ== X-Gm-Message-State: AOAM532z2zjhj5JSw7INc+2YnLXp4m5X/K0MHrymf7Y13GGvDFq3h9I6 OEaLnJfdqxcUlgUQ0nPuLeZqqg== X-Google-Smtp-Source: ABdhPJzZHaEH1l3y11vzj8MTkI33iam9HxbigFWeW6lkbGhbPFLZWVv7BgVfOxEHHhO1ZFsaYLAEXQ== X-Received: by 2002:ae9:eb42:: with SMTP id b63mr1696646qkg.222.1625257146614; Fri, 02 Jul 2021 13:19:06 -0700 (PDT) Received: from bill-the-cat (2603-6081-7b01-cbda-447a-194c-2139-e81a.res6.spectrum.com. [2603:6081:7b01:cbda:447a:194c:2139:e81a]) by smtp.gmail.com with ESMTPSA id e15sm1613914qkl.28.2021.07.02.13.19.05 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 02 Jul 2021 13:19:05 -0700 (PDT) Date: Fri, 2 Jul 2021 16:19:03 -0400 From: Tom Rini To: Simon Glass Cc: Mark Kettenis , U-Boot Mailing List , Ilias Apalodimas , Pali =?iso-8859-1?Q?Roh=E1r?= , Heinrich Schuchardt , Alex Graf , Masahiro Yamada Subject: Re: [PATCH v2 0/9] efi: Various tidy-ups and drop the default Message-ID: <20210702201903.GF9516@bill-the-cat> References: <20210702183620.1542680-1-sjg@chromium.org> <5613820c4bba3707@bloch.sibelius.xs4all.nl> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="LzN+R+/cjkHsoWp9" Content-Disposition: inline In-Reply-To: X-Clacks-Overhead: GNU Terry Pratchett User-Agent: Mutt/1.9.4 (2018-02-28) X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean --LzN+R+/cjkHsoWp9 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Jul 02, 2021 at 02:04:40PM -0600, Simon Glass wrote: > Hi Mark, >=20 > On Fri, 2 Jul 2021 at 13:50, Mark Kettenis wrot= e: > > > > > From: Simon Glass > > > Date: Fri, 2 Jul 2021 12:36:11 -0600 > > > > > > It has come to light that EFI_LOADER adds an extraordinary amount of > > > code to U-Boot. For example, with nokia_rx51 the size delta is about > > > 90KB. About 170 boards explicitly disable the option, but is is clear > > > that many more could, thus saving image size and boot time. > > > > > > The current situation is affecting U-Boot's image as a svelt bootload= er. > > > > > > EFI_LOADER is required by EBBR, a new boot standard which aims to > > > bring in UEFI protocols to U-Boot. But EBRR is not required for > > > booting. U-Boot already provides support for FIT, the 'bootm' command > > > and a suitable hand-off to Linux. EBRR has made the decision to create > > > a parallel infrastructure, e.g. does not use FIT, nor U-Boot's signing > > > infrastructure. > > > > > > EBBR should be truly optional, enabled only by boards that use it. Mo= st > > > don't use it but it is enabled anyway. The default boot path should be > > > one that makes use of the existing U-Boot support. > > > > > > To try to retify this situation, this series adds a new Kconfig option > > > for EBBR so that the naming is more explicit. Then EFI_LOADER is upda= ted > > > to depend on it. > > > > > > The final patch makes EBBR optional. For now, only sandbox enables EB= BR. > > > Other boards can be added as needed, presumably by distributions that > > > require it. Another approach would be to add 'CONFIG_EBBR=3Dy' to the > > > .config before building, in the build system. That might be more frie= ndly > > > to U-Boot users. > > > > > > This series also fixes a minor issue noticed during testing. > > > > I don't understand why you're pushing this series in a form that > > still disables EFI_LOADER by default after last weeks discussions. >=20 > I moved the change to non-default to the last patch. Even if that is > not a good idea, the rest of the series stands. >=20 > But more specifically to your question, I have not seen any discussion > about the size issues identified. Nor has there been any comment on my > suggestion in the cover letter for distros to define CONFIG_EBBR > themselves when building U-Boot. I still think turning it off by > default makes sense given the current situation. It's not "distros", it's board vendors. And then SoM vendors. And their customers. Based on what I see, the default values get re-used 95% of the time, or more. The things we want to Just Work need to be enabled by default. What distros want is for vendors to include firmware in non-volatile storage, not to rebuild every board firmware and have to ship that, and even less to have to tell users to reconfigure things and then build. It's also at this point counter-productive. If you want to run an off the shelf distro on aarch64, it's almost certainly going to use EFI_LOADER. What I wasn't sure about was armv7, but that was confirmed to be used by default in Fedora and OpenBSD and encouraged (I'll assume I just didn't find the right install media) for Debian. I'd rather talk with someone in Armbian about why they don't use EFI_LOADER than try and convince Fedora to go and use extlinux.conf more (and that leaves out entirely *BSD which is not what I want to do either!). --=20 Tom --LzN+R+/cjkHsoWp9 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmDfdLcACgkQFHw5/5Y0 tyxIMAv6A2moEkpHWiESeazHC76fyeiwkE1HEwcQGaR+rPLOWSjyYNSZHVrlmRzg n7N8iYpLtWimEFhHkQJY8z7dYmT5Zh0/69VJyAtCQTR7ETQspUf2tcFaRzhHvWyC 2nw8WBcUxf9TvB9KbAe3kxjuRUskhnMXl3n3QO3PY/YO3WxCpjcjhB6ewmPXVsK4 wFYRYn1rUlcWlcTyJ3ybuOUFDpnwb04dGdVClz2nte+8uR/ZebdiXcnSWFuy3CH6 Oi0whzZ5j5DELJc0TDKqdwRSMGA0jrHVSxqpzeh6E2EG3TmJ50W2juJBTkWUsbok MZeXdhiO3uFP7RlBLhd2T7uTYIYdtVF1IWP3uqWp99calb64a9U7mM+lCinbGX40 dmETJnUeqX9Ccnxgqu4uMJlk2ZxHMz0qsibv+btQpvUiGtJS9oEMxi57uN8ry2oE +2xc4PcLBYDhyg5exVWPCg0d1GzFBVRt6DD/HiuagSYQPY2Y4Ew1kssN9uYPV16o y/d3tzdN =ETjK -----END PGP SIGNATURE----- --LzN+R+/cjkHsoWp9--