qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v7 0/5] don't use NVDIMM for balooning


From: Denis V. Lunev
Subject: Re: [Qemu-devel] [PATCH v7 0/5] don't use NVDIMM for balooning
Date: Mon, 22 Feb 2016 12:16:08 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1

On 02/10/2016 11:49 AM, Vladimir Sementsov-Ogievskiy wrote:
v7:
   02: Reviewed-by: Igor Mammedov
   04: object instead of dimm
   05: arror_abort instead of NULL for getting size property,
       remove superfluous include

v6:
  add stubbed pc_dimm_build_list, fix compilation for
  !CONFIG_MEM_HOTPLUG targets - thx to Cornelia.

v5: do not use qapi
  0002-0004: new patches
  0005: white list instead of black list

v4:
  0001: Reviewed-by: Eric Blake <address@hidden>
  second patch is splitted to 0002 and 0003
  0002: Add 'type' field instead of 'balloonable' to PCDIMMDeviceInfo
  0003: chec 'type' instead of 'balloonable'

v3:
     - do not use additional class variable

NVDIMM for now is planned to use as a backing store for DAX filesystem
in the guest and thus this memory is excluded from guest memory
management and LRUs.

In this case libvirt running QEMU along with configured balloon almost
immediately inflates balloon and effectively kill the guest as
qemu counts nvdimm as part of the ram.


Vladimir Sementsov-Ogievskiy (5):
   move get_current_ram_size to virtio-balloon.c
   pc-dimm: rename pc_dimm_built_list()
   pc-dimm: add pc_dimm_build_list()
   virtio-balloon: rewrite get_current_ram_size()
   balloon: Use only 'pc-dimm' type dimm for ballooning

  hw/mem/pc-dimm.c                | 47 ++++++++++++++++-------------------------
  hw/virtio/virtio-balloon.c      | 18 ++++++++++++++++
  include/exec/cpu-common.h       |  1 -
  include/hw/mem/pc-dimm.h        |  3 +++
  stubs/Makefile.objs             |  2 +-
  stubs/pc_dimm.c                 | 12 +++++++++++
  stubs/qmp_pc_dimm_device_list.c | 12 -----------
  7 files changed, 52 insertions(+), 43 deletions(-)
  create mode 100644 stubs/pc_dimm.c
  delete mode 100644 stubs/qmp_pc_dimm_device_list.c

guys?

ping v2



reply via email to

[Prev in Thread] Current Thread [Next in Thread]