From: Dario Faggioli <dario.faggioli@citrix.com>
To: George Dunlap <george.dunlap@eu.citrix.com>
Cc: Lars Kurth <lars.kurth@citrix.com>,
xumengpanda@gmail.com, Ian Campbell <ian.campbell@citrix.com>,
xisisu@gmail.com,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
Ian Jackson <ian.jackson@eu.citrix.com>,
Robert VanVossen <robert.vanvossen@dornerworks.com>,
Xen-devel <xen-devel@lists.xen.org>,
Nate Studer <nate.studer@gmail.com>,
Josh Whitehead <josh.whitehead@dornerworks.com>,
andrii.anisov@globallogic.com, Jan Beulich <JBeulich@suse.com>,
lichong659@gmail.com, Meng Xu <mengxu@cis.upenn.edu>
Subject: Re: [RFC PATCH 3/4] Updated comments/variables to reflect cbs, fixed formatting and confusing comments/variables
Date: Tue, 17 Jun 2014 19:28:05 +0200 [thread overview]
Message-ID: <1403026085.16864.246.camel@Solace> (raw)
In-Reply-To: <1403021511.16864.194.camel@Solace>
[-- Attachment #1.1: Type: text/plain, Size: 2996 bytes --]
[Adding RT-Xen people to the discussion, as well as Lars and Andrii from
GlobalLogic]
On mar, 2014-06-17 at 18:11 +0200, Dario Faggioli wrote:
> On lun, 2014-06-16 at 16:29 +0100, George Dunlap wrote:
> So, I'm asking, mostly to George, about the overall scheduling aspects
> and implications, and to the tools maintainer, as that's where API
> stability is to be enforced: should this be a concern? In what sense API
> stability applies here? Can we, for example, start to ignore one or more
> SEDF scheduling parameters?
>
> I'm asking explicitly about the parameters because, although I think
> that most of the changes in this series does not actually call for much
> renaming, at least the 'weight' and, to certain extent the 'extra',
> parameters are a bit difficult to deal with (mostly because they're a
> remnant from when SEDF was meant as a general purpose scheduler too!).
>
> Thoughts?
>
Related to this, there are basically two groups of people working on
real-time scheduling:
1) Josh and Robbie @ Dornerworks (+ Nate), working on fixing SEDF;
2) RT-Xen developers, working at isolating the upstreamable bits of
RT-Xen itself (although they've not sent patches here in public
yet).
The very nice part of 1) is that it (in the long run) fixes SEDF, and if
we are to keep it in the tree, I really think it should be fixed!
The very nice part of 2) is that it is already a more advanced and
mature real-time scheduling solution (for example, it already deals with
SMPs correctly, unlike current SEDF and Josh's RFC), but it really does
not have much to do with SEDF (either broken or fixed), both at an
interface and implementation (i.e., code sharing) points of view either.
So, again, depending on whether or not we want to keep SEDF, and how
hard we want for it's interface not only to compile, but to remain
meaningful, the way forward changes. That's why I'm asking what we want
to do with SEDF. :-/
Assuming that we *do* want to keep it, my personally preferred way to
proceed would be:
- we take the implementation changes, but not the renaming, from Josh's
effort
- we merge the sched_sedf.c resulting from above with sched_rtglobal.c
from RT-Xen, so to have a solution that works well on SMP systems
and also implements the existing SEDF interface
The merge can happen 'either way', i.e., we can try to borrow the global
scheduling bits (i.e., the SMP support) from RT-Xen's sched_rtglobal.c
into sched_sedf.c or, vice-versa, import Josh's SEDF/CBS code inside
sched_rtglobal.c... In either case, I'd need Josh and Meng, and their
respective fellows, to collaborate... Are you guys up for that? :-)
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: 198 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2014-06-17 17:28 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-13 19:58 [RFC PATCH 0/4] Repurpose SEDF Scheduler for Real-time use Josh Whitehead
2014-06-13 19:58 ` [RFC PATCH 1/4] Implement cbs algorithm, remove extra queues, latency scaling, and weight support from sedf Josh Whitehead
2014-06-17 15:43 ` Dario Faggioli
2014-06-26 20:17 ` Joshua Whitehead
2014-06-28 2:19 ` Dario Faggioli
2014-06-17 16:06 ` Dario Faggioli
2014-06-26 20:18 ` Joshua Whitehead
2014-06-28 2:27 ` Dario Faggioli
2014-06-13 19:58 ` [RFC PATCH 2/4] Add cbs parameter support to xl tool stack, remove defunct sedf parameters Josh Whitehead
2014-06-17 15:02 ` Dario Faggioli
2014-06-26 19:55 ` Joshua Whitehead
2014-06-13 19:58 ` [RFC PATCH 3/4] Updated comments/variables to reflect cbs, fixed formatting and confusing comments/variables Josh Whitehead
2014-06-16 9:33 ` Jan Beulich
2014-06-16 15:29 ` George Dunlap
2014-06-17 16:11 ` Dario Faggioli
2014-06-17 17:28 ` Dario Faggioli [this message]
2014-06-25 20:13 ` Meng Xu
2014-06-26 21:24 ` Joshua Whitehead
2014-06-28 2:13 ` Dario Faggioli
2014-06-18 11:18 ` George Dunlap
2014-06-26 21:30 ` Joshua Whitehead
2014-06-26 21:23 ` Joshua Whitehead
2014-06-28 2:09 ` Dario Faggioli
2014-06-13 19:58 ` [RFC PATCH 4/4] Changed filenames with sedf to cbs to reflect the actual scheduler Josh Whitehead
2014-06-16 7:25 ` [RFC PATCH 0/4] Repurpose SEDF Scheduler for Real-time use Dario Faggioli
2014-06-17 14:44 ` Dario Faggioli
2014-06-26 19:53 ` Joshua Whitehead
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=1403026085.16864.246.camel@Solace \
--to=dario.faggioli@citrix.com \
--cc=JBeulich@suse.com \
--cc=andrii.anisov@globallogic.com \
--cc=george.dunlap@eu.citrix.com \
--cc=ian.campbell@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=josh.whitehead@dornerworks.com \
--cc=lars.kurth@citrix.com \
--cc=lichong659@gmail.com \
--cc=mengxu@cis.upenn.edu \
--cc=nate.studer@gmail.com \
--cc=robert.vanvossen@dornerworks.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.