[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH master, stable-0.12] Don't load options roms int
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH master, stable-0.12] Don't load options roms intended to be loaded by the bios in qemu |
Date: |
Tue, 22 Dec 2009 10:35:54 +0100 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
On Tue, Dec 22, 2009 at 11:25:13AM +0200, Avi Kivity wrote:
> The first such option rom will load at address 0, which isn't very nice,
> and the second will report a conflict and abort, which is horrible.
>
> Signed-off-by: Avi Kivity <address@hidden>
> ---
> hw/loader.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/hw/loader.c b/hw/loader.c
> index 2ceb8eb..c6bf0f1 100644
> --- a/hw/loader.c
> +++ b/hw/loader.c
> @@ -654,6 +654,9 @@ int rom_load_all(void)
> Rom *rom;
>
> QTAILQ_FOREACH(rom, &roms, next) {
> + if (!rom->addr) {
> + continue;
> + }
Some roms needs to be loaded at address 0, it's the case for example of
the arm versatile bootloader. Adding this code will break the board
emulation.
> if (addr > rom->addr) {
> fprintf(stderr, "rom: requested regions overlap "
> "(rom %s. free=0x" TARGET_FMT_plx
> --
> 1.6.5.3
>
>
>
>
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net