Re: [PATCH edk2-non-osi 2/2] Silicon/AMD/Styx: update ArmTrustedFirmware.bin

[Marc Zyngier <marc.zyngier@arm.com>]

On 01/02/18 16:42, Ard Biesheuvel wrote:
> On 1 February 2018 at 16:04, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
>> The ARM Trusted Firmware build we have been using up until now was built
>> with optimizations disabled (which means every variable manipulation
>> involves a load, the operation itself and a store), and runs with the
>> MMU disabled, making it needlessly slow.
>>
>> This appears to be due to the fact that
>> a) the page tables are not set up correctly, so not all memory can be
>>    accessed from EL3
>> b) the handling of SMC service calls does not take into account that
>>    these calls may be made with the MMU off (e.g., by UEFI PEI).
>>
>> These issues have been fixed in the source code, which should hopefully
>> remove any performance bottlenecks that may become more noticeable now
>> that we are going to call into the secure firmware more often to perform
>> Spectre variant 2 mitigations, which have been backported as well.
>>
>> So update the binary image to a RELEASE build that was created with
>> optimizations enabled, and has the above fixes incorporated.
>>
>> Contributed-under: TianoCore Contribution Agreement 1.1
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> ---
>>  Silicon/AMD/Styx/ArmTrustedFirmware.bin | Bin 75344 -> 34320 bytes
>>  1 file changed, 0 insertions(+), 0 deletions(-)
>>
> 
> Note to Marc: this is the exact image I shared with you as
> bl31.bin-release earlier today, so if that works as expected, could
> you report back here please? Thanks.

Absolutely. I'll give it a shot at the weekend (or earlier if I can).

Thanks,

	M.
-- 
Jazz is not dead. It just smells funny...