From: Haozhong Zhang <haozhong.zhang@intel.com>
To: qemu-devel@nongnu.org
Cc: Eduardo Habkost <ehabkost@redhat.com>,
Igor Mammedov <imammedo@redhat.com>,
Paolo Bonzini <pbonzini@redhat.com>,
mst@redhat.com, Xiao Guangrong <xiaoguangrong.eric@gmail.com>,
Juan Quintela <quintela@redhat.com>,
dgilbert@redhat.com, Stefan Hajnoczi <stefanha@redhat.com>,
Dan Williams <dan.j.williams@intel.com>,
Haozhong Zhang <haozhong.zhang@intel.com>
Subject: [Qemu-devel] [PATCH v3 0/8] nvdimm: guarantee persistence of QEMU writes to persistent memory
Date: Fri, 16 Feb 2018 16:46:07 +0800 [thread overview]
Message-ID: <20180216084615.26483-1-haozhong.zhang@intel.com> (raw)
QEMU writes to vNVDIMM backends in the vNVDIMM label emulation and
live migration. If the backend is on the persistent memory, QEMU needs
to take proper operations to ensure its writes persistent on the
persistent memory. Otherwise, a host power failure may result in the
loss the guest data on the persistent memory.
This v3 patch series is based on Marcel's patch "mem: add share
parameter to memory-backend-ram" [1] because of the changes in patch 1.
[1] https://lists.gnu.org/archive/html/qemu-devel/2018-02/msg03858.html
Previous versions can be found at
v2: https://lists.gnu.org/archive/html/qemu-devel/2018-02/msg01579.html
v1: https://lists.gnu.org/archive/html/qemu-devel/2017-12/msg05040.html
Changes in v3:
* (Patch 5) Add a is_pmem flag to ram_handle_compressed() and handle
PMEM writes in it, so we don't need the _common function.
* (Patch 6) Expose qemu_get_buffer_common so we can remove the
unnecessary qemu_get_buffer_to_pmem wrapper.
* (Patch 8) Add a is_pmem flag to xbzrle_decode_buffer() and handle
PMEM writes in it, so we can remove the unnecessary
xbzrle_decode_buffer_{common, to_pmem}.
* Move libpmem stubs to stubs/pmem.c and fix the compilation failures
of test-{xbzrle,vmstate}.c.
Changes in v2:
* (Patch 1) Use a flags parameter in file ram allocation functions.
* (Patch 2) Add a new option 'pmem' to hostmem-file.
* (Patch 3) Use libpmem to operate on the persistent memory, rather
than re-implementing those operations in QEMU.
* (Patch 5-8) Consider the write persistence in the migration path.
Haozhong Zhang (8):
[1/8] memory, exec: switch file ram allocation functions to 'flags' parameters
[2/8] hostmem-file: add the 'pmem' option
[3/8] configure: add libpmem support
[4/8] mem/nvdimm: ensure write persistence to PMEM in label emulation
[5/8] migration/ram: ensure write persistence on loading zero pages to PMEM
[6/8] migration/ram: ensure write persistence on loading normal pages to PMEM
[7/8] migration/ram: ensure write persistence on loading compressed pages to PMEM
[8/8] migration/ram: ensure write persistence on loading xbzrle pages to PMEM
backends/hostmem-file.c | 27 +++++++++++++++++++-
configure | 35 ++++++++++++++++++++++++++
docs/nvdimm.txt | 14 +++++++++++
exec.c | 23 ++++++++++++++---
hw/mem/nvdimm.c | 9 ++++++-
include/exec/memory.h | 12 +++++++--
include/exec/ram_addr.h | 28 +++++++++++++++++++--
include/migration/qemu-file-types.h | 2 ++
include/qemu/pmem.h | 27 ++++++++++++++++++++
memory.c | 8 +++---
migration/qemu-file.c | 29 ++++++++++++++--------
migration/ram.c | 49 +++++++++++++++++++++++++++----------
migration/ram.h | 2 +-
migration/rdma.c | 2 +-
migration/xbzrle.c | 8 ++++--
migration/xbzrle.h | 3 ++-
numa.c | 2 +-
qemu-options.hx | 9 ++++++-
stubs/Makefile.objs | 1 +
stubs/pmem.c | 37 ++++++++++++++++++++++++++++
tests/Makefile.include | 4 +--
tests/test-xbzrle.c | 4 +--
22 files changed, 288 insertions(+), 47 deletions(-)
create mode 100644 include/qemu/pmem.h
create mode 100644 stubs/pmem.c
--
2.16.1
next reply other threads:[~2018-02-16 8:47 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-16 8:46 Haozhong Zhang [this message]
2018-02-16 8:46 ` [Qemu-devel] [PATCH v3 1/8] memory, exec: switch file ram allocation functions to 'flags' parameters Haozhong Zhang
2018-02-16 8:46 ` [Qemu-devel] [PATCH v3 2/8] hostmem-file: add the 'pmem' option Haozhong Zhang
2018-02-16 8:46 ` [Qemu-devel] [PATCH v3 3/8] configure: add libpmem support Haozhong Zhang
2018-02-16 8:46 ` [Qemu-devel] [PATCH v3 4/8] mem/nvdimm: ensure write persistence to PMEM in label emulation Haozhong Zhang
2018-02-16 8:46 ` [Qemu-devel] [PATCH v3 5/8] migration/ram: ensure write persistence on loading zero pages to PMEM Haozhong Zhang
2018-02-16 8:46 ` [Qemu-devel] [PATCH v3 6/8] migration/ram: ensure write persistence on loading normal " Haozhong Zhang
2018-02-16 8:46 ` [Qemu-devel] [PATCH v3 7/8] migration/ram: ensure write persistence on loading compressed " Haozhong Zhang
2018-02-16 8:46 ` [Qemu-devel] [PATCH v3 8/8] migration/ram: ensure write persistence on loading xbzrle " Haozhong Zhang
2018-02-23 9:25 ` [Qemu-devel] [PATCH v3 0/8] nvdimm: guarantee persistence of QEMU writes to persistent memory no-reply
2018-02-24 13:59 ` no-reply
2018-02-25 8:52 ` no-reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180216084615.26483-1-haozhong.zhang@intel.com \
--to=haozhong.zhang@intel.com \
--cc=dan.j.williams@intel.com \
--cc=dgilbert@redhat.com \
--cc=ehabkost@redhat.com \
--cc=imammedo@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
--cc=stefanha@redhat.com \
--cc=xiaoguangrong.eric@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).