From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1YGGwZ-0008R2-NN for mharc-grub-devel@gnu.org; Tue, 27 Jan 2015 19:57:19 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50901) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YGGwW-0008Q5-QS for grub-devel@gnu.org; Tue, 27 Jan 2015 19:57:17 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YGGwT-0007ES-HX for grub-devel@gnu.org; Tue, 27 Jan 2015 19:57:16 -0500 Received: from cheddar.halon.org.uk ([217.10.144.130]:42062) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YGGwT-0007E0-C2 for grub-devel@gnu.org; Tue, 27 Jan 2015 19:57:13 -0500 Received: from bsmtp by cheddar.halon.org.uk with local-bsmtp (Exim 4.80) (envelope-from ) id 1YGGwQ-0003B3-Tg for grub-devel@gnu.org; Wed, 28 Jan 2015 00:57:10 +0000 Received: from steve by tack.local with local (Exim 4.80) (envelope-from ) id 1YGGvm-0002Ce-S2 for grub-devel@gnu.org; Wed, 28 Jan 2015 00:56:30 +0000 Date: Wed, 28 Jan 2015 00:56:30 +0000 From: Steve McIntyre To: grub-devel@gnu.org Subject: [PATCH] Add support for running a 64-bit Linux kernel on a 32-bit EFI Message-ID: <20150128005625.GC20325@einval.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline X-attached: none User-Agent: Mutt/1.5.21 (2010-09-15) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 217.10.144.130 X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: The development of GNU GRUB List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 28 Jan 2015 00:57:17 -0000 Hi folks, I've been working in Debian on adding support for amd64 platforms which are shipped with 32-bit UEFI firmware, such as the Asus X205TA and other Bay Trail machines. As part of that, I've had a patch accepted by the Linux EFI maintainers [1] to expose the size of the underlying firmware to userland via a new sysfs file. It should make mainline Linux very soon. Using that patch as a basis, I've patched grub to use the new sysfs interface to work out the correct default x86 platform to match the firmware. Colin's happy with the patch and has just accepted it for Debian's grub package for now; I promised to submit it upstream for review, so here it is. :-) Feedback welcome please! [1] http://article.gmane.org/gmane.linux.kernel.efi/5229 ===================================================================== Some platforms might be capable of running a 64-bit Linux kernel but only use a 32-bit EFI. To support such systems, it is necessary to work out the size of the firmware rather than just the size of the kernel. To enable that, there is now an extra EFI sysfs file to describe the underlying firmware. Read that if possible, otherwise fall back to the kernel type as before. Signed-off-by: Steve McIntyre --- grub-core/osdep/linux/platform.c | 38 +++++++++++++++++++++++++++++++++++++- 1 file changed, 37 insertions(+), 1 deletion(-) diff --git a/grub-core/osdep/linux/platform.c b/grub-core/osdep/linux/platform.c index 4b9f6ef..5668ae5 100644 --- a/grub-core/osdep/linux/platform.c +++ b/grub-core/osdep/linux/platform.c @@ -60,6 +60,42 @@ is_64_kernel (void) return strcmp (un.machine, "x86_64") == 0; } +static int +read_platform_size (void) +{ + FILE *fp; + char *buf = NULL; + size_t len = 0; + int ret = 0; + + /* Newer kernels can tell us directly about the size of the + * underlying firmware - let's see if that interface is there. */ + fp = grub_util_fopen ("/sys/firmware/efi/fw_platform_size", "r"); + if (fp != NULL) + { + if (getline (&buf, &len, fp) > 0) + { + if (strncmp (buf, "32", 2) == 0) + ret = 32; + else if (strncmp (buf, "64", 2) == 0) + ret = 64; + } + free (buf); + fclose (fp); + } + + if (ret == 0) + /* Unrecognised - fall back to matching the kernel size instead */ + { + if (is_64_kernel ()) + ret = 64; + else + ret = 32; + } + + return ret; +} + const char * grub_install_get_default_x86_platform (void) { @@ -77,7 +113,7 @@ grub_install_get_default_x86_platform (void) if (is_not_empty_directory ("/sys/firmware/efi")) { grub_util_info ("...found"); - if (is_64_kernel ()) + if (read_platform_size() == 64) return "x86_64-efi"; else return "i386-efi"; -- 1.7.10.4 -- Steve McIntyre, Cambridge, UK. steve@einval.com "You can't barbecue lettuce!" -- Ellie Crane