[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [RFC PATCH v2 6/7] Changed slice to budget in libxc for the sedf scheduler



On Wed, 2014-07-09 at 16:55 -0400, Josh Whitehead wrote:
> From: Robbie VanVossen <robert.vanvossen@xxxxxxxxxxxxxxx>
> 
> Signed-off-by: Robert VanVossen <Robert.VanVossen@xxxxxxxxxxxxxxx>
> Signed-off-by: Nathan Studer <nate.studer@xxxxxxxxx>
> 
> ---
>  tools/libxc/xc_sedf.c             |    8 ++++----
>  tools/libxc/xenctrl.h             |    4 ++--
>  tools/python/xen/lowlevel/xc/xc.c |   20 ++++++++++----------
>  xen/include/public/domctl.h       |    2 +-
>  4 files changed, 17 insertions(+), 17 deletions(-)
> 
> diff --git a/tools/libxc/xc_sedf.c b/tools/libxc/xc_sedf.c
> index 81ff133..ab67546 100644
> --- a/tools/libxc/xc_sedf.c
> +++ b/tools/libxc/xc_sedf.c
> @@ -28,7 +28,7 @@ int xc_sedf_domain_set(
>      xc_interface *xch,
>      uint32_t domid,
>      uint64_t period,
> -    uint64_t slice,
> +    uint64_t budget,

Isn't the going to break the libxl behaviour between this patch and the
next one?

I think it is unavoidable to change this interface and the libxl one at
the same time.

If there is sensible default budget then perhaps you could change libxl
to use that unconditionally in this patch and then do all the plumbing
out to the libxl API and into xl in the next patch.

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.