From: Dario Faggioli <dario.faggioli@citrix.com>
To: Meng Xu <mengxu@cis.upenn.edu>,
"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Cc: George Dunlap <george.dunlap@eu.citrix.com>
Subject: Re: Should we mark RTDS as supported feature from experimental feature?
Date: Tue, 26 Apr 2016 09:56:13 +0200 [thread overview]
Message-ID: <1461657373.25541.26.camel@citrix.com> (raw)
In-Reply-To: <CAENZ-+nnDtBPdiChTMgOeYMP0oj_a-vOtdbma_M=4h7NJ6KPcQ@mail.gmail.com>
[-- Attachment #1.1: Type: text/plain, Size: 3256 bytes --]
On Mon, 2016-04-25 at 21:44 -0400, Meng Xu wrote:
> Hi Dario and all,
>
Hi,
> When RTDS scheduler is initialized, it will print out that the
> scheduler is an experimental feature with the following lines:
>
> printk("Initializing RTDS scheduler\n"
>
> "WARNING: This is experimental software in development.\n"
>
> "Use at your own risk.\n");
>
> On RTDS' wiki [1], it says the RTDS scheduler is experimental
> feature.
>
Yes.
> However, inside MAINTAINERS file, the status of RTDS scheduler is
> marked as Supported (refer to commit point 28041371 by Dario Faggioli
> on 2015-06-25).
>
There's indeed a discrepancy between the way one can read that bit of
MAINTAINERS, and what is generally considered Supported (e.g., subject
to security support, etc).
This is true in general, not only for RTDS (more about this below).
> In my opinion, the RTDS scheduler's functionality is finished and
> tested. So should I send a patch to change the message printed out
> when the scheduler is initialized?
>
So, yes, the scheduler is now feature complete (with the per-vcpu
parameters) and adheres to a much more sensible and scalable design
(event driven). Yet, these features have been merged very recently,
therefore, when you say "tested", I'm not so sure I agree. In fact, we
do test it on OSSTest, but only in a couple of tests. The combination
of these two things make me think that we should allow for at least
another development cycle, before considering switching.
And speaking of OSSTest, there have benn occasional failures, on ARM,
which I haven't yet found the time to properly analyze. It may be just
something related to the fact that the specific board was very slow,
but I'm not sure yet.
And even in that case, I wonder how we should handle such a
situation... I was thinking of adding a work-conserving mode, what do
you think? You may have something similar in RT-Xen already but, even
if you don't, there are a number of ways for achieving that without
disrupting the real-time guarantees.
What do you think?
> If I understand correctly, the status in MAINTAINERS file should have
> the highest priority and information from other sources should keep
> updated with what the MAINTAINERS file says?
>
> Please correct me if I'm wrong.
>
This has been discussed before. Have a look at this thread/messages.
http://lists.xenproject.org/archives/html/xen-devel/2015-06/msg00972.html
http://lists.xenproject.org/archives/html/xen-devel/2015-06/msg01775.html
And at this:
http://lists.xenproject.org/archives/html/xen-devel/2015-06/msg01992.html
The feature document template has been put together:
http://lists.xenproject.org/archives/html/xen-devel/2015-08/msg01929.html
And there are feature documents in tree already.
Actually, writing one for RTDS would be a rather interesting and useful
thing to do, IMO! :-)
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
next prev parent reply other threads:[~2016-04-26 7:56 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-26 1:44 Should we mark RTDS as supported feature from experimental feature? Meng Xu
2016-04-26 7:56 ` Dario Faggioli [this message]
2016-04-26 8:56 ` Andrew Cooper
2016-04-26 18:41 ` Meng Xu
2016-04-26 15:35 ` George Dunlap
2016-04-26 20:00 ` Meng Xu
2016-04-26 23:01 ` Dario Faggioli
2016-04-27 1:16 ` Meng Xu
2016-04-27 12:27 ` Dario Faggioli
2016-04-27 20:04 ` Meng Xu
2016-04-26 22:38 ` Dario Faggioli
2016-04-26 18:38 ` Meng Xu
2016-04-26 22:49 ` Dario Faggioli
2016-04-27 0:02 ` Meng Xu
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=1461657373.25541.26.camel@citrix.com \
--to=dario.faggioli@citrix.com \
--cc=george.dunlap@eu.citrix.com \
--cc=mengxu@cis.upenn.edu \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).