All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dario Faggioli <dario.faggioli@citrix.com>
To: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: ian.campbell@citrix.com, xisisu@gmail.com,
	stefano.stabellini@eu.citrix.com,
	George Dunlap <george.dunlap@eu.citrix.com>,
	lu@cse.wustl.edu, ian.jackson@eu.citrix.com,
	xen-devel@lists.xen.org, ptxlinh@gmail.com,
	xumengpanda@gmail.com, Meng Xu <mengxu@cis.upenn.edu>,
	JBeulich@suse.com, chaowang@wustl.edu, lichong659@gmail.com,
	dgolomb@seas.upenn.edu
Subject: Re: [PATCH v3 1/4] xen: add real time scheduler rtds
Date: Mon, 22 Sep 2014 19:26:28 +0200	[thread overview]
Message-ID: <1411406788.4116.26.camel@Solace.lan> (raw)
In-Reply-To: <20140919164422.GA9619@laptop.dumpdata.com>


[-- Attachment #1.1: Type: text/plain, Size: 2555 bytes --]

On ven, 2014-09-19 at 12:44 -0400, Konrad Rzeszutek Wilk wrote:
> On Thu, Sep 18, 2014 at 05:08:45PM +0100, George Dunlap wrote:

> > deadline, and then address the things I'm bringing up here?  Or would it be
> > better to wait until all the issues are sorted and then check it in (even if
> > it's after the deadline)?
> 
> We can check it in after the deadline - and have those issues resolved.
> 
FWIW, I think the series looks good now, and in fact I sent in my
Reviewed-by for all of it.

> I am basing this on the assumption that:
>  - The risks of regressions to the rest of schedulers is nill (as this is all
>    new codepaths (as this is all
>  - The risks of regressions to the rest of the code-base is nill (as this is all
>    new).
>  - The resolution of the 'couple of things' are not going to lead to more
>    'couple of things' and lead to re-design.
>
This is all true, IMO.

> The common code that is touched does not look scary to me. And both of the
> scheduler maintainers -  you and Dariof are OK with the design and the patchset
> (minus the 'couple of things').
> 
Exactly.

> Are we aim to have this be experimental for Xen 4.5 or do we want this
> to be on the 'stable' ?
>
Not sure. What I'm sure about is that
1) the interface needs to change a bit, to include support for the
   per-vcpu parameters setting (although, that can happen in a
   backward compatible way, i.e., not touching or altering neither the
   look nor the semantic or the interface we'll be checking in if we
   take v4)
2) there is _a_lot_ to gain, from a performance point of view, and Meng
   already agreed on continuing working toward that, after 4.5

Having it in is, IMO, important, especially for the new
embedded/mobile/automotive uses of Xen we're seeing in these days (in
fact, I think GlobalLogic is using RT-Xen already, so the upstreaming of
this scheduler would be quite useful at least to them [correct me if I'm
wrong]).

However, given 2 above, if we mark it as stable, we risk that people
(mostly people not yet involved into Xen development and not on this
mailing list) would try it, run into non-optimal performance, and get
upset/angry. For that reason, I think I'd go for 'experimental for 4.5'.

Regards,
Dario

-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


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

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  reply	other threads:[~2014-09-22 17:26 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-14 21:37 Introduce rtds real-time scheduler for Xen Meng Xu
2014-09-14 21:37 ` [PATCH v3 1/4] xen: add real time scheduler rtds Meng Xu
2014-09-15 10:40   ` Jan Beulich
2014-09-16  8:42   ` Dario Faggioli
2014-09-16  8:52     ` Jan Beulich
2014-09-16  8:59       ` Dario Faggioli
2014-09-16 16:38       ` Meng Xu
2014-09-17  9:22         ` Jan Beulich
2014-09-18 16:08   ` George Dunlap
2014-09-18 18:08     ` Dario Faggioli
2014-09-19 13:11       ` Meng Xu
2014-09-22 17:11         ` Dario Faggioli
2014-09-22 20:40           ` Meng Xu
2014-09-19  9:45     ` Dario Faggioli
2014-09-19 16:44     ` Konrad Rzeszutek Wilk
2014-09-22 17:26       ` Dario Faggioli [this message]
2014-09-22 20:45         ` Meng Xu
2014-09-22 22:43         ` Konrad Rzeszutek Wilk
2014-09-20 21:10     ` Meng Xu
2014-09-23 10:47       ` George Dunlap
2014-09-14 21:37 ` [PATCH v3 2/4] libxc: add rtds scheduler Meng Xu
2014-09-18 16:15   ` George Dunlap
2014-09-14 21:37 ` [PATCH v3 3/4] libxl: " Meng Xu
2014-09-15 22:07   ` Ian Campbell
2014-09-16  1:11     ` Meng Xu
2014-09-16  1:49       ` Ian Campbell
2014-09-16  3:32         ` Meng Xu
2014-09-16  7:27           ` Dario Faggioli
2014-09-16 16:54             ` Ian Campbell
2014-09-17 10:19               ` Dario Faggioli
2014-09-16  8:04   ` Dario Faggioli
2014-09-16 16:56     ` Ian Campbell
2014-09-18 16:24   ` George Dunlap
2014-09-18 17:19     ` Ian Campbell
2014-09-14 21:37 ` [PATCH v3 4/4] xl: introduce " Meng Xu
2014-09-15 22:18   ` Ian Campbell
2014-09-16  7:49   ` Dario Faggioli
2014-09-18 16:35   ` George Dunlap
2014-09-16  7:43 ` Introduce rtds real-time scheduler for Xen Dario Faggioli
2014-09-17 14:15 ` Dario Faggioli
2014-09-17 14:33   ` Meng Xu
2014-09-18 16:00   ` Meng Xu
2014-09-23 13:50 ` Ian Campbell
2014-09-24 20:59   ` Meng Xu
2014-09-24 21:14     ` Wei Liu
2014-09-25  7:39       ` Ian Campbell

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=1411406788.4116.26.camel@Solace.lan \
    --to=dario.faggioli@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=chaowang@wustl.edu \
    --cc=dgolomb@seas.upenn.edu \
    --cc=george.dunlap@eu.citrix.com \
    --cc=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=konrad.wilk@oracle.com \
    --cc=lichong659@gmail.com \
    --cc=lu@cse.wustl.edu \
    --cc=mengxu@cis.upenn.edu \
    --cc=ptxlinh@gmail.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xen.org \
    --cc=xisisu@gmail.com \
    --cc=xumengpanda@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.