From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: Stuart_Hayes@Dell.com
Cc: bug-grub@gnu.org, grub-devel@gnu.org, Jared_Dominguez@DELL.com
Subject: Re: [bug #36532] boot in EFI mode (x86_64) fails on some systems
Date: Wed, 27 Jun 2012 22:50:38 +0200 [thread overview]
Message-ID: <4FEB721E.3080908@gmail.com> (raw)
In-Reply-To: <959D45574D89AF41A9DADF6F446A2E9A2AEC020701@AUSX7MCPS310.AMER.DELL.COM>
[-- Attachment #1: Type: text/plain, Size: 1323 bytes --]
On 27.06.2012 21:59, Stuart_Hayes@Dell.com wrote:
>>
>> Well, this works, too (on top of HEAD from 7 june).
>>
>>
>> --- ../../grub/grub-core/loader/i386/linux.c 2012-05-31
>> 12:59:19.000000000 -0400
>> +++ grub-core/loader/i386/linux.c 2012-06-12 20:47:51.455718055 -0400
>> @@ -139,7 +139,7 @@ find_efi_mmap_size (void)
>>
>> /* Increase the size a bit for safety, because GRUB allocates more
>> on
>> later, and EFI itself may allocate more. */
>> - mmap_size += (1 << 12);
>> + mmap_size += (3 << 12);
>>
>> mmap_size = page_align (mmap_size);
>> return mmap_size;
>
> Vladimir,
>
> The 2.00rc1 version of grub2 still doesn't fix the efi memory map buffer size I've been working on (though I can see you are now allocating the efi memory map buffer very shortly before you are calling grub_efi_finish_boot_services()).
>
> Increasing the mmap_size in find_efi_mmap_size()--as in the patch immediately above this text--does fix the problem. Even adding (2 << 12) (instead of (1 << 12)) to the mmap_size will work on the system I'm testing with.
>
I've changed it to 3. Thanks. It's annoying that even such simple
operations as we have between find_efi_mmap and finish drastically
increase memory map size.
--
Regards
Vladimir 'φ-coder/phcoder' Serbinenko
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]
next prev parent reply other threads:[~2012-06-27 20:50 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20120523-193311.sv88235.49420@savannah.gnu.org>
[not found] ` <20120531-112908.sv72589.64321@savannah.gnu.org>
[not found] ` <959D45574D89AF41A9DADF6F446A2E9A2AE507A772@AUSX7MCPS310.AMER.DELL.COM>
[not found] ` <959D45574D89AF41A9DADF6F446A2E9A2AE55A707B@AUSX7MCPS310.AMER.DELL.COM>
[not found] ` <4FD1180B.8090709@gmail.com>
[not found] ` <959D45574D89AF41A9DADF6F446A2E9A2AE5668374@AUSX7MCPS310.AMER.DELL.COM>
[not found] ` <4FD646FE.4050301@gmail.com>
[not found] ` <959D45574D89AF41A9DADF6F446A2E9A2AE56AEA8E@AUSX7MCPS310.AMER.DELL.COM>
[not found] ` <4FD79D9E.6010702@gmail.com>
2012-06-27 19:59 ` [bug #36532] boot in EFI mode (x86_64) fails on some systems Stuart_Hayes
2012-06-27 20:50 ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]
2012-06-27 20:58 ` Stuart_Hayes
2012-06-28 21:39 ` Stuart_Hayes
2012-06-29 20:49 ` Stuart_Hayes
2012-06-29 21:14 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-07-02 17:49 ` Stuart_Hayes
2012-05-24 17:27 Stuart_Hayes
2012-05-26 10:18 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-05-27 14:52 ` Vladimir 'φ-coder/phcoder' Serbinenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FEB721E.3080908@gmail.com \
--to=phcoder@gmail.com \
--cc=Jared_Dominguez@DELL.com \
--cc=Stuart_Hayes@Dell.com \
--cc=bug-grub@gnu.org \
--cc=grub-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.