[Xen-devel] Re: [PATCH v2] qemu_ram_ptr_length: take ram_addr_t as arguments

[Peter Maydell <peter.maydell@xxxxxxxxxx>]

On 27 June 2011 18:26,  <stefano.stabellini@xxxxxxxxxxxxx> wrote:
> From: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
>
> qemu_ram_ptr_length should take ram_addr_t as argument rather than
> target_phys_addr_t because is doing comparisons with RAMBlock addresses.
>
> cpu_physical_memory_map should create a ram_addr_t address to pass to
> qemu_ram_ptr_length from PhysPageDesc phys_offset.
>
> Remove code after abort() in qemu_ram_ptr_length.

Reviewed-by: Peter Maydell <peter.maydell@xxxxxxxxxx>

Tested and confirmed this fixes vexpress and looks good to me -- thanks.

-- PMM

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel