From: Wei Liu <wei.liu2@citrix.com>
To: Dongli Zhang <dongli.zhang@oracle.com>
Cc: sstabellini@kernel.org, wei.liu2@citrix.com,
George.Dunlap@eu.citrix.com, jinsong.liu@alibaba-inc.com,
dario.faggioli@citrix.com, tim@xen.org, xen-devel@lists.xen.org,
jbeulich@suse.com, andrew.cooper3@citrix.com,
ian.jackson@eu.citrix.com
Subject: Re: [PATCH v6 2/2] xen: move TLB-flush filtering out into populate_physmap during vm creation
Date: Wed, 21 Sep 2016 12:23:56 +0100 [thread overview]
Message-ID: <20160921112356.GF30319@citrix.com> (raw)
In-Reply-To: <70c53411-bf97-4f4e-a0e2-9becfb1b2269@default>
On Tue, Sep 20, 2016 at 06:39:32PM -0700, Dongli Zhang wrote:
> > > This patch implemented parts of TODO left in commit id
> > > a902c12ee45fc9389eb8fe54eeddaf267a555c58 (More efficient TLB-flush
> > > filtering in alloc_heap_pages()). It moved TLB-flush filtering out into
> > > populate_physmap. Because of TLB-flush in alloc_heap_pages, it's very slow
> > > to create a guest with memory size of more than 100GB on host with 100+
> > > cpus.
> > >
> > >
> > Do you have some actual numbers on how much faster after applying this
> > patch?
> >
> > This is mostly for writing release note etc, so it is fine if you don't
> > have numbers at hand.
>
> I do not have data of upstream version at hand now. I always tested the
> performance of this patchset by backporting it to an Oracle VM based on a
> very old Xen version, before I sent out the patchset for review. The
> backport just involves: (1) copy & paste code, (2) change bool to bool_t
> and (3) change true/false to 1/0.
>
> The test machine has 8 nodes of 2048GB memory and 128 cpu. With this
> patchset applied, the time to re-create a VM (with 135GB memory and 12
> vcpu) is reduced from 5min to 20s.
Thanks, this is useful information.
Wei.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2016-09-21 11:23 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-21 1:39 [PATCH v6 2/2] xen: move TLB-flush filtering out into populate_physmap during vm creation Dongli Zhang
2016-09-21 11:23 ` Wei Liu [this message]
-- strict thread matches above, loose matches on Subject: below --
2016-09-20 2:31 [PATCH v6 1/2] xen: replace tlbflush check and operation with inline functions Dongli Zhang
2016-09-20 2:31 ` [PATCH v6 2/2] xen: move TLB-flush filtering out into populate_physmap during vm creation Dongli Zhang
2016-09-20 9:01 ` Jan Beulich
2016-09-20 11:20 ` George Dunlap
2016-09-20 12:14 ` Dario Faggioli
2016-09-20 14:54 ` Wei Liu
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=20160921112356.GF30319@citrix.com \
--to=wei.liu2@citrix.com \
--cc=George.Dunlap@eu.citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=dario.faggioli@citrix.com \
--cc=dongli.zhang@oracle.com \
--cc=ian.jackson@eu.citrix.com \
--cc=jbeulich@suse.com \
--cc=jinsong.liu@alibaba-inc.com \
--cc=sstabellini@kernel.org \
--cc=tim@xen.org \
--cc=xen-devel@lists.xen.org \
/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.