public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Ard Biesheuvel via groups.io" <ardb=kernel.org@groups.io>
To: "Hernandez, Ronal" <ronal.hernandez1@hp.com>
Cc: devel@edk2.groups.io
Subject: Re: [edk2-devel] tianocore/edk2-test patch contribution, Adding new EFI_RT_PROPERTIES_TABLE test case.
Date: Thu, 5 Dec 2024 18:14:46 +0100	[thread overview]
Message-ID: <CAMj1kXHG3DVHPQaBC+guO79xn+Jywaqnd3ZNKkh6ciFEzThkEA@mail.gmail.com> (raw)
In-Reply-To: <6705.1733416611612760184@groups.io>

On Thu, 5 Dec 2024 at 17:36, Hernandez, Ronal via groups.io
<ronal.hernandez1=hp.com@groups.io> wrote:
>
> Thank you for your feedback, Ard Biesheuvel.
>
> After reviewing the UEFI Specification our interpretation is that because it is now optional to support any of the EFI Runtime Services, the EFI_RT_PROPERTIES_TABLE needs to be published for the caller at runtime to know which runtime services have been implemented.
>

No. This table is optional.

Runtime services *MAY* return EFI_UNSUPPORTED at runtime if the
service in question cannot be supported after ExitBootServices().

If this does not apply to any runtime service, there is no requirement
to publish this table.


> What we are testing in this patch is whether the EFI_RT_PROPERTIES_TABLE has been installed and whether the RuntimeServicesSupported value has been populated to reflect which optional runtime services are implemented.
>

As I said, not doing so is permitted by the UEFI specl, and so it is
not an error.

> 34. If a platform cannot support calls defined in EFI_RUNTIME_SERVICES after
> ExitBootServices() is called, that platform is permitted to provide implementations of those
> runtime services that return EFI_UNSUPPORTED when invoked at runtime. On such systems, an
> EFI_RT_PROPERTIES_TABLE configuration table should be published describing which runtime services
> are supported at runtime

*On such systems* means systems where not all runtime services are
available at runtime. On systems where all runtime services are
available at runtime, there is no requirement to publish the table.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120883): https://edk2.groups.io/g/devel/message/120883
Mute This Topic: https://groups.io/mt/109931209/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



  reply	other threads:[~2024-12-05 17:15 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-04 17:20 [edk2-devel] tianocore/edk2-test patch contribution, Adding new EFI_RT_PROPERTIES_TABLE test case Hernandez, Ronal via groups.io
2024-12-05 11:46 ` Ard Biesheuvel via groups.io
2024-12-05 16:36   ` Hernandez, Ronal via groups.io
2024-12-05 17:14     ` Ard Biesheuvel via groups.io [this message]
2024-12-06 20:11       ` Hernandez, Ronal via groups.io

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-list 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=CAMj1kXHG3DVHPQaBC+guO79xn+Jywaqnd3ZNKkh6ciFEzThkEA@mail.gmail.com \
    --to=devel@edk2.groups.io \
    /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