From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Dario Faggioli <dario.faggioli@citrix.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 18:43:27 -0400 [thread overview]
Message-ID: <20140922224327.GA2844@laptop.dumpdata.com> (raw)
In-Reply-To: <1411406788.4116.26.camel@Solace.lan>
On Mon, Sep 22, 2014 at 07:26:28PM +0200, Dario Faggioli wrote:
> 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'.
OK, will make sure that it is label as such in the release notes/Wiki.
>
> 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)
>
next prev parent reply other threads:[~2014-09-22 22:43 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
2014-09-22 20:45 ` Meng Xu
2014-09-22 22:43 ` Konrad Rzeszutek Wilk [this message]
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=20140922224327.GA2844@laptop.dumpdata.com \
--to=konrad.wilk@oracle.com \
--cc=JBeulich@suse.com \
--cc=chaowang@wustl.edu \
--cc=dario.faggioli@citrix.com \
--cc=dgolomb@seas.upenn.edu \
--cc=george.dunlap@eu.citrix.com \
--cc=ian.campbell@citrix.com \
--cc=ian.jackson@eu.citrix.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.