public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Marvin Häuser" <mhaeuser@posteo.de>
To: devel@edk2.groups.io, ray.ni@intel.com
Cc: Andrew Fish <afish@apple.com>,
	"leif@nuviainc.com" <leif@nuviainc.com>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>,
	"Gao, Zhichao" <zhichao.gao@intel.com>,
	"Wang, Jian J" <jian.j.wang@intel.com>,
	"Wu, Hao A" <hao.a.wu@intel.com>,
	"Bi, Dandan" <dandan.bi@intel.com>,
	"Dong, Eric" <eric.dong@intel.com>,
	Bret Barkelew <Bret.Barkelew@microsoft.com>,
	Vitaly Cheptsov <vit9696@protonmail.com>
Subject: Re: [edk2-devel] [RFC] Expose HII package list via C variables
Date: Thu, 26 Aug 2021 18:32:33 +0000	[thread overview]
Message-ID: <a788c30c-befd-33a9-f29a-1a3b7fbb14db@posteo.de> (raw)
In-Reply-To: <CO1PR11MB4930C7C6BB8D0BD51BEB89708CC79@CO1PR11MB4930.namprd11.prod.outlook.com>

On 26/08/2021 20:19, Ni, Ray wrote:
>> 3. Saves error-prone parsing work
> This might be the key reason? Is it theoretically possible to write a secure parsing code?

Hey Ray,

Yes it is, but the most secure parsing code is that which doesn't exist. 
I'm confident in the one I wrote for the new PE/COFF library, but if 
variables suit the need, I really would like to use them, even if it is 
only by a toggle. The new proposal (please check the chain with Mike) 
was adapted to preserve the old behaviour fully for any platform that 
wants it, permanently. Amaranth will very likely drop support for the 
PE/COFF section, either nicely with an upstream PCD, or not-so-nicely 
with maintaining patches to remove the functionality downstream.

In the same chain I outlined I'm sketching a new terse file format and 
I'd prefer to avoid any unnecessary data or parsing burden. The current 
library I am sketching privately is *much* smaller than both the current 
and the new PE/COFF library. The overall format in an UEFI context (no 
dynamic linking etc.) is at least as powerful as PE/COFF and pretty much 
always smaller than TE. I'd like to keep it that way. :)

Best regards,
Marvin

>
> Thanks,
> Ray
>
>
> 
>
>


  reply	other threads:[~2021-08-26 18:32 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-25 21:21 [RFC] Expose HII package list via C variables Marvin Häuser
2021-08-25 22:33 ` [edk2-devel] " Michael D Kinney
2021-08-26  8:50   ` Marvin Häuser
2021-08-26 14:37     ` Michael D Kinney
2021-08-26 14:47       ` Marvin Häuser
2021-08-26 16:01       ` Tim Lewis
2021-08-26 16:07         ` Marvin Häuser
2021-08-26 16:44           ` Michael D Kinney
2021-08-26 17:29             ` Marvin Häuser
2021-08-26 19:50             ` Andrew Fish
2021-08-26 16:49           ` Samer El-Haj-Mahmoud
2021-08-26 17:02             ` Marvin Häuser
2021-08-26 17:07           ` Tim Lewis
2021-08-26 18:19 ` Ni, Ray
2021-08-26 18:32   ` Marvin Häuser [this message]
2021-08-26 18:38     ` [edk2-devel] " Ni, Ray
2021-08-26 19:28       ` Marvin Häuser

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=a788c30c-befd-33a9-f29a-1a3b7fbb14db@posteo.de \
    --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