All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
To: Wei Wang <wei.w.wang@intel.com>
Cc: yang.zhang.wz@gmail.com, virtio-dev@lists.oasis-open.org,
	riel@redhat.com, quan.xu0@gmail.com, kvm@vger.kernel.org,
	"Michael S. Tsirkin" <mst@redhat.com>,
	nilal@redhat.com, liliang.opensource@gmail.com,
	linux-kernel@vger.kernel.org,
	virtualization@lists.linux-foundation.org, linux-mm@kvack.org,
	pbonzini@redhat.com, akpm@linux-foundation.org,
	mhocko@kernel.org, torvalds@linux-foundation.org
Subject: Re: [PATCH v36 0/5] Virtio-balloon: support free page reporting
Date: Fri, 7 Sep 2018 13:29:56 +0100	[thread overview]
Message-ID: <20180907122955.GD2544__44821.3274278261$1536323291$gmane$org@work-vm> (raw)
In-Reply-To: <5B911B03.2060602@intel.com>

* Wei Wang (wei.w.wang@intel.com) wrote:
> On 07/23/2018 10:36 PM, Dr. David Alan Gilbert wrote:
> > * Michael S. Tsirkin (mst@redhat.com) wrote:
> > > On Fri, Jul 20, 2018 at 04:33:00PM +0800, Wei Wang wrote:
> > > > This patch series is separated from the previous "Virtio-balloon
> > > > Enhancement" series. The new feature, VIRTIO_BALLOON_F_FREE_PAGE_HINT,
> > > > implemented by this series enables the virtio-balloon driver to report
> > > > hints of guest free pages to the host. It can be used to accelerate live
> > > > migration of VMs. Here is an introduction of this usage:
> > > > 
> > > > Live migration needs to transfer the VM's memory from the source machine
> > > > to the destination round by round. For the 1st round, all the VM's memory
> > > > is transferred. From the 2nd round, only the pieces of memory that were
> > > > written by the guest (after the 1st round) are transferred. One method
> > > > that is popularly used by the hypervisor to track which part of memory is
> > > > written is to write-protect all the guest memory.
> > > > 
> > > > This feature enables the optimization by skipping the transfer of guest
> > > > free pages during VM live migration. It is not concerned that the memory
> > > > pages are used after they are given to the hypervisor as a hint of the
> > > > free pages, because they will be tracked by the hypervisor and transferred
> > > > in the subsequent round if they are used and written.
> > > > 
> > > > * Tests
> > > > - Test Environment
> > > >      Host: Intel(R) Xeon(R) CPU E5-2699 v4 @ 2.20GHz
> > > >      Guest: 8G RAM, 4 vCPU
> > > >      Migration setup: migrate_set_speed 100G, migrate_set_downtime 2 second
> > > > 
> > > > - Test Results
> > > >      - Idle Guest Live Migration Time (results are averaged over 10 runs):
> > > >          - Optimization v.s. Legacy = 409ms vs 1757ms --> ~77% reduction
> > > > 	(setting page poisoning zero and enabling ksm don't affect the
> > > >           comparison result)
> > > >      - Guest with Linux Compilation Workload (make bzImage -j4):
> > > >          - Live Migration Time (average)
> > > >            Optimization v.s. Legacy = 1407ms v.s. 2528ms --> ~44% reduction
> > > >          - Linux Compilation Time
> > > >            Optimization v.s. Legacy = 5min4s v.s. 5min12s
> > > >            --> no obvious difference
> > > I'd like to see dgilbert's take on whether this kind of gain
> > > justifies adding a PV interfaces, and what kind of guest workload
> > > is appropriate.
> > > 
> > > Cc'd.
> > Well, 44% is great ... although the measurement is a bit weird.
> > 
> > a) A 2 second downtime is very large; 300-500ms is more normal
> > b) I'm not sure what the 'average' is  - is that just between a bunch of
> > repeated migrations?
> > c) What load was running in the guest during the live migration?
> > 
> > An interesting measurement to add would be to do the same test but
> > with a VM with a lot more RAM but the same load;  you'd hope the gain
> > would be even better.
> > It would be interesting, especially because the users who are interested
> > are people creating VMs allocated with lots of extra memory (for the
> > worst case) but most of the time migrating when it's fairly idle.
> > 
> > Dave
> > 
> 
> Hi Dave,
> 
> The results of the added experiments have been shown in the v37 cover
> letter.
> Could you have a look at https://lkml.org/lkml/2018/8/27/29 . Thanks.

OK, that's much better.
The ~50% reducton with a 8G VM and a real workload is great,
and it does what you expect when you put a lot more RAM in and see the
84% reduction on a guest with 128G RAM - 54s vs ~9s is a big win!

(The migrate_set_speed is a bit high, since that's in bytes/s - but it's
not important).

That looks good,

Thanks!

Dave

> Best,
> Wei
> 
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK

  reply	other threads:[~2018-09-07 12:29 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-20  8:33 [PATCH v36 0/5] Virtio-balloon: support free page reporting Wei Wang
2018-07-20  8:33 ` [virtio-dev] " Wei Wang
2018-07-20  8:33 ` Wei Wang
2018-07-20  8:33 ` [PATCH v36 1/5] virtio-balloon: remove BUG() in init_vqs Wei Wang
2018-07-20  8:33 ` Wei Wang
2018-07-20  8:33   ` [virtio-dev] " Wei Wang
2018-07-20  8:33 ` [PATCH v36 2/5] virtio_balloon: replace oom notifier with shrinker Wei Wang
2018-07-20  8:33   ` [virtio-dev] " Wei Wang
2018-07-22 14:48   ` Michael S. Tsirkin
2018-07-22 14:48     ` [virtio-dev] " Michael S. Tsirkin
2018-07-23 10:30     ` Wei Wang
2018-07-23 10:30       ` [virtio-dev] " Wei Wang
2018-07-23 14:13       ` Michael S. Tsirkin
2018-07-23 14:13       ` Michael S. Tsirkin
2018-07-23 14:13         ` [virtio-dev] " Michael S. Tsirkin
2018-07-24  1:49         ` Wei Wang
2018-07-24  1:49         ` Wei Wang
2018-07-24  1:49           ` [virtio-dev] " Wei Wang
2018-07-23 10:30     ` Wei Wang
2018-07-22 14:48   ` Michael S. Tsirkin
2018-07-20  8:33 ` Wei Wang
2018-07-20  8:33 ` [PATCH v36 3/5] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_HINT Wei Wang
2018-07-20  8:33 ` Wei Wang
2018-07-20  8:33   ` [virtio-dev] " Wei Wang
2018-07-20  8:33 ` [PATCH v36 4/5] mm/page_poison: expose page_poisoning_enabled to kernel modules Wei Wang
2018-07-20  8:33 ` Wei Wang
2018-07-20  8:33   ` [virtio-dev] " Wei Wang
2018-07-20  8:33 ` [PATCH v36 5/5] virtio-balloon: VIRTIO_BALLOON_F_PAGE_POISON Wei Wang
2018-07-20  8:33   ` [virtio-dev] " Wei Wang
2018-07-20  8:33 ` Wei Wang
2018-07-20 12:51 ` [PATCH v36 0/5] Virtio-balloon: support free page reporting Michael S. Tsirkin
2018-07-20 12:51   ` [virtio-dev] " Michael S. Tsirkin
2018-07-22 11:11   ` Wang, Wei W
2018-07-22 11:11     ` [virtio-dev] " Wang, Wei W
2018-07-22 11:11     ` Wang, Wei W
2018-07-22 11:11   ` Wang, Wei W
2018-07-20 12:51 ` Michael S. Tsirkin
2018-07-23 14:07 ` Michael S. Tsirkin
2018-07-23 14:07 ` Michael S. Tsirkin
2018-07-23 14:07   ` [virtio-dev] " Michael S. Tsirkin
2018-07-23 14:36   ` Dr. David Alan Gilbert
2018-07-23 14:36     ` Dr. David Alan Gilbert
2018-07-24  8:12     ` Wei Wang
2018-07-24  8:12     ` Wei Wang
2018-07-24  8:12       ` [virtio-dev] " Wei Wang
2018-09-06 12:18     ` Wei Wang
2018-09-06 12:18       ` [virtio-dev] " Wei Wang
2018-09-07 12:29       ` Dr. David Alan Gilbert [this message]
2018-09-07 12:29       ` Dr. David Alan Gilbert
2018-09-08  1:46         ` Wang, Wei W
2018-09-08  1:46           ` [virtio-dev] " Wang, Wei W
2018-09-08  1:46           ` Wang, Wei W
2018-09-08  1:46           ` Wang, Wei W
2018-09-06 12:18     ` Wei Wang

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='20180907122955.GD2544__44821.3274278261$1536323291$gmane$org@work-vm' \
    --to=dgilbert@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=kvm@vger.kernel.org \
    --cc=liliang.opensource@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mhocko@kernel.org \
    --cc=mst@redhat.com \
    --cc=nilal@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=quan.xu0@gmail.com \
    --cc=riel@redhat.com \
    --cc=torvalds@linux-foundation.org \
    --cc=virtio-dev@lists.oasis-open.org \
    --cc=virtualization@lists.linux-foundation.org \
    --cc=wei.w.wang@intel.com \
    --cc=yang.zhang.wz@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 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.