All of lore.kernel.org
 help / color / mirror / Atom feed
From: George Dunlap <george.dunlap@eu.citrix.com>
To: Wei Liu <wei.liu2@citrix.com>
Cc: Sander Eikelenboom <linux@eikelenboom.it>,
	Dario Faggioli <dario.faggioli@citrix.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: Call to write tests for osstest
Date: Wed, 14 Aug 2013 16:03:09 +0100	[thread overview]
Message-ID: <520B9C2D.6080601@eu.citrix.com> (raw)
In-Reply-To: <20130809120008.GF18798@zion.uk.xensource.com>

On 09/08/13 13:00, Wei Liu wrote:
> On Fri, Aug 09, 2013 at 01:46:42PM +0200, Sander Eikelenboom wrote:
>> Friday, August 9, 2013, 1:19:54 PM, you wrote:
>>
>>> On gio, 2013-08-08 at 19:32 +0200, Sander Eikelenboom wrote:
>>>> * Some performance testing (network, block, cpu/mem/fork/real apps benchmarks, some other metrics)
>>>>     - perhaps makes separate graphs of these, so one can see performance increase or decrease over a larger time frame, and see at around what commits that occurred.
>>>>     - only after all basic tests succeeded and a push was done.
>>>>
>>> We are after this too. I am looking at how to make it possible to do
>>> something like that on top of OSSTest.
>>> Perf benchmarking is a little bit different from regression
>>> smoke-testing, but still I think (hope? :-)) that most of the
>>> infrastructure can be reused.
>> Yes it does require to keep the rest of the circumstances the same for a longer period of time.
>> But it could be quite valuable, slow introduced and minor performance regressions are hard to discover.
>>
>>> Also, what hardware to use and how to properly schedule these kind of
>>> "tests" is something that needs a bit more of thinking/discussion, I
>>> think.
>> Yes since you have to keep the "environment" the same, the machine should only run the perf tests at that time.
>>
> This worries me. It is really hard to keep the "environment" the same.
> AIUI network / block etc. performance can be affected by other kernel
> subsystems. Also the kernel config could also have impact on
> performance.

The same is true for functional regressions -- a bug could be introduced 
by Xen, qemu, or the kernel.  The tester already keeps track of this, as 
far as I know, and only moves one thing at a time.

  -George

  parent reply	other threads:[~2013-08-14 15:03 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-08 16:01 Call to write tests for osstest George Dunlap
2013-08-08 17:32 ` Sander Eikelenboom
2013-08-08 19:42   ` Konrad Rzeszutek Wilk
2013-08-09 11:19   ` Dario Faggioli
2013-08-09 11:46     ` Sander Eikelenboom
2013-08-09 12:00       ` Wei Liu
2013-08-09 12:04         ` Sander Eikelenboom
2013-08-09 12:16           ` Dario Faggioli
2013-08-14 15:03         ` George Dunlap [this message]
2013-08-08 17:37 ` Sander Eikelenboom
2013-08-14 13:51   ` George Dunlap
2013-08-08 19:40 ` Konrad Rzeszutek Wilk
2013-08-14 13:52   ` George Dunlap
2013-08-13 14:24 ` Ben Guthro

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=520B9C2D.6080601@eu.citrix.com \
    --to=george.dunlap@eu.citrix.com \
    --cc=dario.faggioli@citrix.com \
    --cc=linux@eikelenboom.it \
    --cc=wei.liu2@citrix.com \
    --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.