vl: move rom_load_all after machine init done
commit6b3f7f639ed8861cd034292f9bb85b00c73658a6
authorEric Auger <eric.auger@linaro.org>
Tue, 16 Jun 2015 16:07:54 +0000 (16 17:07 +0100)
committerPaolo Bonzini <pbonzini@redhat.com>
Tue, 7 Jul 2015 09:03:16 +0000 (7 11:03 +0200)
treeff302f94d2b7a969a9a1990ae7b1dcd92823c67e
parentf6e3035f75e5c6a73485335765ae070304c7a110
vl: move rom_load_all after machine init done

On ARM, commit ac9d32e39664e060cd1b538ff190980d57ad69e4 postponed the
memory preparation for boot until the machine init done notifier. This
has for consequence to insert ROM at machine init done time.

However the rom_load_all function stayed called before the ROM are
inserted. As a consequence the rom_load_all function does not do
everything it is expected to do, on ARM.

It currently registers the ROM reset notifier but does not iterate through
the registered ROM list. the isrom field is not set properly. This latter
is used to report info in the monitor and also to decide whether the
rom->data can be freed on ROM reset notifier.

To fix that regression the patch moves the rom_load_all call after
machine init done. We also take the opportunity to rename the rom_load_all
function into rom_check_and_resgister_reset() and integrate the
rom_load_done in it.

Signed-off-by: Eric Auger <eric.auger@linaro.org>
Reported-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com>
Message-Id: <1434470874-22573-1-git-send-email-eric.auger@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
hw/core/loader.c
include/hw/loader.h
vl.c