Re: [Xen-devel] [PATCH v2 1/7] libxl: get rid of the SEDF scheduler

2015-07-03 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH v2 1/7] libxl: get rid of the SEDF scheduler"): > On Fri, 2015-07-03 at 16:33 +0200, Dario Faggioli wrote: > > I think I'd go for _REMOVED, as deprecated suggests (to me at least) > > that it's still there but nshould not be used. > > REMOVED > DEPRECATED > GONE IM

Re: [Xen-devel] [PATCH v2 1/7] libxl: get rid of the SEDF scheduler

2015-07-03 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH v2 1/7] libxl: get rid of the SEDF scheduler"): > On Fri, 2015-07-03 at 12:24 +0200, Dario Faggioli wrote: > > @@ -5932,7 +5865,8 @@ int libxl_domain_sched_params_set(libxl_ctx *ctx, > > uint32_t domid, ... > > case LIBXL_SCHEDULER_SEDF: > > -ret=sched

Re: [Xen-devel] [PATCH v2 1/7] libxl: get rid of the SEDF scheduler

2015-07-03 Thread Ian Campbell
On Fri, 2015-07-03 at 16:33 +0200, Dario Faggioli wrote: > On Fri, 2015-07-03 at 15:23 +0100, Ian Campbell wrote: > > On Fri, 2015-07-03 at 12:24 +0200, Dario Faggioli wrote: > > > > > @@ -5932,7 +5865,8 @@ int libxl_domain_sched_params_set(libxl_ctx *ctx, > > > uint32_t domid, > > > > > >

Re: [Xen-devel] [PATCH v2 1/7] libxl: get rid of the SEDF scheduler

2015-07-03 Thread Dario Faggioli
On Fri, 2015-07-03 at 15:23 +0100, Ian Campbell wrote: > On Fri, 2015-07-03 at 12:24 +0200, Dario Faggioli wrote: > > > @@ -5932,7 +5865,8 @@ int libxl_domain_sched_params_set(libxl_ctx *ctx, > > uint32_t domid, > > > > switch (sched) { > > case LIBXL_SCHEDULER_SEDF: > > -ret=

Re: [Xen-devel] [PATCH v2 1/7] libxl: get rid of the SEDF scheduler

2015-07-03 Thread Ian Campbell
On Fri, 2015-07-03 at 12:24 +0200, Dario Faggioli wrote: > @@ -5932,7 +5865,8 @@ int libxl_domain_sched_params_set(libxl_ctx *ctx, > uint32_t domid, > > switch (sched) { > case LIBXL_SCHEDULER_SEDF: > -ret=sched_sedf_domain_set(gc, domid, scinfo); > +LOG(ERROR, "SEDF s

[Xen-devel] [PATCH v2 1/7] libxl: get rid of the SEDF scheduler

2015-07-03 Thread Dario Faggioli
only the interface is left in place, for backward compile-time compatibility, but every attempt to use it would throw an error. Signed-off-by: Dario Faggioli Reviewed-by: George Dunlap --- Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu --- tools/libxl/libxl.c|