Il 18/03/2013 04:18, mrhi...@linux.vnet.ibm.com ha scritto:
> From: "Michael R. Hines" <mrhi...@us.ibm.com>
>
> This introduces:
> 1. qemu_ram_foreach_block
> 2. qemu_ram_count_blocks
>
> Both used in communicating the RAMBlocks
> to each side for later memory registration.
>
> Signed-off-by: Michael R. Hines <mrhi...@us.ibm.com>
> ---
> exec.c | 21 +++++++++++++++++++++
> include/exec/cpu-common.h | 6 ++++++
> 2 files changed, 27 insertions(+)
>
> diff --git a/exec.c b/exec.c
> index 8a6aac3..a985da8 100644
> --- a/exec.c
> +++ b/exec.c
> @@ -2629,3 +2629,24 @@ bool cpu_physical_memory_is_io(hwaddr phys_addr)
> memory_region_is_romd(section->mr));
> }
> #endif
> +
> +void qemu_ram_foreach_block(RAMBlockIterFunc func, void *opaque)
> +{
> + RAMBlock *block;
> +
> + QTAILQ_FOREACH(block, &ram_list.blocks, next) {
> + func(block->host, block->offset, block->length, opaque);
> + }
> +}
> +
> +int qemu_ram_count_blocks(void)
> +{
> + RAMBlock *block;
> + int total = 0;
> +
> + QTAILQ_FOREACH(block, &ram_list.blocks, next) {
> + total++;
> + }
Please move this to rdma.c, and implement it using qemu_ram_foreach_block.
Otherwise looks good.
Paolo
> + return total;
> +}
> diff --git a/include/exec/cpu-common.h b/include/exec/cpu-common.h
> index 2e5f11f..aea3fe0 100644
> --- a/include/exec/cpu-common.h
> +++ b/include/exec/cpu-common.h
> @@ -119,6 +119,12 @@ extern struct MemoryRegion io_mem_rom;
> extern struct MemoryRegion io_mem_unassigned;
> extern struct MemoryRegion io_mem_notdirty;
>
> +typedef void (RAMBlockIterFunc)(void *host_addr,
> + ram_addr_t offset, ram_addr_t length, void *opaque);
> +
> +void qemu_ram_foreach_block(RAMBlockIterFunc func, void *opaque);
> +int qemu_ram_count_blocks(void);
> +
> #endif
>
> #endif /* !CPU_COMMON_H */
>
