All of lore.kernel.org
 help / color / mirror / Atom feed
From: George Dunlap <george.dunlap@citrix.com>
To: Andrew Cooper <andrew.cooper3@citrix.com>,
	xen-devel@lists.xenproject.org
Cc: Stefano Stabellini <sstabellini@kernel.org>,
	Wei Liu <wei.liu2@citrix.com>, Tim Deegan <tim@xen.org>,
	Dario Faggioli <dfaggioli@suse.com>,
	Jan Beulich <jbeulich@suse.com>,
	Ian Jackson <ian.jackson@citrix.com>
Subject: Re: [PATCH 1/3] Make credit2 the default scheduler
Date: Mon, 5 Feb 2018 13:01:45 +0000	[thread overview]
Message-ID: <d9a9e4c9-1a78-ac26-0bda-cea8e842e05d@citrix.com> (raw)
In-Reply-To: <edf342de-0dee-69c0-d239-75f928960bd4@citrix.com>

On 02/05/2018 11:45 AM, Andrew Cooper wrote:
> On 05/02/18 11:36, George Dunlap wrote:
>> Credit2 was declared "supported" in 4.8, and as of 4.10 had two other
>> critical features implemented (soft affinity / NUMA and caps).
>>
>> Signed-off-by: George Dunlap <george.dunlap@citrix.com>
> 
> On what justification?
> 
> Switching the default involves demonstrating that the new default is
> better than the old one, and simply "we've implemented more features"
> isn't good enough IMO.

Well the code is certainly better: more predictable, less jitter, easier
to determine how modifications will affect overall behavior, easier in
the future to make load-balancing behavior more subtle (e.g., taking
into account the cost of powering up extra cores, &c).

> The last set of measurements I recall seeing said that credit1 and
> credit2 were mostly kneck-and-kneck, but there was still a noticeable
> difference in credit1's favour when it came to aggregate small-packet
> TCP throughput, and no benchmark where credit2 was clearly a winner.

It's somewhat of a mixed bag.  Unfortunately most of what I have are
tests using XenServer's internal perf testing system, so I can't share
the raw data (via links anyway).

Here is a summary of data from an intern e-mail Dario sent about 6
months ago:

* DVDbench: On underloaded systems, credit2 outperformed credit1 by
about 4%.  On overloaded systems, credit2 underperformed by about 3%.

* On a range of tests (unixbench, lmbench, &c), credit and credit2
perform within 5% of each other (up and down).

* Credit2 fairly consistently beats credit for TCP-style workloads.

* Credit2 is sometimes equal to, sometimes 5-15% worse than, credit for
synthetic CPU workloads (e.g., Dhrystone).

* On LoginVSI, credit2 fairly consistently outperforms credit by about 10%.

Credit2, like credit, has a number of workloads / setups for which
performance could be improved.  Personally I think networking and
partially-loaded systems is going to be more representative of what Xen
is actually used for; so I think credit2 is on the whole the better
scheduler to use by default.  And in any case, making those improvements
on credit2 will be easier than on credit.

 -George

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

  reply	other threads:[~2018-02-05 13:01 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-05 11:36 [PATCH 1/3] Make credit2 the default scheduler George Dunlap
2018-02-05 11:36 ` [PATCH 2/3] xen: Fix credit1 Kconfig entry George Dunlap
2018-02-05 13:35   ` Jan Beulich
2018-02-05 14:43     ` Dario Faggioli
2018-02-05 11:36 ` [PATCH 3/3] xen: Disable ARINC653 scheduler by default for non-DEBUG builds George Dunlap
2018-02-05 13:36   ` Jan Beulich
2018-02-05 14:11     ` George Dunlap
2018-02-05 14:42     ` Dario Faggioli
2018-02-05 14:45       ` George Dunlap
2018-02-05 11:45 ` [PATCH 1/3] Make credit2 the default scheduler Andrew Cooper
2018-02-05 13:01   ` George Dunlap [this message]
2018-02-05 16:53     ` Dario Faggioli
2018-02-06  6:18       ` Juergen Gross
2018-02-06 17:02         ` George Dunlap
2018-02-06 22:39           ` Dario Faggioli
2018-02-07  9:31             ` George Dunlap

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=d9a9e4c9-1a78-ac26-0bda-cea8e842e05d@citrix.com \
    --to=george.dunlap@citrix.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=dfaggioli@suse.com \
    --cc=ian.jackson@citrix.com \
    --cc=jbeulich@suse.com \
    --cc=sstabellini@kernel.org \
    --cc=tim@xen.org \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xenproject.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.