From: Luca Fancellu <Luca.Fancellu@arm.com>
To: Stefano Stabellini <sstabellini@kernel.org>
Cc: "Demi Marie Obenour" <demi@invisiblethingslab.com>,
"Julien Grall" <julien@xen.org>,
"Bertrand Marquis" <Bertrand.Marquis@arm.com>,
"Volodymyr Babchuk" <Volodymyr_Babchuk@epam.com>,
"Jan Beulich" <jbeulich@suse.com>,
"Andrew Cooper" <andrew.cooper3@citrix.com>,
"Roger Pau Monné" <roger.pau@citrix.com>, "Wei Liu" <wl@xen.org>,
"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Subject: Re: [PATCH v2] Grab the EFI System Resource Table and check it
Date: Tue, 5 Apr 2022 19:42:16 +0000 [thread overview]
Message-ID: <ADA1764C-C67F-41A8-9CD3-8C421188A5FE@arm.com> (raw)
In-Reply-To: <alpine.DEB.2.22.394.2204051220520.2910984@ubuntu-linux-20-04-desktop>
> On 5 Apr 2022, at 20:21, Stefano Stabellini <sstabellini@kernel.org> wrote:
>
> On Mon, 4 Apr 2022, Luca Fancellu wrote:
>>> On 2 Apr 2022, at 00:14, Demi Marie Obenour <demi@invisiblethingslab.com> wrote:
>>>
>>> The EFI System Resource Table (ESRT) is necessary for fwupd to identify
>>> firmware updates to install. According to the UEFI specification §23.4,
>>> the table shall be stored in memory of type EfiBootServicesData.
>>> Therefore, Xen must avoid reusing that memory for other purposes, so
>>> that Linux can access the ESRT. Additionally, Xen must mark the memory
>>> as reserved, so that Linux knows accessing it is safe.
>>>
>>> See https://lore.kernel.org/xen-devel/20200818184018.GN1679@mail-itl/T/
>>> for details.
>>>
>>> Signed-off-by: Demi Marie Obenour <demi@invisiblethingslab.com>
>>
>> Hi,
>>
>> I’ve tested the patch on an arm machine booting Xen+Dom0 through EFI, unfortunately
>> I could not test the functionality.
>
> I understand you couldn't test ESRT but did the basic Xen+Dom0 boot via
> EFI on ARM work?
Yes, I realise now I should have added *and it works* before the comma, without it the sentence is
misleading.
Cheers,
Luca
next prev parent reply other threads:[~2022-04-05 19:43 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-01 23:14 [PATCH v2] Grab the EFI System Resource Table and check it Demi Marie Obenour
2022-04-04 11:08 ` Luca Fancellu
2022-04-05 19:21 ` Stefano Stabellini
2022-04-05 19:42 ` Luca Fancellu [this message]
2022-04-06 10:10 ` Jan Beulich
2022-04-18 21:36 ` Demi Marie Obenour
-- strict thread matches above, loose matches on Subject: below --
2022-03-30 20:01 Demi Marie Obenour
2022-03-31 7:04 ` Luca Fancellu
2022-03-31 11:57 ` Demi Marie Obenour
2022-03-31 12:05 ` Luca Fancellu
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=ADA1764C-C67F-41A8-9CD3-8C421188A5FE@arm.com \
--to=luca.fancellu@arm.com \
--cc=Bertrand.Marquis@arm.com \
--cc=Volodymyr_Babchuk@epam.com \
--cc=andrew.cooper3@citrix.com \
--cc=demi@invisiblethingslab.com \
--cc=jbeulich@suse.com \
--cc=julien@xen.org \
--cc=roger.pau@citrix.com \
--cc=sstabellini@kernel.org \
--cc=wl@xen.org \
--cc=xen-devel@lists.xenproject.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.