All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dario Faggioli <dfaggioli@suse.com>
To: Ian Jackson <ian.jackson@citrix.com>,
	Roger Pau Monne <roger.pau@citrix.com>
Cc: "xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Subject: Re: [PATCH OSSTEST 2/2] make-flight: add a core scheduling job
Date: Thu, 16 Apr 2020 18:28:33 +0200	[thread overview]
Message-ID: <ca19a888e95d00ce664b9d777510e366547327cc.camel@suse.com> (raw)
In-Reply-To: <24215.1729.892903.300149@mariner.uk.xensource.com>

[-- Attachment #1: Type: text/plain, Size: 1592 bytes --]

On Wed, 2020-04-15 at 14:06 +0100, Ian Jackson wrote:
> Roger Pau Monne writes ("[PATCH OSSTEST 2/2] make-flight: add a core
> scheduling job"):
> > Run a simple core scheduling tests on a host that has SMT support.
> > This is only enabled for Xen >= 4.13.
> ...
> > +  # Core-scheduling tests are x86 only
> > +  if [ x$test_coresched = xy -a $xenarch = amd64 ]; then
> > +    job_create_test test-$xenarch$kern-coresched-$dom0arch-xl \
> > +                    test-debian xl $xenarch $dom0arch
> > $debian_runvars \
> > +                    all_hostflags=$most_hostflags,smt \
> > +                    xen_boot_append='sched-gran=core'
> > +
> > +  fi
> 
> This seems fine as far as it goes, but all it does is check that
> things still work if sched-gran=core is passed. 
>
Yep, and that's enough for enabling and starting using ore-scheduling.
So, doing like this, core-scheduling should get the same amount and
kind of testing that scheduling in general gets.

>  I'm not sure whether
> anything more sophisticated is needed, and in any case this is a step
> in the right direction, so:
> 
Indeed.

One question, as my OSSTest-fu is a bit rusty... does this create
"sched-gran=core" tests for all the schedulers? Or just one of them for
th default scheduler?

Thanks and Regards
-- 
Dario Faggioli, Ph.D
http://about.me/dario.faggioli
Virtualization Software Engineer
SUSE Labs, SUSE https://www.suse.com/
-------------------------------------------------------------------
<<This happens because _I_ choose it to happen!>> (Raistlin Majere)


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2020-04-16 16:29 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-15  8:52 [PATCH OSSTEST 1/2] exanime: test for SMT and add a host flag Roger Pau Monne
2020-04-15  8:52 ` [PATCH OSSTEST 2/2] make-flight: add a core scheduling job Roger Pau Monne
2020-04-15 13:06   ` Ian Jackson
2020-04-16 16:28     ` Dario Faggioli [this message]
2020-04-16 16:36       ` Roger Pau Monné
2020-04-16 16:55         ` Dario Faggioli
2020-04-16 17:10           ` Ian Jackson
2020-04-16 16:58         ` Ian Jackson
2020-04-15 10:15 ` [PATCH OSSTEST v2 1/2] exanime: test for SMT and add a host flag Roger Pau Monne
2020-04-15 13:07   ` Ian Jackson
2020-04-15 13:04 ` [PATCH OSSTEST " Ian Jackson
2020-04-15 13:47   ` Roger Pau Monné
2020-04-15 13:50     ` Ian Jackson

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=ca19a888e95d00ce664b9d777510e366547327cc.camel@suse.com \
    --to=dfaggioli@suse.com \
    --cc=ian.jackson@citrix.com \
    --cc=roger.pau@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.