[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() rout
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine |
Date: |
Fri, 23 Sep 2016 09:17:58 +0100 |
On 23 September 2016 at 08:19, Cédric Le Goater <address@hidden> wrote:
> But the goal is to boot from the device, so I added a memory region alias
> at 0 to trigger the flash module mmios at boot time, as this is where
> u-boot expects to be.
>
> and I fell in this trap :/
>
> aspeed_smc_flash_read: To 0x0 of size 1: 0xbe mode:0
> Bad ram pointer (nil)
> Aborted (core dumped)
>
> There is a failure in get_page_addr_code(), possibly because qemu uses
> byte per byte reads of the code (cpu_ldub_code). But this is beyond my
> understanding of qemu's internal.
This is a bug in how we report the problem, but the underlying
issue here is attempting to execute from something that's not RAM
or ROM. You can't execute code out of something backed by MMIO.
thanks
-- PMM
- Re: [Qemu-devel] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine, Cédric Le Goater, 2016/09/23
- Re: [Qemu-devel] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine, Cédric Le Goater, 2016/09/23
- Re: [Qemu-devel] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine, mar.krzeminski, 2016/09/23
- Re: [Qemu-devel] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine, Cédric Le Goater, 2016/09/24
- Re: [Qemu-devel] [Qemu-arm] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine, Edgar E. Iglesias, 2016/09/24
- Re: [Qemu-devel] [Qemu-arm] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine, KONRAD Frederic, 2016/09/26
- Re: [Qemu-devel] [Qemu-arm] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine, Cédric Le Goater, 2016/09/26
- Re: [Qemu-devel] [Qemu-arm] [PATCH 4/7] m25p80: add a m25p80_set_rom_storage() routine, mar.krzeminski, 2016/09/26