Commit 56eb90af authored by Junyan He's avatar Junyan He Committed by Michael S. Tsirkin
Browse files

migration/ram: ensure write persistence on loading all data to PMEM. 



Because we need to make sure the pmem kind memory data is synced
after migration, we choose to call pmem_persist() when the migration
finish. This will make sure the data of pmem is safe and will not
lose if power is off.

Signed-off-by: default avatarJunyan He <junyan.he@intel.com>
Reviewed-by: default avatarStefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: default avatarIgor Mammedov <imammedo@redhat.com>
Reviewed-by: default avatarMichael S. Tsirkin <mst@redhat.com>
Signed-off-by: default avatarMichael S. Tsirkin <mst@redhat.com>
parent 469dd51b

include/qemu/pmem.h

+6 −0
Original line number Diff line number Diff line
@@ -25,6 +25,12 @@ pmem_memcpy_persist(void *pmemdest, const void *src, size_t len) 
    return NULL;

}



static inline void

pmem_persist(const void *addr, size_t len)

{

    g_assert_not_reached();

}



#endif /* CONFIG_LIBPMEM */



#endif /* !QEMU_PMEM_H */

migration/ram.c

+8 −0
Original line number Diff line number Diff line
@@ -33,6 +33,7 @@ 
#include "qemu/bitops.h"

#include "qemu/bitmap.h"

#include "qemu/main-loop.h"

#include "qemu/pmem.h"

#include "xbzrle.h"

#include "ram.h"

#include "migration.h"
@@ -3547,6 +3548,13 @@ static int ram_load_setup(QEMUFile *f, void *opaque) 
static int ram_load_cleanup(void *opaque)

{

    RAMBlock *rb;



    RAMBLOCK_FOREACH_MIGRATABLE(rb) {

        if (ramblock_is_pmem(rb)) {

            pmem_persist(rb->host, rb->used_length);

        }

    }



    xbzrle_load_cleanup();

    compress_threads_load_cleanup();