[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/12] qemu_ram_ptr_length: take ram_addr_t as a
From: |
Vasily Khoruzhick |
Subject: |
Re: [Qemu-devel] [PATCH 08/12] qemu_ram_ptr_length: take ram_addr_t as arguments |
Date: |
Wed, 6 Jul 2011 16:12:31 +0300 |
User-agent: |
KMail/1.13.7 (Linux/2.6.39-gentoo-anarsoul; KDE/4.6.3; i686; ; ) |
On Tuesday 05 July 2011 19:51:09 Alexander Graf wrote:
> From: Stefano Stabellini <address@hidden>
>
> 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.
>
> Changes in v2:
>
> - handle 0 size in qemu_ram_ptr_length;
>
> - rename addr1 to raddr;
>
> - initialize raddr to ULONG_MAX.
>
> Signed-off-by: Stefano Stabellini <address@hidden>
> Reviewed-by: Peter Maydell <address@hidden>
> Signed-off-by: Alexander Graf <address@hidden>
Tested-by: Vasily Khoruzhick <address@hidden>
> ---
> cpu-common.h | 2 +-
> exec.c | 21 ++++++++++++++-------
> 2 files changed, 15 insertions(+), 8 deletions(-)
>
> diff --git a/cpu-common.h b/cpu-common.h
> index b027e43..e4fcded 100644
> --- a/cpu-common.h
> +++ b/cpu-common.h
> @@ -65,7 +65,7 @@ void qemu_ram_free_from_ptr(ram_addr_t addr);
> void qemu_ram_remap(ram_addr_t addr, ram_addr_t length);
> /* This should only be used for ram local to a device. */
> void *qemu_get_ram_ptr(ram_addr_t addr);
> -void *qemu_ram_ptr_length(target_phys_addr_t addr, target_phys_addr_t
> *size); +void *qemu_ram_ptr_length(ram_addr_t addr, ram_addr_t *size);
> /* Same but slower, to use for migration, where the order of
> * RAMBlocks must not change. */
> void *qemu_safe_ram_ptr(ram_addr_t addr);
> diff --git a/exec.c b/exec.c
> index 5604946..c0673c2 100644
> --- a/exec.c
> +++ b/exec.c
> @@ -3167,8 +3167,11 @@ void *qemu_safe_ram_ptr(ram_addr_t addr)
>
> /* Return a host pointer to guest's ram. Similar to qemu_get_ram_ptr
> * but takes a size argument */
> -void *qemu_ram_ptr_length(target_phys_addr_t addr, target_phys_addr_t
> *size) +void *qemu_ram_ptr_length(ram_addr_t addr, ram_addr_t *size)
> {
> + if (*size == 0) {
> + return NULL;
> + }
> if (xen_enabled()) {
> return xen_map_cache(addr, *size, 1);
> } else {
> @@ -3184,9 +3187,6 @@ void *qemu_ram_ptr_length(target_phys_addr_t addr,
> target_phys_addr_t *size)
>
> fprintf(stderr, "Bad ram offset %" PRIx64 "\n", (uint64_t)addr);
> abort();
> -
> - *size = 0;
> - return NULL;
> }
> }
>
> @@ -4052,7 +4052,9 @@ void *cpu_physical_memory_map(target_phys_addr_t
> addr, target_phys_addr_t page;
> unsigned long pd;
> PhysPageDesc *p;
> - target_phys_addr_t addr1 = addr;
> + ram_addr_t raddr = ULONG_MAX;
> + ram_addr_t rlen;
> + void *ret;
>
> while (len > 0) {
> page = addr & TARGET_PAGE_MASK;
> @@ -4080,13 +4082,18 @@ void *cpu_physical_memory_map(target_phys_addr_t
> addr, *plen = l;
> return bounce.buffer;
> }
> + if (!todo) {
> + raddr = (pd & TARGET_PAGE_MASK) + (addr & ~TARGET_PAGE_MASK);
> + }
>
> len -= l;
> addr += l;
> todo += l;
> }
> - *plen = todo;
> - return qemu_ram_ptr_length(addr1, plen);
> + rlen = todo;
> + ret = qemu_ram_ptr_length(raddr, &rlen);
> + *plen = rlen;
> + return ret;
> }
>
> /* Unmaps a memory region previously mapped by cpu_physical_memory_map().
- [Qemu-devel] [PULL 00/12] Xen patch queue 2011-07-05, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 05/12] xen_console: fix memory leak, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 09/12] xen_disk: treat "aio" as "raw", Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 08/12] qemu_ram_ptr_length: take ram_addr_t as arguments, Alexander Graf, 2011/07/05
- Re: [Qemu-devel] [PATCH 08/12] qemu_ram_ptr_length: take ram_addr_t as arguments,
Vasily Khoruzhick <=
- [Qemu-devel] [PATCH 04/12] xen: enable console and disk backend in HVM mode, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 06/12] xen: add vkbd support for PV on HVM guests, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 02/12] xen: Clean up map cache API naming, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 10/12] checkpatch: don't error out on }, { lines, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 07/12] xen_disk: cope with missing xenstore "params" node, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 01/12] xen: Clean up build system, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 11/12] xen_console: support the new extended xenstore protocol, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 03/12] xen: Fold CONFIG_XEN_MAPCACHE into CONFIG_XEN, Alexander Graf, 2011/07/05
- [Qemu-devel] [PATCH 12/12] xen_console: fall back to qemu serial device, Alexander Graf, 2011/07/05
- Re: [Qemu-devel] [PULL 00/12] Xen patch queue 2011-07-05, Anthony Liguori, 2011/07/19