Re: [PATCH 2/2] ArmVirtPkg: enable EBC interpreter for AArch64 QEMU

[Laszlo Ersek <lersek@redhat.com>]

On 08/17/16 15:47, Ard Biesheuvel wrote:
> On 17 August 2016 at 15:45, Leif Lindholm <leif.lindholm@linaro.org> wrote:
>> Since we now have EBC support for AArch64, enable it by default
>> on the QEMU platform.
>>
>> Contributed-under: TianoCore Contribution Agreement 1.0
>> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
>> ---
>>  ArmVirtPkg/ArmVirtQemu.dsc           | 1 +
>>  ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc | 5 +++++
>>  2 files changed, 6 insertions(+)
>>
>> diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc
>> index 9f88786..fb851e7 100644
>> --- a/ArmVirtPkg/ArmVirtQemu.dsc
>> +++ b/ArmVirtPkg/ArmVirtQemu.dsc
>> @@ -401,3 +401,4 @@ [Components.AARCH64]
>>      <LibraryClasses>
>>        NULL|ArmVirtPkg/Library/FdtPciPcdProducerLib/FdtPciPcdProducerLib.inf
>>    }
>> +  MdeModulePkg/Universal/EbcDxe/EbcDxe.inf
> 
> Please add it to ArmVirtPkg/ArmVirtQemuKernel.dsc as well:
> ArmVirtQemuFvMain.fdf.inc is shared between them.
> 
> With that,
> 
> Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

Any reason for not adding it to the Xen platform as well?

What kind of modules do we expect to come in EBC? Option ROMs perhaps?
In that case, Xen may indeed be irrelevant here (I have no clue about
adding oproms to emulated or assigned devices under Xen, so I can't make
an argument either way.) If we expect general stuff (for example, even
UEFI_APPLICATION modules) to arrive in EBC, then I think Xen would be a
justified addition.

Thanks
Laszlo

>> diff --git a/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc b/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc
>> index ad7037f..0a5b899 100644
>> --- a/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc
>> +++ b/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc
>> @@ -146,6 +146,11 @@ [FV.FvMain]
>>    #
>>    INF MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableDxe.inf
>>    INF OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpiPlatformDxe.inf
>> +
>> +  #
>> +  # EBC support
>> +  #
>> +  INF MdeModulePkg/Universal/EbcDxe/EbcDxe.inf
>>  !endif
>>
>>    #
>> --
>> 2.1.4
>>