[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [GRUB2 PATCH v3 3/4] multiboot2: Do not pass memory maps to image if
From: |
Daniel Kiper |
Subject: |
Re: [GRUB2 PATCH v3 3/4] multiboot2: Do not pass memory maps to image if EFI boot services are enabled |
Date: |
Fri, 11 Mar 2016 14:30:10 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Mar 10, 2016 at 09:28:25PM +0100, Vladimir 'phcoder' Serbinenko wrote:
> On Wednesday, March 2, 2016, Daniel Kiper <address@hidden> wrote:
>
> > Do not pass memory maps to image if it asked for EFI boot services.
> > Main reason for not providing maps is because they will likely be
> > invalid. We do a few allocations after filling them, e.g. for relocator
> > needs. Usually we do not care as we would already finish boot services.
> > If we keep boot services then it is easier to not provide maps. However,
> > if image needs memory maps and they are not provided by bootloader then
> > it should get them itself just before ExitBootServices() call.
> >
> Patch gets too cluttered. Can you provide 2 variants: normal (for commit)
> and with ignoring whitespaces (for review)
Sure thing! I will prepare whole updated patch series next week.
I hope that it is OK for you.
Daniel
- Re: [GRUB2 PATCH v3 4/4] multiboot2: Add support for relocatable images, (continued)
Re: [GRUB2 PATCH v3 4/4] multiboot2: Add support for relocatable images, Vladimir 'phcoder' Serbinenko, 2016/03/10
[GRUB2 PATCH v3 3/4] multiboot2: Do not pass memory maps to image if EFI boot services are enabled, Daniel Kiper, 2016/03/02
Re: [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions, Daniel Kiper, 2016/03/09
- Re: [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions, Vladimir 'phcoder' Serbinenko, 2016/03/11
- Re: [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions, Daniel Kiper, 2016/03/11
- Re: [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions, Vladimir 'phcoder' Serbinenko, 2016/03/11
- Re: [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions, Daniel Kiper, 2016/03/11
- Re: [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions, Vladimir 'phcoder' Serbinenko, 2016/03/11
- Re: [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions, Daniel Kiper, 2016/03/11