Re: [GIT PULL] x86 fixes

Kees Cook
2014-11-17T21:02:32Z

On Mon, Nov 17, 2014 at 5:58 AM, Ingo Molnar <mingo@kernel.org> wrote:
>
> * Markus Trippelsdorf <markus@trippelsdorf.de> wrote:
>
>> On 2014.11.17 at 08:42 +0100, Markus Trippelsdorf wrote:
>> > On 2014.11.16 at 10:07 +0100, Ingo Molnar wrote:
>> > >
>> > > Junjie Mao (1):
>> > > x86, kaslr: Prevent .bss from overlaping initrd
>> >
>> > This breaks the build for me:
>> > ...
>> > OBJCOPY arch/x86/boot/compressed/vmlinux.bin
>> > HOSTCC arch/x86/boot/compressed/mkpiggy
>> > CC arch/x86/boot/compressed/cpuflags.o
>> > CC arch/x86/boot/version.o
>> > CC arch/x86/boot/video-vga.o
>> > CC arch/x86/boot/video-vesa.o
>> > CC arch/x86/boot/video-bios.o
>> > LZ4 arch/x86/boot/compressed/vmlinux.bin.lz4
>> > HOSTCC arch/x86/boot/tools/build
>> > CPUSTR arch/x86/boot/cpustr.h
>> > CC arch/x86/boot/cpu.o
>> > .bss and .brk lack common file offset
>> > .bss and .brk lack common file offset
>> > .bss and .brk lack common file offset
>> > .bss and .brk lack common file offset
>> > MKPIGGY arch/x86/boot/compressed/piggy.S
>> > Usage: arch/x86/boot/compressed/mkpiggy compressed_file run_size
>> > arch/x86/boot/compressed/Makefile:86: recipe for target 'arch/x86/boot/compressed/piggy.S' failed
>
> Oops ...
>
>> from my config:
>> # CONFIG_BLK_DEV_INITRD is not set
>>
>> So only running calc_run_size.pl when CONFIG_BLK_DEV_INITRD is set would
>> probably fix the issue.
>
> I've Cc:-ed Junjie Mao and Kees Cook as well.

Eek, well, the warning worked, at least (".bss and .brk lack common
file offset") since that was kind of an assumption in the script I
wanted to make sure we'd catch if it wasn't true.

Can you send your full .config? I built without CONFIG_BLK_DEV_INITRD,
and it didn't fail, so something else must be tickling this...

Sorry for the glitch!

-Kees

>
> Thanks,
>
> Ingo

--
Kees Cook
Chrome OS Security