From: anis chali <chalianis1@gmail.com>
To: Ahmad Fatoum <a.fatoum@pengutronix.de>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH v3 0/4] state: generic devicetree-overlay based state node injection
Date: Wed, 26 Aug 2026 01:23:46 +0200 [thread overview]
Message-ID: <CAL+1fyD-X9awDg=JmKCMZdHqermbozsFU5dMce=E7PNNW_AjGQ@mail.gmail.com> (raw)
In-Reply-To: <b51c5e54-7ce7-40f5-b65d-1d8ef7a07be6@pengutronix.de>
Hi
Le mar. 25 août 2026 à 19:10, Ahmad Fatoum <a.fatoum@pengutronix.de> a écrit :
>
> Hi,
>
> On 8/25/26 5:02 PM, Ahmad Fatoum wrote:
> > On 8/25/26 5:05 AM, chalianis1@gmail.com wrote:
> >> so a real state.dtb collides with whatever root the EFI
> >> payload already registered at boot and is rejected with -EBUSY, and a
> >> rejected tree's /aliases entries never reach the global alias cache
> >> of_alias_get() relies on either.
> >
> > This argumentation follows from a bug, so it doesn't say anything to the
> > merit of this new feature.
>
> I have now read through the series and my thoughts are that applying
> overlays to the barebox live tree from outside can be useful (I wouldn't
> limit it to state), but:
Ack, and if we have overlays we can apply whatever we want (state, other things)
it more useful than the state, because it permits to add more stuff
and permits to
address state and more.
> - it needs better argumentation what it allows that is not feasible with
> fragments
> - it needs some way to account for multi-image
Yes, agreed. One approach could be to use a global label, or
alternatively scan all
barebox state partitions and check for a tag that matches the
corresponding state
fragment or overlay. The matching backend partition would then be used.
> I am going to send out my EFI-payload-state-via-fragment series soon and
> I am interested to hear your thoughts on it.
>
> Thanks,
> Ahmad
>
> --
> Pengutronix e.K. | |
> Steuerwalder Str. 21 | http://www.pengutronix.de/ |
> 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
>
next prev parent reply other threads:[~2026-08-25 23:25 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-25 3:05 chalianis1
2026-08-25 3:05 ` [PATCH v3 1/4] state: make of_state_fixup() usable outside common/state/ chalianis1
2026-08-25 17:00 ` Ahmad Fatoum
2026-08-25 23:34 ` anis chali
2026-08-25 3:05 ` [PATCH v3 2/4] state: add CONFIG_STATE_OVERLAY to inject a state node via devicetree overlay chalianis1
2026-08-25 16:57 ` Ahmad Fatoum
2026-08-25 23:36 ` anis chali
2026-08-25 3:05 ` [PATCH v3 3/4] efi: payload: export resolved state as a BareboxState UEFI variable chalianis1
2026-08-25 17:04 ` Ahmad Fatoum
2026-08-25 23:30 ` anis chali
2026-08-25 3:05 ` [PATCH v3 4/4] efi: payload: make CONFIG_STATE_OVERLAY (and BareboxState export) work on x86 chalianis1
2026-08-25 17:06 ` Ahmad Fatoum
2026-08-25 15:02 ` [PATCH v3 0/4] state: generic devicetree-overlay based state node injection Ahmad Fatoum
2026-08-25 17:10 ` Ahmad Fatoum
2026-08-25 23:23 ` anis chali [this message]
2026-08-25 23:13 ` anis chali
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='CAL+1fyD-X9awDg=JmKCMZdHqermbozsFU5dMce=E7PNNW_AjGQ@mail.gmail.com' \
--to=chalianis1@gmail.com \
--cc=a.fatoum@pengutronix.de \
--cc=barebox@lists.infradead.org \
/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