All of lore.kernel.org
 help / color / mirror / Atom feed
From: baskov@ispras.ru
To: Paul Menzel <pmenzel@molgen.mpg.de>
Cc: grub-devel@gnu.org
Subject: Re: [PATCH RFC 1/3] efi: provide definitions of DXE services table
Date: Thu, 14 Apr 2022 18:15:20 +0300	[thread overview]
Message-ID: <f3f3e282801c78ceca2d84dc63a75178@ispras.ru> (raw)
In-Reply-To: <55222746-e3ce-11f2-e528-235c63f7f469@molgen.mpg.de>

On 2022-04-11 10:12, Paul Menzel wrote:
> Dear Baskov,
> 
> 
> Thank you for your patches.
> 
> 
> Am 07.04.22 um 05:32 schrieb Baskov Evgeniy:
>> DXE services can be used to change memory attributes
>> on systems where EFI use stricter policies about memory
>> access rights and sets NX flag on some pages required by
>> grub to executable.
> 
> If you resend, please use (at least) 72 characters per line.

Thank you, I will take that in mind for the next version.

> 
> For the UEFI ignorant, could you please reference the UEFI
> specification name, revision, and section needed for review?
> 

DXE services are part of UEFI Platform Initialization Specification [1]
Functions used in this patch are described in section 7.2.4, Global
Coherency Domain Services, starting with page 325 (2-41) in version 
1.7A.

Actually, the fact that DXE services are not the part of the main UEFI
spec, was one of the reasons to create the new protocol.
The one remaining problem with it is that it is not supported yet by
the implementation which was the reason for this patch to exist.

[1] 
https://uefi.org/sites/default/files/resources/PI_Spec_1_7_A_final_May1.pdf

Thanks,
Baskov Evgeniy



  reply	other threads:[~2022-04-14 15:15 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-07  3:32 [PATCH RFC 0/3] EFI memory attributes Baskov Evgeniy
2022-04-07  3:32 ` [PATCH RFC 1/3] efi: provide definitions of DXE services table Baskov Evgeniy
2022-04-11  7:12   ` Paul Menzel
2022-04-14 15:15     ` baskov [this message]
2022-04-07  3:32 ` [PATCH RFC 2/3] relocator: allocate trampoline in lower 640k Baskov Evgeniy
2022-04-07  3:32 ` [PATCH RFC 3/3] efi: explicitly set memory attributes for memory Baskov Evgeniy

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=f3f3e282801c78ceca2d84dc63a75178@ispras.ru \
    --to=baskov@ispras.ru \
    --cc=grub-devel@gnu.org \
    --cc=pmenzel@molgen.mpg.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 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.