[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/4] hw/arm/boot: enable DTB support when boo
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/4] hw/arm/boot: enable DTB support when booting ELF images |
Date: |
Wed, 10 Sep 2014 12:35:02 +0100 |
On 10 September 2014 12:28, Ard Biesheuvel <address@hidden> wrote:
> On 10 September 2014 13:21, Peter Maydell <address@hidden> wrote:
>> The conditional means we won't try to load the DTB even if the
>> ELF file fit into the address space entirely below loader_start,
>> which doesn't look right.
>>
>
> Ah right, I though loader_start was the start of usable RAM
It is, but the ELF file doesn't necessarily have to be in RAM:
it could be linked so as to load at an address in the flash...
> What about assigning elf_high_addr as well, and changing the test to
>
> if ((elf_low_addr > info->loader_start
> || elf_high_addr < info_loader_start)
> && load_dtb(info->loader_start, info,
> (elf_high_addr < info_loader_start) ? 0 : elf_low_addr) < 0) {
I think that's right, though maybe there's a less confusing
way of phrasing it.
-- PMM