[Qemu-devel] [PATCH] exec, Fix guest memory access.

[Qemu-devel] [PATCH] exec, Fix guest memory access.

[Anthony PERARD]

In cpu_physical_memory_rw, a change has been introduced and qemu_get_ram_ptr is
no longuer called with the ram addr we want to access, but only with the
section address. This patch fixes this. (All other call to qemu_get_ram_ptr are
already called with the right address.)

This patch fixes Xen guest.

Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>

---
 exec.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/exec.c b/exec.c
index be392e2..be08930 100644
--- a/exec.c
+++ b/exec.c
@@ -3851,8 +3851,8 @@ void cpu_physical_memory_rw(target_phys_addr_t addr, uint8_t *buf,
                 }
             } else {
                 /* RAM case */
-                ptr = qemu_get_ram_ptr(section->mr->ram_addr)
-                    + section_addr(section, addr);
+                ptr = qemu_get_ram_ptr(section->mr->ram_addr
+                                       + section_addr(section, addr));
                 memcpy(buf, ptr, l);
                 qemu_put_ram_ptr(ptr);
             }
-- 
tg: (30f51d7..) fix/memory_access (depends on: master)

Re: [Qemu-devel] [PATCH] exec, Fix guest memory access.

[Avi Kivity]

On 03/19/2012 05:54 PM, Anthony PERARD wrote:
> In cpu_physical_memory_rw, a change has been introduced and qemu_get_ram_ptr is
> no longuer called with the ram addr we want to access, but only with the
> section address. This patch fixes this. (All other call to qemu_get_ram_ptr are
> already called with the right address.)
>
> This patch fixes Xen guest.
>

Thanks, applied to memory/urgent, will push shortly.

-- 
error compiling committee.c: too many arguments to function