All of lore.kernel.org
 help / color / mirror / Atom feed
From: George Dunlap <george.dunlap@eu.citrix.com>
To: Meng Xu <xumengpanda@gmail.com>
Cc: Ian Jackson <Ian.Jackson@citrix.com>,
	Ian Campbell <ian.campbell@citrix.com>,
	Sisu Xi <xisisu@gmail.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	Dario Faggioli <dario.faggioli@citrix.com>,
	Ian Jackson <ian.jackson@eu.citrix.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	Meng Xu <mengxu@cis.upenn.edu>, Jan Beulich <JBeulich@suse.com>,
	Chao Wang <chaowang@wustl.edu>, Chong Li <lichong659@gmail.com>,
	Dagaen Golomb <dgolomb@seas.upenn.edu>
Subject: Re: [PATCH v1 3/4] libxl: add rt scheduler
Date: Thu, 4 Sep 2014 15:51:33 +0100	[thread overview]
Message-ID: <54087C75.8090907@eu.citrix.com> (raw)
In-Reply-To: <CAENZ-+nBnkSZyMVxd-fh3CmDVgz592s0+DozT+TT7=78pOMA1w@mail.gmail.com>


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

On 09/04/2014 03:47 PM, Meng Xu wrote:
> Hi George,
>
>
> 2014-09-04 10:27 GMT-04:00 George Dunlap <George.Dunlap@eu.citrix.com 
> <mailto:George.Dunlap@eu.citrix.com>>:
>
>     On Wed, Sep 3, 2014 at 4:33 PM, George Dunlap
>     <George.Dunlap@eu.citrix.com <mailto:George.Dunlap@eu.citrix.com>>
>     wrote:
>     >
>     > While the domctl interface is not stable, the libxl interface *is*
>     > stable, so we definitely need to think carefully about what we want
>     > this to look like.
>     >
>     > Let me give that a think. :-)
>
>     OK, so we had a chat about this at our team meeting today, and here is
>     what we came up with.
>
>     The feature freeze for 4.5 is next Wednesday.
>
>     The core scheduler is in good enough shape to be checked in as an
>     "experimental" mode, so it would be really nice to be able to get this
>     checked in.
>
>     The DOMCTL interface isn't stable so we can change that if we need to;
>     however, the libxl interface *is* stable.
>
>     The current libxl scheduler parameter interface assumes one set of
>     parameters per domain; it's not yet setup for per-vcpu parameters.  It
>     is unlikely that we would be able to converge on a new interface by
>     next week.
>
>     So the suggestion was this: For the moment, use the existing libxl
>     interface on a per-domain basis.  Internally, this will set all vcpus
>     to the same values.  This will allow us to check in a useable version
>     of the scheduler for people to test and improve.  Then for 4.6 we can
>     start working on a suitable libxl interface for setting per-vcpu
>     scheduling parameters.
>
>     Dario / Ian, did I miss anything?
>
>     Meng / &c, does that sound reasonable?
>
>
> I have a question as to the user interface.
> For 4.5, we only allow users to set all vcpus to the same values (I'm 
> totally fine with it.);
> But how about the get function? When users issue the command "xl 
> sched-rt", how should we display the parameters of vcpus? We just give 
> the "period", "budget" and "#VCPU" for a domain? I'm fine with this 
> display for 4.5.
>
> However ,my concerns is: In 4.6, when we allow vcpus to have different 
> parameters and need to display every vcpu's parameters, how should we 
> display when users use command "xl sched-rt"? When vcpus have 
> different period and budget, we cannot display like what we did in 4.5 
> then. :-(
>
> It's just my thought, just in case we neglect it. :-)

I think the xl interface doesn't have quite the same consistency 
guarantees as the libxl interface.  For now, I think just make it print 
one budget / period for the domain; and we can change it later.

I'll ping Ian Jackson to make sure he's OK with that.

  -George

[-- Attachment #1.2: Type: text/html, Size: 5913 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-04 14:51 UTC|newest]

Thread overview: 72+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-24 22:58 Introduce rt real-time scheduler for Xen Meng Xu
2014-08-24 22:58 ` [PATCH v1 1/4] xen: add real time scheduler rt Meng Xu
2014-08-26 14:27   ` Jan Beulich
2014-08-27  2:07     ` Meng Xu
2014-08-27  6:26       ` Jan Beulich
2014-08-27 14:28         ` Meng Xu
2014-08-27 15:04           ` Jan Beulich
2014-08-28 16:06             ` Meng Xu
2014-08-29  9:05               ` Jan Beulich
2014-08-29 19:35                 ` Meng Xu
2014-09-03 14:08                 ` George Dunlap
2014-09-03 14:24                   ` Meng Xu
2014-09-03 14:35                     ` Dario Faggioli
2014-09-03 13:40   ` George Dunlap
2014-09-03 14:11     ` Meng Xu
2014-09-03 14:15       ` George Dunlap
2014-09-03 14:35         ` Meng Xu
2014-09-05  9:46     ` Dario Faggioli
2014-09-03 14:20   ` George Dunlap
2014-09-03 14:45     ` Jan Beulich
2014-09-03 14:59     ` Dario Faggioli
2014-09-03 15:27       ` Meng Xu
2014-09-03 15:46         ` Dario Faggioli
2014-09-03 17:13           ` George Dunlap
2014-09-03 15:13     ` Meng Xu
2014-09-03 16:06       ` George Dunlap
2014-09-03 16:57         ` Dario Faggioli
2014-09-03 17:18           ` George Dunlap
2014-09-04  2:15             ` Meng Xu
2014-09-04 14:27             ` Dario Faggioli
2014-09-04 15:30               ` Meng Xu
2014-09-05  9:36                 ` Dario Faggioli
2014-09-05 15:06                   ` Meng Xu
2014-09-05 15:09                     ` Dario Faggioli
2014-09-04  2:11         ` Meng Xu
2014-09-04 11:00           ` Dario Faggioli
2014-09-04 13:03           ` George Dunlap
2014-09-04 14:00             ` Meng Xu
2014-09-05 17:17   ` Dario Faggioli
2014-09-07  3:56     ` Meng Xu
2014-09-08 10:33       ` Dario Faggioli
2014-09-09 13:43         ` Meng Xu
2014-08-24 22:58 ` [PATCH v1 2/4] libxc: add rt scheduler Meng Xu
2014-09-05 10:34   ` Dario Faggioli
2014-09-05 17:17     ` Meng Xu
2014-09-05 17:50       ` Dario Faggioli
2014-08-24 22:58 ` [PATCH v1 3/4] libxl: " Meng Xu
2014-08-25 13:17   ` Wei Liu
2014-08-25 15:55     ` Meng Xu
2014-08-26  9:51       ` Wei Liu
2014-09-03 15:33   ` George Dunlap
2014-09-03 20:52     ` Meng Xu
2014-09-04 14:27     ` George Dunlap
2014-09-04 14:45       ` Dario Faggioli
2014-09-04 14:47       ` Meng Xu
2014-09-04 14:51         ` George Dunlap [this message]
2014-09-04 15:07           ` Meng Xu
2014-09-04 15:44             ` Dario Faggioli
2014-09-04 15:55               ` George Dunlap
2014-09-04 16:12                 ` Meng Xu
2014-09-05  9:19                   ` Dario Faggioli
2014-09-04 15:25         ` Dario Faggioli
2014-09-05 10:21   ` Dario Faggioli
2014-09-05 15:45     ` Meng Xu
2014-09-05 17:41       ` Dario Faggioli
2014-08-24 22:58 ` [PATCH v1 4/4] xl: introduce " Meng Xu
2014-08-25 13:31   ` Wei Liu
2014-08-25 16:12     ` Meng Xu
2014-09-03 15:52   ` George Dunlap
2014-09-03 22:28     ` Meng Xu
2014-09-05  9:40       ` Dario Faggioli
2014-09-05 14:43         ` 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=54087C75.8090907@eu.citrix.com \
    --to=george.dunlap@eu.citrix.com \
    --cc=Ian.Jackson@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=chaowang@wustl.edu \
    --cc=dario.faggioli@citrix.com \
    --cc=dgolomb@seas.upenn.edu \
    --cc=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=lichong659@gmail.com \
    --cc=mengxu@cis.upenn.edu \
    --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.