[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] hw/core/generic-loader: Fix crash when running

From: Peter Maydell
Subject: Re: [Qemu-devel] [PATCH] hw/core/generic-loader: Fix crash when running without CPU
Date: Thu, 26 Jan 2017 23:06:48 +0000

On 26 January 2017 at 05:50, Thomas Huth <address@hidden> wrote:
> I think there is no real use case yet. But this fix is 1) simpler than
> doing an error_report() + exit() here, and 2) maybe the vision of
> constructing machines on the fly with QEMU will eventually come true one
> day in the distant future, so with that patch here, the code would
> already be prepared for the case when QEMU gets started without CPUs and
> the CPUs are then later added via QOM...

For the latter to work you'd need to do something more anyway,
because you still need to be able to say "load this image
into this CPU's address space", even if the CPUs are getting
added later via QOM. You'd need to say "delay the image
load until we actually have a CPU to load it for" or
something... Passing a NULL address space to load_elf_as()
etc is saying "I am definitely a board which has
system_address_space as its one and only address space",
which is OK if you're a board model but a bit more dubious
if you're a generic device like this one.

-- PMM

reply via email to

[Prev in Thread] Current Thread [Next in Thread]