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=-5.8 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 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 987C3C4338F for ; Wed, 25 Aug 2021 14:43:11 +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 B0FFB60E97 for ; Wed, 25 Aug 2021 14:43:10 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org B0FFB60E97 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 7F62B82986; Wed, 25 Aug 2021 16:43:07 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="VRJRFep1"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 58D3782986; Wed, 25 Aug 2021 16:43:06 +0200 (CEST) Received: from mail-pj1-x1030.google.com (mail-pj1-x1030.google.com [IPv6:2607:f8b0:4864:20::1030]) (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 0762080612 for ; Wed, 25 Aug 2021 16:43:01 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=takahiro.akashi@linaro.org Received: by mail-pj1-x1030.google.com with SMTP id w19-20020a17090aaf9300b00191e6d10a19so4360264pjq.1 for ; Wed, 25 Aug 2021 07:43:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:mail-followup-to:references :mime-version:content-disposition:in-reply-to; bh=Ov270SnYGqnxGR+cBjXj8szQt6oDKZFsu4Wijpkj01s=; b=VRJRFep1f/8HZawLQV1OZWrky7Mxxv8hkOPhJj0AD4BhK3+EesnRBAoFr2GOjd4Taz 40p9X7rhBPU+BxpwgGVo8Q8yaw6P1zb/DKK3TNm6QEcUp0qG5uGeTjNHJDE9B8i3qqsY qrggyavcb4grrCaLVYhwLfRrWUuL1mxYgtpCjVfZXX76vKT06pIqdJMNub9NIsgmbTT1 FFfnFYwYHye33O9MUp1QHV3gTBmTl9OFG4shGtYAj60Z6gDM3Cf0AHHQaykBIh1ZIhkC TkbN2n65AE7RNiedmkhSwNa0H3VyHf0OlKsIEyl0+9aFRlg62ZlNxZjfV7CnFHyiR82/ SMpg== 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 :mail-followup-to:references:mime-version:content-disposition :in-reply-to; bh=Ov270SnYGqnxGR+cBjXj8szQt6oDKZFsu4Wijpkj01s=; b=o6uHtUgG6S/5f5OdkObyxKWBsz7FeGggDz15oCYkNWXnFjQZNIwAra17ZJBvUWYPjR GE5pywbrL6Kx+2ksoZjNI9QjcEW/xNsQqh2AXbmiXuxMfz6LBb1Vpx+ti8iAgY6HxqKW GiT/MioXf+5xY6zepVR2Yb/DodYayuSX5Ikop4a3ecquNEqMjwuAugDKBL2PjA5k5VkU wazCpYb8+OwedkqANmQ8Kt3nOjkRk9VJeQpau1UIlEfqdlhSRcnYj0rr/PpxmnRAzNSf QNOe7VGWaZ6IzJFdCivRjuzSWCLcx11f96gYyv9Icoto4kGCCd+9GpoykB+/qjLsQlSr vb6w== X-Gm-Message-State: AOAM530U0RBQ+wPR7Ic1rorIdvGdPaPNZrtqgNkLUbMkHF2d9nC0Twqt 9Nw/8WPUYIfyvdyP+3Dh0AnKCw== X-Google-Smtp-Source: ABdhPJzwUtAPrrUbmRQU8YVmiZ6heA+ASuWH0sti+meOiOfq8BFKKyBBOIgNXK+6mf4cCRus3o5j0g== X-Received: by 2002:a17:90a:9cf:: with SMTP id 73mr11147918pjo.136.1629902579067; Wed, 25 Aug 2021 07:42:59 -0700 (PDT) Received: from laputa (pdb6272e8.tkyea130.ap.so-net.ne.jp. [219.98.114.232]) by smtp.gmail.com with ESMTPSA id y14sm100378pfi.82.2021.08.25.07.42.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 25 Aug 2021 07:42:58 -0700 (PDT) Date: Wed, 25 Aug 2021 23:42:51 +0900 From: AKASHI Takahiro To: Simon Glass Cc: Emmanuel Vadot , Mark Kettenis , Tom Rini , U-Boot Mailing List , Ilias Apalodimas , Steffen Jaeckel , Michal Simek , Dennis Gilmore , Daniel Schwierzeck , Lukas Auer , Jaehoon Chung , Matthias Brugger , Peng Fan , Stephen Warren , Stephen Warren Subject: Re: [PATCH 00/28] Initial implementation of bootmethod/bootflow Message-ID: <20210825144251.GB89209@laputa> Mail-Followup-To: AKASHI Takahiro , Simon Glass , Emmanuel Vadot , Mark Kettenis , Tom Rini , U-Boot Mailing List , Ilias Apalodimas , Steffen Jaeckel , Michal Simek , Dennis Gilmore , Daniel Schwierzeck , Lukas Auer , Jaehoon Chung , Matthias Brugger , Peng Fan , Stephen Warren , Stephen Warren References: <20210819034601.1618773-1-sjg@chromium.org> <56140f0c4976b9f9@bloch.sibelius.xs4all.nl> <20210823200146.GG858@bill-the-cat> <561412964a77c660@bloch.sibelius.xs4all.nl> <20210825124523.abf62a19a25e7491a4b4dae1@bidouilliste.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 Simon, On Wed, Aug 25, 2021 at 07:11:44AM -0600, Simon Glass wrote: > Hi, > > On Wed, 25 Aug 2021 at 04:45, Emmanuel Vadot wrote: > > > > On Tue, 24 Aug 2021 12:22:42 +0200 (CEST) > > Mark Kettenis wrote: > > > > > > Date: Mon, 23 Aug 2021 16:01:46 -0400 > > > > From: Tom Rini > > > > > > > > On Mon, Aug 23, 2021 at 11:25:42AM -0600, Simon Glass wrote: > > > > > Hi Mark, > > > > > > > > > > On Mon, 23 Aug 2021 at 05:54, Mark Kettenis wrote: > > > > > > > > > > > > > From: Simon Glass > > > > > > > Date: Wed, 18 Aug 2021 21:45:33 -0600 > > > > > > > > > > > > > > Bootmethod and bootflow provide a built-in way for U-Boot to automatically boot > > > > > > > an Operating System without custom scripting and other customisation: > > > > > > > > > > > > > > - bootmethod - a method to scan a device to find bootflows (owned by U-Boot) > > > > > > > - bootflow - a description of how to boot (owned by the distro) > > > > > > > > > > > > > > This series provides an initial implementation of these, enable to scan > > > > > > > for bootflows from MMC and Ethernet. The only bootflow supported is > > > > > > > distro boot, i.e. an extlinux.conf file included on a filesystem or > > > > > > > tftp server. It works similiarly to the existing script-based approach, > > > > > > > but is native to U-Boot. > > > > > > > > > > > > > > With this we can boot on a Raspberry Pi 3 with just one command: > > > > > > > > > > > > > > bootflow scan -lb > > > > > > > > > > > > > > which means to scan, listing (-l) each bootflow and trying to boot each > > > > > > > one (-b). The final patch shows this. > > > > > > > > > > > > > > It is intended that this approach be expanded to support mechanisms other > > > > > > > than distro boot, including EFI-related ones. With a standard way to > > > > > > > identify boot devices, these features become easier. It also should > > > > > > > support U-Boot scripts, for backwards compatibility only. > > > > > > > > > > > > > > The first patch of this series moves boot-related code out of common/ and > > > > > > > into a new boot/ directory. This helps to collect these related files > > > > > > > in one place, as common/ is quite large. > > > > > > > > > > > > > > Like sysboot, this feature makes use of the existing PXE implementation. > > > > > > > Much of this series consists of cleaning up that code and refactoring it > > > > > > > into something closer to a module that can be called, teasing apart its > > > > > > > reliance on the command-line interpreter to access filesystems and the > > > > > > > like. Also it now uses function arguments and its own context struct > > > > > > > internally rather than environment variables, which is very hard to > > > > > > > follow. No core functional change is included in the included PXE patches. > > > > > > > > > > > > > > For documentation, see the 'doc' patch. > > > > > > > > > > > > > > There is quite a long list of future work included in the documentation. > > > > > > > One question is the choice of naming. Since this is a bootloader, should > > > > > > > we just call this a 'method' and a 'flow' ? The 'boot' prefix is already > > > > > > > shared by other commands like bootm, booti, etc. > > > > > > > > > > > > > > The design is described here: > > > > > > > > > > > > > > https://drive.google.com/file/d/1ggW0KJpUOR__vBkj3l61L2dav4ZkNC12/view?usp=sharing > > > > > > > > > > > > > > The series is available at u-boot-dm/bmea-working > > > > > > > > > > > > How does the user control the order in which devices are scanned/booted? > > > > > > > > > > That is not supported in distroboot at present, at least so far as I > > > > > can see. For Fedora it seems to happen in grub. Do I have that right? > > > > > > > > Well, there's "find the next stage", which is boot_targets environment > > > > variable, and then "where that next stage looks for stuff" which is > > > > OS-dependent. Sometimes the ESP grub.cfg file is just enough to tell > > > > grub to find the full grub.cfg file elsewhere, and sometimes it's a full > > > > grub.cfg file. I think Mark is talking about the former, and you've > > > > said it's not part of this series, yet, but on the TODO list. > > > > > > Right. With the current distroboot code the order of the devices that > > > appears in boot_targets is determined by per-board/SOC/machine config > > > files and the order isn't the same for all of them. Users can change > > > the order if necessary by modifying the environment variable and > > > saving the environment. And for a one-off boot from a different > > > device they can simply run an appropriate boot command. The > > > boot_targets variable in particular is documented in various install > > > documents so it would probably be good of the new "bootmethod" code > > > would respect this variable. > > > > > > For OpenBSD I'm not really interested in the bootflow part. As I > > > explained in the past, that part of the problem is solved in a > > > (mostly) uniform way across platforms by the OpenBSD bootloader which > > > can read an /etc/boot.conf that allows bootflow customization. So as > > > long as the default of the new code still results in > > > \EFI\BOOT\BOOT{machine type short-name}.EFI being loaded and run if > > > there is no U-Boot specific bootflow configured, I'm happy. > > > > Mostly the same for FreeBSD, as long as the efi boot.efi is > > loaded and run by default (respecting the boot_targets order) we will > > be fine. > > OK thanks for the info. My expectation is that bootmethod/bootflow can > support this easily enough (it is actually simpler than distro boot). > > > > > > I can't speak for the other BSDs, but my impression is that they are > > > pretty much in the same position. The FreeBSD bootloader for example > > > supports a high-degree of "bootflow" customization and I doubt that > > > taking it out of the loop is a viable option for most users. > > I think the same may happen with grub. E.g. with Ubuntu I see quite a > bit of code in the grub.cfg file and it's not clear to me that it can > be replaced with a 'data instead of code' approach. Still, a valid > bootflow is simply to jump to an EFI app, which seems to be what is > happening here. The bootflow side is really just about describing what > to do, and this case is no different. For now I see three types of > bootflow, PXE/syslinux, EFI boot manager and EFI app. By "EFI app", do you mean a way of booting "/efi/boot/bootXX.efi" (default file name in case that no image path is specified)? In fact, this behavior, or removable media support, is defined as part of UEFI boot manager in UEFI specification. (See section 3.5) What this means is that the boot order, including a removable media case and user-provided BootXXXX cases, should be controlled solely by "BootOrder" variable. So the current combination of distro_bootcmd + UEFI boot manger doesn't fully comply with the specification. Even if those two cases are integrated, I don't know how "BootOrder" semantics can be preserved in your approach. -Takahiro Akashi > I'm travelling for three weeks soon, so if it doesn't happen this week > I'll continue this after that. > > Regards, > Simon