Re: [PATCH 0/2] Dynamic Tables

[Ard Biesheuvel <ard.biesheuvel@linaro.org>]

On 12 October 2017 at 16:43, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Tue, Oct 10, 2017 at 06:52:02PM +0000, Evan Lloyd wrote:
>> > I notice you only defined ARM namespace in this patch, and implemented
>> > ARM library instance.
>> > Also most consumers of ConfigurationManager are from ARM platform
>> > package. So if it urgent from ARM platform, you may consider to check into
>> > ArmPkg at first.
>>
>> [[Evan Lloyd]] This sounds sensible, and I will discuss it with
>> Leif.  Another option might be to start up a new module.
>
> My preference would be for this to start as a branch in edk2-staging.
>
> While there, we can bikeshed over the naming and placement (but I will
> admit to a certain affinity for Sean's suggestion to keep it separate).
>
> I do not think it is a good fit for ArmPkg.
>

+1

We are working very hard to reduce ArmPkg to pieces that are actually
specific to the ARM architecture. The fact that the Intel engineers
may not be interested in using this new dynamic table feature for
their platforms does not make it an ARM thing.

> If Evan can send out a request for creation as described in
> https://github.com/tianocore/edk2-staging/blob/about/README,
> I am happy to do the mechanics.
>