Re: [PATCH v1 3/4] libxl: add rt scheduler

[Dario Faggioli <dario.faggioli@citrix.com>]

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

On gio, 2014-09-04 at 12:12 -0400, Meng Xu wrote:

> 2014-09-04 11:55 GMT-04:00 George Dunlap 
>         
>         >> I see. I'm totally ok with the decision! :-)
>         >> So I will only use the existing libxl interface without
>         adding an
>         >> array to it, to set/get the vcpus' parameters of a domain.
>         Am I right?
>         >>
>         > Yep, no array. You just add a 'period' and a 'budget' fields
>         _inside_
>         > libxl_domain_sched_params, without putting them inside any
>         wrapping
>         > struct, union or array.
>         
>         
>         Except that you don't need to add a "period" field, since
>         there's
>         already one there (for the SEDF scheduler).
>         
Right, I overlooked that, sorry. :-P

>         We could re-use the "slice" field instead of adding "budget",
>         but I
>         think probably for clarity adding "budget" is better (although
>         I'm
>         open to other opinions on that one).
> 
> 
> ​I like the idea of adding "budget" because it's much clearer. ​
> 
I'm fine with adding a "budget" parameter too.

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