aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael S. Tsirkin <mst@redhat.com>2011-03-16 12:09:09 +0200
committerMichael S. Tsirkin <mst@redhat.com>2011-03-28 18:34:23 +0200
commit7b67b18adf921c77aa8443f3b4cddab87908ee1e (patch)
tree6fb4d19ff11b54df14c902d4ac2690addf9bd072
parentfbe0c5591077814eead05217fc96f087b254a6a8 (diff)
downloadqemu-7b67b18adf921c77aa8443f3b4cddab87908ee1e.zip
qemu-7b67b18adf921c77aa8443f3b4cddab87908ee1e.tar.gz
qemu-7b67b18adf921c77aa8443f3b4cddab87908ee1e.tar.bz2
vhost: fix dirty page handling
vhost was passing a physical address to cpu_physical_memory_set_dirty, which is wrong: we need to translate to ram address first. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Note: this lead to crashes during migration, so the patch is needed on the stable branch too.
-rw-r--r--hw/vhost.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/hw/vhost.c b/hw/vhost.c
index 0ca3507..14b571d 100644
--- a/hw/vhost.c
+++ b/hw/vhost.c
@@ -47,8 +47,10 @@ static void vhost_dev_sync_region(struct vhost_dev *dev,
log = __sync_fetch_and_and(from, 0);
while ((bit = sizeof(log) > sizeof(int) ?
ffsll(log) : ffs(log))) {
+ ram_addr_t ram_addr;
bit -= 1;
- cpu_physical_memory_set_dirty(addr + bit * VHOST_LOG_PAGE);
+ ram_addr = cpu_get_physical_page_desc(addr + bit * VHOST_LOG_PAGE);
+ cpu_physical_memory_set_dirty(ram_addr);
log &= ~(0x1ull << bit);
}
addr += VHOST_LOG_CHUNK;