[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 3/3] mkimage: Clarify file alignment in efi case
From: |
Daniel Kiper |
Subject: |
Re: [PATCH v5 3/3] mkimage: Clarify file alignment in efi case |
Date: |
Mon, 28 Jan 2019 14:17:01 +0100 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Mon, Jan 28, 2019 at 01:34:20PM +0100, Alexander Graf wrote:
> On 28.01.19 13:27, Daniel Kiper wrote:
> > On Fri, Jan 25, 2019 at 12:45:16PM +0100, Alexander Graf wrote:
> >> There are a few spots in the PE generation code for EFI binaries that uses
> >> the section alignment rather than file alignment, even though the alignment
> >> is really only file bound.
> >>
> >> Replace those cases with the file alignment constant instead.
> >>
> >> Reported-by: Daniel Kiper <address@hidden>
> >> Signed-off-by: Alexander Graf <address@hidden>
> >
> > Great! However, this patch misses changes for EFI32_HEADER_SIZE
> > and EFI64_HEADER_SIZE macros. In general I think about
> > s/GRUB_PE32_SECTION_ALIGNMENT/GRUB_PE32_FILE_ALIGNMENT/
> > I have asked about that in my earlier emails too...
>
> If you have such a strong opinion, why don't you just simply do the
> patch and I review it? The way we're bouncing this back and forth is
> unproductive for both of us.
Yes, I agree. However, I am asking you about that for quite long time.
You do not object and you do not take into account this comment. So,
I assume that you are missing it. If you would do one of above things
earlier I would not chase you up until now. Just please read my comments
more carefully.
Daniel