On Mon, Apr 01, 2013 at 09:21:57AM +0800, liguang wrote: > if head magic is missing or wrong unexpectedly, we'd > better to reject booting. > e.g. > I make a mistake to boot a vmlinuz for MIPS(which > I think it's for x86) like this: > qemu-system-x86_64 -kernel vmlinuz -initrd demord > then qemu report: > "qemu: linux kernel too old to load a ram disk" > that's misleading. > > Signed-off-by: liguang <lig.f...@cn.fujitsu.com> > --- > hw/i386/pc.c | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/hw/i386/pc.c b/hw/i386/pc.c > index b1e06fa..bfbb5fe 100644 > --- a/hw/i386/pc.c > +++ b/hw/i386/pc.c > @@ -683,6 +683,9 @@ static void load_linux(void *fw_cfg, > if (load_multiboot(fw_cfg, f, kernel_filename, initrd_filename, > kernel_cmdline, kernel_size, header)) { > return; > + } else { > + fprintf(stderr, "not a valid multiboot image!\n"); > + exit(1); > } > protocol = 0;
The point of protocol = 0 is to support old Linux kernels. If we exit when probing load_multiboot() fails then QEMU will no longer load old Linux kernels! I suggest just: fprintf(stderr, "warning: not a valid multiboot or modern kernel image\n"); Stefan