From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753428AbaIYOle (ORCPT ); Thu, 25 Sep 2014 10:41:34 -0400 Received: from mail-wi0-f171.google.com ([209.85.212.171]:41712 "EHLO mail-wi0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752501AbaIYOlc (ORCPT ); Thu, 25 Sep 2014 10:41:32 -0400 Date: Thu, 25 Sep 2014 16:41:27 +0200 From: Ingo Molnar To: Matt Fleming , Linus Torvalds Cc: "H. Peter Anvin" , Thomas Gleixner , linux-efi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [GIT PULL] EFI urgent fixes Message-ID: <20140925144127.GA3828@gmail.com> References: <20140925073133.GQ18635@console-pimps.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140925073133.GQ18635@console-pimps.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Matt Fleming wrote: > Folks, > > Please consider pulling the following fixes. > > The first two are releated to the bug report I got from Linus and Josh > Boyer, whereby Fedora20 + grub2 systems were no longer booting. Linus > reverted the offending commit that broke grub2 boot, but that left us in > a state where Macbooks still didn't boot with EFI. This is fixed with > the below revert and I'm dropping a noisey boot time efi_printk() based > on Josh's request and Linus' OK. > > The third patch fixes a 32-bit EFI boot stub bug where garbled text is > displayed on the screen if any of the efi_printk() statements run. > > The following changes since commit 3eddc69ffeba092d288c386646bfa5ec0fce25fd: > > x86 early_ioremap: Increase FIX_BTMAPS_SLOTS to 8 (2014-09-14 15:24:31 +0100) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git tags/efi-urgent > > for you to fetch changes up to 115c6628a59044958c205f8468a1b3ba3d539e61: > > x86/efi: Truncate 64-bit values when calling 32-bit OutputString() (2014-09-24 21:56:46 +0100) > > ---------------------------------------------------------------- > * Revert the static library changes from the merge window since they're > causing issues for Macbooks and Fedora + Grub2 - Matt Fleming > > * Delete the misleading "setup_efi_pci() failed!" message which some > people are seeing when booting EFI - Matt Fleming > > * Fix printing strings from the 32-bit EFI boot stub by only passing > 32-bit addresses to the firmware - Matt Fleming > > ---------------------------------------------------------------- > Matt Fleming (3): > Revert "efi/x86: efistub: Move shared dependencies to " > x86/efi: Delete misleading efi_printk() error message > x86/efi: Truncate 64-bit values when calling 32-bit OutputString() > > arch/x86/boot/compressed/Makefile | 3 +-- > arch/x86/boot/compressed/eboot.c | 44 +++++++++++++++++++++++---------------- > arch/x86/boot/compressed/eboot.h | 16 ++++++++++++++ > arch/x86/include/asm/efi.h | 24 --------------------- > drivers/firmware/efi/Makefile | 2 +- > 5 files changed, 44 insertions(+), 45 deletions(-) > > -- > Matt Fleming, Intel Open Source Technology Center Pulled, thanks Matt! I'll give it some testing and then send it to Linus with other x86 fixes. Thanks, Ingo From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Subject: Re: [GIT PULL] EFI urgent fixes Date: Thu, 25 Sep 2014 16:41:27 +0200 Message-ID: <20140925144127.GA3828@gmail.com> References: <20140925073133.GQ18635@console-pimps.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20140925073133.GQ18635-HNK1S37rvNbeXh+fF434Mdi2O/JbrIOy@public.gmane.org> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Matt Fleming , Linus Torvalds Cc: "H. Peter Anvin" , Thomas Gleixner , linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-efi@vger.kernel.org * Matt Fleming wrote: > Folks, > > Please consider pulling the following fixes. > > The first two are releated to the bug report I got from Linus and Josh > Boyer, whereby Fedora20 + grub2 systems were no longer booting. Linus > reverted the offending commit that broke grub2 boot, but that left us in > a state where Macbooks still didn't boot with EFI. This is fixed with > the below revert and I'm dropping a noisey boot time efi_printk() based > on Josh's request and Linus' OK. > > The third patch fixes a 32-bit EFI boot stub bug where garbled text is > displayed on the screen if any of the efi_printk() statements run. > > The following changes since commit 3eddc69ffeba092d288c386646bfa5ec0fce25fd: > > x86 early_ioremap: Increase FIX_BTMAPS_SLOTS to 8 (2014-09-14 15:24:31 +0100) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git tags/efi-urgent > > for you to fetch changes up to 115c6628a59044958c205f8468a1b3ba3d539e61: > > x86/efi: Truncate 64-bit values when calling 32-bit OutputString() (2014-09-24 21:56:46 +0100) > > ---------------------------------------------------------------- > * Revert the static library changes from the merge window since they're > causing issues for Macbooks and Fedora + Grub2 - Matt Fleming > > * Delete the misleading "setup_efi_pci() failed!" message which some > people are seeing when booting EFI - Matt Fleming > > * Fix printing strings from the 32-bit EFI boot stub by only passing > 32-bit addresses to the firmware - Matt Fleming > > ---------------------------------------------------------------- > Matt Fleming (3): > Revert "efi/x86: efistub: Move shared dependencies to " > x86/efi: Delete misleading efi_printk() error message > x86/efi: Truncate 64-bit values when calling 32-bit OutputString() > > arch/x86/boot/compressed/Makefile | 3 +-- > arch/x86/boot/compressed/eboot.c | 44 +++++++++++++++++++++++---------------- > arch/x86/boot/compressed/eboot.h | 16 ++++++++++++++ > arch/x86/include/asm/efi.h | 24 --------------------- > drivers/firmware/efi/Makefile | 2 +- > 5 files changed, 44 insertions(+), 45 deletions(-) > > -- > Matt Fleming, Intel Open Source Technology Center Pulled, thanks Matt! I'll give it some testing and then send it to Linus with other x86 fixes. Thanks, Ingo