Re: [PATCH] arm64/efi: set PE/COFF file alignment to 512 bytes

Ard Biesheuvel
2014-10-22T08:02:37Z

On 21 October 2014 14:35, Matt Fleming <matt-HNK1S37rvNbeXh+fF434Mdi2O/JbrIOy@public.gmane.org> wrote:
> On Fri, 10 Oct, at 06:58:55PM, Ard Biesheuvel wrote:
>> Change our PE/COFF header to use the minimum file alignment of
>> 512 bytes (0x200), as mandated by the PE/COFF spec v8.3
>>
>> Also update the linker script so that the Image file itself is also a
>> round multiple of FileAlignment.
>>
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
>> ---
>>
>> Another one from the PE/COFF rabbit hole. I will follow up next week and repost
>> as a coherent series, once I have collected all the feedback.
>
> It appears a though next week never came.
>
> Should I pick up this version and apply ACK tags from the mailing list?
> Or are you planning on sending a larger series?
>
> It sounds like this should be tagged for stable, and placed into the EFI
> urgent queue?
>

I have a couple of patches like this one in my queue, and I don't
think any of them are urgent. These alignment fixes etc are mainly for
full compliance with the various specs, and there are no known issues
being solved by them.

I was going to wait for -rc1 and rebase on top of that, but it appears
your branch hasn't been picked up?

--
Ard.