[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/6] memory: Implement memory_region_get_ram_add
From: |
Gonglei (Arei) |
Subject: |
Re: [Qemu-devel] [PATCH 3/6] memory: Implement memory_region_get_ram_addr with mr->ram_block |
Date: |
Tue, 1 Mar 2016 02:32:37 +0000 |
> Subject: [Qemu-devel] [PATCH 3/6] memory: Implement
> memory_region_get_ram_addr with mr->ram_block
>
> Signed-off-by: Fam Zheng <address@hidden>
> ---
> include/exec/memory.h | 8 +-------
> memory.c | 5 +++++
> 2 files changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/include/exec/memory.h b/include/exec/memory.h
> index d5284c2..810d2c0 100644
> --- a/include/exec/memory.h
> +++ b/include/exec/memory.h
> @@ -978,14 +978,8 @@ void
> memory_region_add_subregion_overlap(MemoryRegion *mr,
> /**
> * memory_region_get_ram_addr: Get the ram address associated with a
> memory
> * region
> - *
> - * DO NOT USE THIS FUNCTION. This is a temporary workaround while the
> Xen
> - * code is being reworked.
> */
> -static inline ram_addr_t memory_region_get_ram_addr(MemoryRegion *mr)
> -{
> - return mr->ram_addr;
> -}
> +ram_addr_t memory_region_get_ram_addr(MemoryRegion *mr);
>
> uint64_t memory_region_get_alignment(const MemoryRegion *mr);
> /**
> diff --git a/memory.c b/memory.c
> index fe70075..769825e 100644
> --- a/memory.c
> +++ b/memory.c
> @@ -1596,6 +1596,11 @@ void
> *memory_region_get_ram_ptr(MemoryRegion *mr)
> return ptr + offset;
> }
>
> +ram_addr_t memory_region_get_ram_addr(MemoryRegion *mr)
> +{
> + return mr->ram_block->offset;
> +}
> +
Do we need add NULL check for mr->ram_block ?
Regards,
-Gonglei
> void memory_region_ram_resize(MemoryRegion *mr, ram_addr_t newsize,
> Error **errp)
> {
> assert(mr->ram_addr != RAM_ADDR_INVALID);
> --
> 2.4.3
>
- [Qemu-devel] [PATCH 0/6] memory: Clean up MemoryRegion.ram_addr and optimize address_space_translate, Fam Zheng, 2016/02/28
- [Qemu-devel] [PATCH 1/6] exec: Return RAMBlock pointer from allocating functions, Fam Zheng, 2016/02/28
- [Qemu-devel] [PATCH 2/6] memory: Move assignment to ram_block to memory_region_init_*, Fam Zheng, 2016/02/28
- [Qemu-devel] [PATCH 3/6] memory: Implement memory_region_get_ram_addr with mr->ram_block, Fam Zheng, 2016/02/28
- Re: [Qemu-devel] [PATCH 3/6] memory: Implement memory_region_get_ram_addr with mr->ram_block,
Gonglei (Arei) <=
- [Qemu-devel] [PATCH 4/6] memory: Drop MemoryRegion.ram_addr, Fam Zheng, 2016/02/28
- [Qemu-devel] [PATCH 5/6] exec: Pass RAMBlock pointer to qemu_ram_free, Fam Zheng, 2016/02/28
- [Qemu-devel] [PATCH 6/6] exec: Introduce AddressSpaceDispatch.mru_section, Fam Zheng, 2016/02/28