Re: [Qemu-devel] [PATCH 02/40] memory: allow memory_region_find() to run on non-root memory regions

[Peter Maydell]

On 7 May 2013 15:16, Paolo Bonzini <address@hidden> wrote:
> memory_region_find() is similar to registering a MemoryListener and
> checking for the MemoryRegionSections that come from a particular
> region.  There is no reason for this to be limited to a root memory
> region.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
>  include/exec/memory.h |   13 +++++++------
>  memory.c              |   20 +++++++++++++++-----
>  2 files changed, 22 insertions(+), 11 deletions(-)
>
> diff --git a/include/exec/memory.h b/include/exec/memory.h
> index 9e88320..efe210b 100644
> --- a/include/exec/memory.h
> +++ b/include/exec/memory.h
> @@ -725,17 +725,18 @@ void memory_region_set_alias_offset(MemoryRegion *mr,
>   *
>   * Returns a #MemoryRegionSection that describes a contiguous overlap.
>   * It will have the following characteristics:
> - *    address@hidden >= @addr
> - *    address@hidden + address@hidden <= @addr + @size
>   *    address@hidden = 0 iff no overlap was found
>   *    address@hidden is non-%NULL iff an overlap was found
>   *
> - * @address_space: a top-level (i.e. parentless) region that contains
> - *       the region to be found
> - * @addr: start of the area within @address_space to be searched
> + * If @mr is parent-less,
> + *    address@hidden >= @addr
> + *    address@hidden + address@hidden <= @addr + @size

Shouldn't we also say something about what you get back
in the "@mr is not parentless" case?

> + *
> + * @mr: a (possibly indirect) parent that contains the region to be found
> + * @addr: start of the area within @as to be searched

There's no @as parameter...

>   * @size: size of the area to be searched
>   */
> -MemoryRegionSection memory_region_find(MemoryRegion *address_space,
> +MemoryRegionSection memory_region_find(MemoryRegion *mr,
>                                         hwaddr addr, uint64_t size);

thanks
-- PMM