linux-kernel - Re: [GIT PULL] EFI urgent fixes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 26 Sep 2014 13:27:34 +0200
From:	Paul Bolle <pebolle@...cali.nl>
To:	Matt Fleming <matt@...sole-pimps.org>,
	Ingo Molnar <mingo@...nel.org>
Cc:	Linus Torvalds <torvalds@...ux-foundation.org>,
	"H. Peter Anvin" <hpa@...or.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	linux-efi@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [GIT PULL] EFI urgent fixes

On Thu, 2014-09-25 at 16:41 +0200, Ingo Molnar wrote:
> * Matt Fleming <matt@...sole-pimps.org> 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 <asm/efi.h>"
> >       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 <asm/efi.h>""),
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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/