On Wed, 09 Jul, at 10:41:34PM, Michael Brown wrote: > The PE/COFF headers currently describe only the initialised-data > portions of the image, and result in no space being allocated for the > uninitialised-data portions. Consequently, the EFI boot stub will end > up overwriting unexpected areas of memory, with unpredictable results. > > Fix by including a .bss section in the PE/COFF headers (functionally > equivalent to the init_size field in the bzImage header). > > Signed-off-by: Michael Brown <mbrown@xxxxxxxxxxxxxxxx> > --- > arch/x86/boot/header.S | 26 ++++++++++++++++++++++---- > arch/x86/boot/tools/build.c | 37 +++++++++++++++++++++++++++++-------- > 2 files changed, 51 insertions(+), 12 deletions(-) Yeah ouch, that's a particularly bad bug. Thanks Michael, this fix looks great. I've placed this in the urgent EFI queue and tagged it for stable. Thomas, the patch in question is here this one, http://git.kernel.org/cgit/linux/kernel/git/mfleming/efi.git/commit/?h=urgent&id=db0f1ff0ee1750cc52ead0ba1ddf95c47b3bd133 it would be good if you could carry it in archlinux for some additional testing. I suspect this fix may solve some of the problems people have reported in, https://bugzilla.kernel.org/show_bug.cgi?id=68761 Let me know if you'd like a backported version. -- Matt Fleming, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
