[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] use an uint64_t for the max_sz parameter in
From: |
Mark Langsdorf |
Subject: |
Re: [Qemu-devel] [PATCH v3] use an uint64_t for the max_sz parameter in load_image_targphys |
Date: |
Mon, 12 Mar 2012 12:23:46 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.2) Gecko/20120216 Thunderbird/10.0.2 |
On 03/12/2012 12:13 PM, Peter Maydell wrote:
> On 12 March 2012 16:47, Andreas Färber <address@hidden> wrote:
>> A very non-intrusive solution for allowing large theoretical limits.
>> I've skimmed through the callers and it looks fine.
>>
>> Reviewed-by: Andreas Färber <address@hidden>
>>
>> Independently David's off_t patch or a variation thereof would still be
>> needed to make use of the enlarged limit, wouldn't it? Image sizes
>> remain int here.
>
> Yes. Also the arm_boot.c code is still passing things around in
> int variables, which should be updated to use uint64_t. (The
> arm_boot device tree code needs updating too, to check whether
> the dtb is using 32 bit or 64 bit cell sizes for its RAM size).
> I'm happy to put together a patch to do this at some point if
> Mark doesn't already have one lined up.
I don't have one lined up for that.
--Mark Langsdorf
Calxeda, Inc.
- Re: [Qemu-devel] [PATCH v2] use an unsigned long for the max_sz parameter in load_image_targphys, (continued)
[Qemu-devel] [PATCH v3] use an uint64_t for the max_sz parameter in load_image_targphys, Mark Langsdorf, 2012/03/12
Re: [Qemu-devel] [PATCH v3] use an uint64_t for the max_sz parameter in load_image_targphys, Alexander Graf, 2012/03/12