commit 9ecc996a3d39bdbf64a488936f97a9496b74ebd8
author Philippe Mathieu-Daudé <philmd@redhat.com> Fri May 08 08:24:54 2020 +0200
committer Stefan Hajnoczi <stefanha@redhat.com> Fri Jun 05 09:54:48 2020 +0100
tree a3726cca80e61a981dfdadec1e3ecfc557cc12ed
parent 4dfe59d187d9b218efca8d89c0c2fac1298d8712

memory: Extract memory_region_msync() from memory_region_writeback()

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-id: 20200508062456.23344-3-philmd@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>

memory.c

diff --git a/memory.c b/memory.c
index bb84334..93febe4 100644
--- a/memory.c
+++ b/memory.c
@@ -2197,6 +2197,12 @@
     qemu_ram_resize(mr->ram_block, newsize, errp);
 }
 
+void memory_region_msync(MemoryRegion *mr, hwaddr addr, hwaddr size)
+{
+    if (mr->ram_block) {
+        qemu_ram_writeback(mr->ram_block, addr, size);
+    }
+}
 
 void memory_region_writeback(MemoryRegion *mr, hwaddr addr, hwaddr size)
 {
@@ -2204,8 +2210,8 @@
      * Might be extended case needed to cover
      * different types of memory regions
      */
-    if (mr->ram_block && mr->dirty_log_mask) {
-        qemu_ram_writeback(mr->ram_block, addr, size);
+    if (mr->dirty_log_mask) {
+        memory_region_msync(mr, addr, size);
     }
 }