On Fri, Oct 31, 2014 at 04:31:32PM -0700, Yinghai Lu wrote: > On Fri, Oct 31, 2014 at 1:58 PM, Kees Cook <[email protected]> wrote: > > This is useful for reporting various addresses or other values while > > debugging early boot. For example, when CONFIG_X86_VERBOSE_BOOTUP is set, > > this is now visible at boot time: > > > > early console in setup code > > early console in decompress_kernel > > input_data: 0x0000000001e1526e > > input_len: 0x0000000000732236 > > output: 0x0000000001000000 > > output_len: 0x0000000001535640 > > run_size: 0x00000000021fb000 > > KASLR using RDTSC... > > > > Signed-off-by: Kees Cook <[email protected]> > > --- > > Since this displays run_size, this patch depends on Junjie Mao's patch > > "x86, kaslr: Prevent .bss from overlaping initrd" > > > > --- > > arch/x86/boot/compressed/misc.c | 24 ++++++++++++++++++++++++ > > arch/x86/boot/compressed/misc.h | 11 +++++++++++ > > 2 files changed, 35 insertions(+) > ... > > We can reuse printf.c in arch/x86/boot. > > I had attached one in local tree for a while. or even sent it before > several years ago.
I don't think we need the full generality of printf in the decompression stub. I prefer Kees' patch, though I'd still like to see __puthex made conditional. - Josh Triplett -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
