[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH 23/25] PPC: e500: dt: use target_phys_addr_t for r
From: |
Scott Wood |
Subject: |
Re: [Qemu-ppc] [PATCH 23/25] PPC: e500: dt: use target_phys_addr_t for ramsize |
Date: |
Thu, 31 May 2012 17:24:17 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 |
On 05/31/2012 05:18 PM, Alexander Graf wrote:
>
> On 01.06.2012, at 00:07, Scott Wood wrote:
>
>> On 05/30/2012 06:00 AM, Alexander Graf wrote:
>>> We're passing the ram size as uint32_t, capping it to 32 bits atm.
>>> Change to target_phys_addr_t (uint64_t) to make sure we have all
>>> the bits.
>>
>> Wouldn't ram_addr_t be more appropriate?
>
> I never quite grasped the difference, but wasn't ram_addr_t something for the
> host?
I don't fully understand QEMU's RAM handling, but from what I recall RAM
is treated differently from arbitrary guest physical addresses, with a
QEMU-internal contiguous address space. Guest RAM needs to be mappable
by QEMU as well as the target, so ram_addr_t is 32-bit on a 32-bit host,
even if target_phys_addr_t is different.
But again, it was a while ago that I looked at this, and I didn't fully
understand it then, so I may be missing or misremembering something.
-Scott
- Re: [Qemu-ppc] [PATCH 15/25] PPC: e500: dt: create /soc8544 node dynamically, (continued)
- [Qemu-ppc] [PATCH 20/25] PPC: e500: dt: start with empty device tree, Alexander Graf, 2012/05/30
- [Qemu-ppc] [PATCH 07/25] dt: add helper for phandle allocation, Alexander Graf, 2012/05/30
- [Qemu-ppc] [PATCH 11/25] PPC: e500: dt: create /cpus node dynamically, Alexander Graf, 2012/05/30
- [Qemu-ppc] [PATCH 18/25] PPC: e500: dt: create global-utils node dynamically, Alexander Graf, 2012/05/30
- [Qemu-ppc] [PATCH 23/25] PPC: e500: dt: use target_phys_addr_t for ramsize, Alexander Graf, 2012/05/30
[Qemu-ppc] [PATCH 16/25] PPC: e500: dt: create serial nodes dynamically, Alexander Graf, 2012/05/30
[Qemu-ppc] [PATCH 06/25] dt: add helper for empty dt creation, Alexander Graf, 2012/05/30