[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/8] memory: address_space_iterate
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/8] memory: address_space_iterate |
Date: |
Tue, 12 Dec 2017 00:50:51 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 |
On 11/12/2017 20:46, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <address@hidden>
>
> Iterate through an address space calling a function for each
> section. The iteration is done in order.
>
> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
It seems to me that you can achieve the same effect by implementing the
region_add and region_nop callbacks, and leaving out region_del. Am I
missing something?
Thanks,
Paolo
> ---
> include/exec/memory.h | 23 +++++++++++++++++++++++
> memory.c | 22 ++++++++++++++++++++++
> 2 files changed, 45 insertions(+)
>
> diff --git a/include/exec/memory.h b/include/exec/memory.h
> index 5ed4042f87..f5a9df642e 100644
> --- a/include/exec/memory.h
> +++ b/include/exec/memory.h
> @@ -1987,6 +1987,29 @@ address_space_write_cached(MemoryRegionCache *cache,
> hwaddr addr,
> address_space_write(cache->as, cache->xlat + addr,
> MEMTXATTRS_UNSPECIFIED, buf, len);
> }
>
> +/**
> + * ASIterateCallback: Function type called by address_space_iterate
> + *
> + * Return 0 on success or a negative error code.
> + *
> + * @mrs: Memory region section for this range
> + * @opaque: The opaque value passed in to the iterator.
> + */
> +typedef int (*ASIterateCallback)(MemoryRegionSection *mrs, void *opaque);
> +
> +/**
> + * address_space_iterate: Call the function for each address range in the
> + * AddressSpace, in sorted order.
> + *
> + * Return 0 on success or a negative error code.
> + *
> + * @as: Address space to iterate over
> + * @cb: Function to call. If the function returns none-0 the iteration will
> + * stop.
> + * @opaque: Value to pass to the function
> + */
> +int
> +address_space_iterate(AddressSpace *as, ASIterateCallback cb, void *opaque);
> #endif
>
> #endif
> diff --git a/memory.c b/memory.c
> index e26e5a3b1d..f45137f25e 100644
> --- a/memory.c
> +++ b/memory.c
> @@ -2810,6 +2810,28 @@ void address_space_destroy(AddressSpace *as)
> call_rcu(as, do_address_space_destroy, rcu);
> }
>
> +int address_space_iterate(AddressSpace *as, ASIterateCallback cb,
> + void *opaque)
> +{
> + int res = 0;
> + FlatView *fv = address_space_to_flatview(as);
> + FlatRange *range;
> +
> + flatview_ref(fv);
> +
> + FOR_EACH_FLAT_RANGE(range, fv) {
> + MemoryRegionSection mrs = section_from_flat_range(range, fv);
> + res = cb(&mrs, opaque);
> + if (res) {
> + break;
> + }
> + }
> +
> + flatview_unref(fv);
> +
> + return res;
> +}
> +
> static const char *memory_region_type(MemoryRegion *mr)
> {
> if (memory_region_is_ram_device(mr)) {
>
- [Qemu-devel] [PATCH v3 0/8] Rework vhost memory region updates, Dr. David Alan Gilbert (git), 2017/12/11
- [Qemu-devel] [PATCH v3 1/8] memory: address_space_iterate, Dr. David Alan Gilbert (git), 2017/12/11
- Re: [Qemu-devel] [PATCH v3 1/8] memory: address_space_iterate,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH v3 1/8] memory: address_space_iterate, Igor Mammedov, 2017/12/12
- Re: [Qemu-devel] [PATCH v3 1/8] memory: address_space_iterate, Dr. David Alan Gilbert, 2017/12/12
- Re: [Qemu-devel] [PATCH v3 1/8] memory: address_space_iterate, Paolo Bonzini, 2017/12/12
- Re: [Qemu-devel] [PATCH v3 1/8] memory: address_space_iterate, Dr. David Alan Gilbert, 2017/12/12
- Re: [Qemu-devel] [PATCH v3 1/8] memory: address_space_iterate, Paolo Bonzini, 2017/12/12
[Qemu-devel] [PATCH v3 2/8] vhost: Move log_dirty check, Dr. David Alan Gilbert (git), 2017/12/11
[Qemu-devel] [PATCH v3 3/8] vhost: Simplify ring verification checks, Dr. David Alan Gilbert (git), 2017/12/11
[Qemu-devel] [PATCH v3 4/8] vhost: New memory update functions, Dr. David Alan Gilbert (git), 2017/12/11
[Qemu-devel] [PATCH v3 5/8] vhost: update_mem_cb implementation, Dr. David Alan Gilbert (git), 2017/12/11
[Qemu-devel] [PATCH v3 7/8] vhost: Remove old vhost_set_memory etc, Dr. David Alan Gilbert (git), 2017/12/11