From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754670AbaIZL1j (ORCPT ); Fri, 26 Sep 2014 07:27:39 -0400 Received: from cpsmtpb-ews03.kpnxchange.com ([213.75.39.6]:55499 "EHLO cpsmtpb-ews03.kpnxchange.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753609AbaIZL1h (ORCPT ); Fri, 26 Sep 2014 07:27:37 -0400 Message-ID: <1411730854.7866.10.camel@x220> Subject: Re: [GIT PULL] EFI urgent fixes From: Paul Bolle To: Matt Fleming , Ingo Molnar Cc: Linus Torvalds , "H. Peter Anvin" , Thomas Gleixner , linux-efi@vger.kernel.org, linux-kernel@vger.kernel.org Date: Fri, 26 Sep 2014 13:27:34 +0200 In-Reply-To: <20140925144127.GA3828@gmail.com> References: <20140925073133.GQ18635@console-pimps.org> <20140925144127.GA3828@gmail.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4 (3.10.4-3.fc20) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 26 Sep 2014 11:27:35.0146 (UTC) FILETIME=[DEA374A0:01CFD97C] X-RcptDomain: vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2014-09-25 at 16:41 +0200, Ingo Molnar wrote: > * 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 +- This Makefile was changed in the first patch. That became 84be880560fb ("Revert "efi/x86: efistub: Move shared dependencies to ""), which just landed in next-20140926. It appears to have introduced a typo, because: CONFIG_EFI_ARM_STUB should probably have been: CONFIG_EFI_ARMSTUB > > 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. Paul Bolle From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Bolle Subject: Re: [GIT PULL] EFI urgent fixes Date: Fri, 26 Sep 2014 13:27:34 +0200 Message-ID: <1411730854.7866.10.camel@x220> References: <20140925073133.GQ18635@console-pimps.org> <20140925144127.GA3828@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140925144127.GA3828-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Matt Fleming , Ingo Molnar Cc: Linus Torvalds , "H. Peter Anvin" , Thomas Gleixner , linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-efi@vger.kernel.org On Thu, 2014-09-25 at 16:41 +0200, Ingo Molnar wrote: > * 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 +- This Makefile was changed in the first patch. That became 84be880560fb ("Revert "efi/x86: efistub: Move shared dependencies to ""), which just landed in next-20140926. It appears to have introduced a typo, because: CONFIG_EFI_ARM_STUB should probably have been: CONFIG_EFI_ARMSTUB > > 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. Paul Bolle