From: Xiao Guangrong <guangrong.xiao@linux.intel.com>
To: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>,
qemu-devel@nongnu.org
Cc: mst@redhat.com, armbru@redhat.com, lcapitulino@redhat.com,
stefanha@redhat.com, imammedo@redhat.com,
cornelia.huck@de.ibm.com, den@openvz.org
Subject: Re: [Qemu-devel] [PATCH v7 0/5] don't use NVDIMM for balooning
Date: Tue, 23 Feb 2016 13:15:29 +0800 [thread overview]
Message-ID: <56CBEAF1.1040004@linux.intel.com> (raw)
In-Reply-To: <1455094166-7381-1-git-send-email-vsementsov@virtuozzo.com>
On 02/10/2016 04:49 PM, 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 <eblake@redhat.com>
> 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.
>
Reviewed-by: Xiao Guangrong <guangrong.xiao@linux.intel.com>
prev parent reply other threads:[~2016-02-23 5:23 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-10 8:49 [Qemu-devel] [PATCH v7 0/5] don't use NVDIMM for balooning Vladimir Sementsov-Ogievskiy
2016-02-10 8:49 ` [Qemu-devel] [PATCH 1/5] move get_current_ram_size to virtio-balloon.c Vladimir Sementsov-Ogievskiy
2016-02-10 8:49 ` [Qemu-devel] [PATCH 2/5] pc-dimm: rename pc_dimm_built_list() Vladimir Sementsov-Ogievskiy
2016-02-10 8:49 ` [Qemu-devel] [PATCH 3/5] pc-dimm: add pc_dimm_build_list() Vladimir Sementsov-Ogievskiy
2016-02-10 8:49 ` [Qemu-devel] [PATCH 4/5] virtio-balloon: rewrite get_current_ram_size() Vladimir Sementsov-Ogievskiy
2016-02-10 8:49 ` [Qemu-devel] [PATCH 5/5] balloon: Use only 'pc-dimm' type dimm for ballooning Vladimir Sementsov-Ogievskiy
2016-02-16 5:56 ` [Qemu-devel] [PATCH v7 0/5] don't use NVDIMM for balooning Denis V. Lunev
2016-02-22 9:16 ` Denis V. Lunev
2016-02-23 5:15 ` Xiao Guangrong [this message]
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=56CBEAF1.1040004@linux.intel.com \
--to=guangrong.xiao@linux.intel.com \
--cc=armbru@redhat.com \
--cc=cornelia.huck@de.ibm.com \
--cc=den@openvz.org \
--cc=imammedo@redhat.com \
--cc=lcapitulino@redhat.com \
--cc=mst@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
--cc=vsementsov@virtuozzo.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.