From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42117) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aVYcm-0000Gs-4r for qemu-devel@nongnu.org; Tue, 16 Feb 2016 00:56:37 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aVYci-00004r-Uf for qemu-devel@nongnu.org; Tue, 16 Feb 2016 00:56:36 -0500 Received: from mx2.parallels.com ([199.115.105.18]:36705) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aVYci-0008Vh-OR for qemu-devel@nongnu.org; Tue, 16 Feb 2016 00:56:32 -0500 References: <1455094166-7381-1-git-send-email-vsementsov@virtuozzo.com> From: "Denis V. Lunev" Message-ID: <56C2B9F6.9000303@openvz.org> Date: Tue, 16 Feb 2016 08:56:06 +0300 MIME-Version: 1.0 In-Reply-To: <1455094166-7381-1-git-send-email-vsementsov@virtuozzo.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v7 0/5] don't use NVDIMM for balooning List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Vladimir Sementsov-Ogievskiy , qemu-devel@nongnu.org Cc: guangrong.xiao@linux.intel.com, mst@redhat.com, armbru@redhat.com, lcapitulino@redhat.com, stefanha@redhat.com, cornelia.huck@de.ibm.com, imammedo@redhat.com 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 > 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 > ping