From: Tom Rini <trini@konsulko.com>
To: Simon Glass <sjg@chromium.org>
Cc: "U-Boot Mailing List" <u-boot@lists.denx.de>,
"AKASHI Takahiro" <akashi.tkhro@gmail.com>,
"Alexander Gendin" <agendin@matrox.com>,
"Caleb Connolly" <caleb.connolly@linaro.org>,
"Dragan Simic" <dsimic@manjaro.org>,
"Eddie James" <eajames@linux.ibm.com>,
"Emil Kronborg" <emil.kronborg@protonmail.com>,
"Francis Laniel" <francis.laniel@amarulasolutions.com>,
"Guillaume La Roque" <glaroque@baylibre.com>,
"Heinrich Schuchardt" <xypron.glpk@gmx.de>,
"Hou Zhiqiang" <Zhiqiang.Hou@nxp.com>,
"Igor Opaniuk" <igor.opaniuk@gmail.com>,
"Ilias Apalodimas" <ilias.apalodimas@linaro.org>,
"Ion Agorria" <ion@agorria.com>,
"Jerome Forissier" <jerome.forissier@linaro.org>,
"Jonas Karlman" <jonas@kwiboo.se>,
"Julien Masson" <jmasson@baylibre.com>,
"Marek Vasut" <marek.vasut+renesas@mailbox.org>,
"Marek Vasut" <marex@denx.de>,
"Mark Kettenis" <kettenis@openbsd.org>,
"Martyn Welch" <martyn.welch@collabora.com>,
"Mattijs Korpershoek" <mkorpershoek@baylibre.com>,
"Michael Trimarchi" <michael@amarulasolutions.com>,
"Michal Simek" <michal.simek@amd.com>,
"Nam Cao" <namcao@linutronix.de>,
"Neil Armstrong" <neil.armstrong@linaro.org>,
"Oliver Gaskell" <Oliver.Gaskell@analog.com>,
"Pavel Herrmann" <morpheus.ibis@gmail.com>,
"Peter Robinson" <pbrobinson@gmail.com>,
"Quentin Schulz" <quentin.schulz@cherry.de>,
"Rasmus Villemoes" <rasmus.villemoes@prevas.dk>,
"Robert Marko" <robert.marko@sartura.hr>,
"Sam Protsenko" <semen.protsenko@linaro.org>,
"Sean Edmond" <seanedmond@microsoft.com>,
"Sebastian Reichel" <sebastian.reichel@collabora.com>,
"Shantur Rathore" <i@shantur.com>, "Stefan Roese" <sr@denx.de>,
"Sughosh Ganu" <sughosh.ganu@linaro.org>,
"Sumit Garg" <sumit.garg@linaro.org>,
"Svyatoslav Ryhel" <clamor95@gmail.com>,
"Thomas Weißschuh" <thomas.weissschuh@linutronix.de>,
"Tim Harvey" <tharvey@gateworks.com>,
"Tom Fitzhenry" <tom@tom-fitzhenry.me.uk>,
"Tony Dinh" <mibodhi@gmail.com>,
"Vincent Stehlé" <vincent.stehle@arm.com>
Subject: Re: [PATCH v2 00/31] bootstd: Support recording images
Date: Sat, 19 Oct 2024 10:31:16 -0600 [thread overview]
Message-ID: <20241019163116.GV4959@bill-the-cat> (raw)
In-Reply-To: <20241019152211.1014896-1-sjg@chromium.org>
[-- Attachment #1: Type: text/plain, Size: 997 bytes --]
On Sat, Oct 19, 2024 at 09:21:40AM -0600, Simon Glass wrote:
> This series provides a way to keep track of the images used in bootstd,
> including the type of each image.
>
> At present this is sort-of handled by struct bootflow but in quite an
> ad-hoc way. The structure has become quite large and is hard to query.
> Future work will be able to reduce its size.
>
> Ultimately the 'bootflow info' command may change to also show images as
> a list, but that is left for later, as this series is already fairly
> long. So for now, just introduce the concept and adjust bootstd to use
> it, with a simple command to list the images.
>
> This series includes various alist enhancements, to make use of this new
> data structure a little easier.
>
> Changes in v2:
> - Add an image type extension in bootflow.h
I really wish you hadn't posted v2 so quickly because this isn't what I
want to see happen and I'm not convinced that this is the right path
yet.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
prev parent reply other threads:[~2024-10-20 0:21 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-19 15:21 [PATCH v2 00/31] bootstd: Support recording images Simon Glass
2024-10-19 15:21 ` [PATCH v2 01/31] alist: Mention the error condition in alist_add_placeholder() Simon Glass
2024-10-19 15:21 ` [PATCH v2 02/31] alist: Add a comment for alist_init_struct() Simon Glass
2024-10-19 15:21 ` [PATCH v2 03/31] alist: Expand the comment for alist_get() Simon Glass
2024-10-19 15:21 ` [PATCH v2 04/31] alist: Add a way to get the next element Simon Glass
2024-10-19 15:21 ` [PATCH v2 05/31] alist: Add for-loop helpers Simon Glass
2024-10-19 15:21 ` [PATCH v2 06/31] alist: Add a function to empty the list Simon Glass
2024-10-19 15:21 ` [PATCH v2 07/31] alist: Add a way to efficiently filter an alist Simon Glass
2024-10-19 15:21 ` [PATCH v2 08/31] alist: Add maintainer Simon Glass
2024-10-19 15:21 ` [PATCH v2 09/31] dm: core: Add a function to see if a device exists Simon Glass
2024-10-19 15:21 ` [PATCH v2 10/31] test: boot: Use a consistent name for the script bootmeth Simon Glass
2024-10-22 13:51 ` Mattijs Korpershoek
2024-10-19 15:21 ` [PATCH v2 11/31] bootstd: Move bootflow-adding to bootstd Simon Glass
2024-10-19 15:21 ` [PATCH v2 12/31] bootstd: Move bootflow-clearing " Simon Glass
2024-10-19 15:21 ` [PATCH v2 13/31] bootstd: Add a function to get bootstd only if available Simon Glass
2024-10-19 15:21 ` [PATCH v2 14/31] bootstd: Drop the bootdev-specific list of bootflows Simon Glass
2024-10-19 15:21 ` [PATCH v2 15/31] bootstd: Move the bootflow list into an alist Simon Glass
2024-10-19 15:21 ` [PATCH v2 16/31] test: Expand implementation of ut_list_has_dm_tests() Simon Glass
2024-10-19 15:21 ` [PATCH v2 17/31] test: Drop the duplicate line in setup_bootmenu_image() Simon Glass
2024-10-19 15:21 ` [PATCH v2 18/31] test: boot: Update bootflow_iter() for console checking Simon Glass
2024-10-19 15:21 ` [PATCH v2 19/31] bootstd: cros: Correct the x86-setup address Simon Glass
2024-10-19 15:22 ` [PATCH v2 20/31] bootstd: Maintain a list of images Simon Glass
2024-10-19 15:22 ` [PATCH v2 21/31] bootstd: Update bootmeth_alloc_file() to record images Simon Glass
2024-10-19 15:22 ` [PATCH v2 22/31] boot: pxe: Drop the duplicate comment on get_pxe_file() Simon Glass
2024-10-19 15:22 ` [PATCH v2 23/31] bootmeth_efi: Simplify reading files by using the common function Simon Glass
2024-10-21 10:25 ` Ilias Apalodimas
2024-10-19 15:22 ` [PATCH v2 24/31] bootmeth: Update the read_file() method to include a type Simon Glass
2024-10-22 13:55 ` Mattijs Korpershoek
2024-10-19 15:22 ` [PATCH v2 25/31] bootmeth_efi: Check the filename-allocation in the network path Simon Glass
2024-10-19 15:22 ` [PATCH v2 26/31] boot: Update extlinux pxe_getfile_func() to include type Simon Glass
2024-10-19 15:22 ` [PATCH v2 27/31] boot: Update pxe bootmeth to record images Simon Glass
2024-10-19 15:22 ` [PATCH v2 28/31] Update bootmeth_alloc_other() " Simon Glass
2024-10-19 15:22 ` [PATCH v2 29/31] bootstd: Avoid showing an invalid buffer address Simon Glass
2024-10-19 15:22 ` [PATCH v2 30/31] bootstd: Update cros bootmeth to record images Simon Glass
2024-10-19 15:22 ` [PATCH v2 31/31] bootstd: Add a simple command to list images Simon Glass
2024-10-19 16:31 ` Tom Rini [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=20241019163116.GV4959@bill-the-cat \
--to=trini@konsulko.com \
--cc=Oliver.Gaskell@analog.com \
--cc=Zhiqiang.Hou@nxp.com \
--cc=agendin@matrox.com \
--cc=akashi.tkhro@gmail.com \
--cc=caleb.connolly@linaro.org \
--cc=clamor95@gmail.com \
--cc=dsimic@manjaro.org \
--cc=eajames@linux.ibm.com \
--cc=emil.kronborg@protonmail.com \
--cc=francis.laniel@amarulasolutions.com \
--cc=glaroque@baylibre.com \
--cc=i@shantur.com \
--cc=igor.opaniuk@gmail.com \
--cc=ilias.apalodimas@linaro.org \
--cc=ion@agorria.com \
--cc=jerome.forissier@linaro.org \
--cc=jmasson@baylibre.com \
--cc=jonas@kwiboo.se \
--cc=kettenis@openbsd.org \
--cc=marek.vasut+renesas@mailbox.org \
--cc=marex@denx.de \
--cc=martyn.welch@collabora.com \
--cc=mibodhi@gmail.com \
--cc=michael@amarulasolutions.com \
--cc=michal.simek@amd.com \
--cc=mkorpershoek@baylibre.com \
--cc=morpheus.ibis@gmail.com \
--cc=namcao@linutronix.de \
--cc=neil.armstrong@linaro.org \
--cc=pbrobinson@gmail.com \
--cc=quentin.schulz@cherry.de \
--cc=rasmus.villemoes@prevas.dk \
--cc=robert.marko@sartura.hr \
--cc=seanedmond@microsoft.com \
--cc=sebastian.reichel@collabora.com \
--cc=semen.protsenko@linaro.org \
--cc=sjg@chromium.org \
--cc=sr@denx.de \
--cc=sughosh.ganu@linaro.org \
--cc=sumit.garg@linaro.org \
--cc=tharvey@gateworks.com \
--cc=thomas.weissschuh@linutronix.de \
--cc=tom@tom-fitzhenry.me.uk \
--cc=u-boot@lists.denx.de \
--cc=vincent.stehle@arm.com \
--cc=xypron.glpk@gmx.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