Re: [Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-04-07 Thread Wei Liu
On Fri, Apr 07, 2017 at 12:05:01PM +0200, Dario Faggioli wrote: > On Fri, 2017-04-07 at 10:42 +0100, Wei Liu wrote: > > On Thu, Apr 06, 2017 at 05:18:33PM +0200, Dario Faggioli wrote: > > > > > > Wei acked this patch in v1 (20170321170902.ndk6h5ylyfkk4coo@citrix. > > > com) > > > but that was befo

Re: [Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-04-07 Thread Dario Faggioli
On Fri, 2017-04-07 at 10:42 +0100, Wei Liu wrote: > On Thu, Apr 06, 2017 at 05:18:33PM +0200, Dario Faggioli wrote: > > > > Wei acked this patch in v1 (20170321170902.ndk6h5ylyfkk4coo@citrix. > > com) > > but that was before you raised this, so I'm happy to resend with > > this > > changed, and do

Re: [Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-04-07 Thread Wei Liu
On Thu, Apr 06, 2017 at 05:18:33PM +0200, Dario Faggioli wrote: > On Thu, 2017-04-06 at 14:59 +0100, George Dunlap wrote: > > On 06/04/17 11:49, Dario Faggioli wrote: > > > > > > If that fails (I've tried that), domain creation fails too. So > > > either > > > it returns success, or we'd have to m

Re: [Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-04-06 Thread Dario Faggioli
On Thu, 2017-04-06 at 14:59 +0100, George Dunlap wrote: > On 06/04/17 11:49, Dario Faggioli wrote: > > > > If that fails (I've tried that), domain creation fails too. So > > either > > it returns success, or we'd have to modify (at least) > > liblx__build_post(), teaching it about acceptable failu

Re: [Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-04-06 Thread George Dunlap
On 06/04/17 11:49, Dario Faggioli wrote: > On Mon, 2017-03-27 at 11:50 +0100, George Dunlap wrote: >> On 17/03/17 18:43, Dario Faggioli wrote: >>> --- a/tools/libxl/libxl_sched.c >>> +++ b/tools/libxl/libxl_sched.c >>> @@ -178,6 +178,20 @@ static int sched_arinc653_domain_set(libxl__gc >>> *gc, uin

Re: [Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-04-06 Thread Dario Faggioli
On Mon, 2017-03-27 at 11:50 +0100, George Dunlap wrote: > On 17/03/17 18:43, Dario Faggioli wrote: > > --- a/tools/libxl/libxl_sched.c > > +++ b/tools/libxl/libxl_sched.c > > @@ -178,6 +178,20 @@ static int sched_arinc653_domain_set(libxl__gc > > *gc, uint32_t domid, > >  return 0; > >  } > >  

Re: [Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-03-27 Thread George Dunlap
On 17/03/17 18:43, Dario Faggioli wrote: > It being very very basic, also means this scheduler does > not need much support at the tools level (for now). > > Basically, just the definition of the symbol of the > scheduler itself and a couple of stubs. > > Signed-off-by: Dario Faggioli > --- > Cc

Re: [Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-03-21 Thread Wei Liu
On Fri, Mar 17, 2017 at 07:43:13PM +0100, Dario Faggioli wrote: > It being very very basic, also means this scheduler does > not need much support at the tools level (for now). > > Basically, just the definition of the symbol of the > scheduler itself and a couple of stubs. > > Signed-off-by: Dar

Re: [Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-03-20 Thread Stefano Stabellini
On Fri, 17 Mar 2017, Dario Faggioli wrote: > It being very very basic, also means this scheduler does > not need much support at the tools level (for now). > > Basically, just the definition of the symbol of the > scheduler itself and a couple of stubs. > > Signed-off-by: Dario Faggioli Acked-b

[Xen-devel] [PATCH 3/3] tools: sched: add support for 'null' scheduler

2017-03-17 Thread Dario Faggioli
It being very very basic, also means this scheduler does not need much support at the tools level (for now). Basically, just the definition of the symbol of the scheduler itself and a couple of stubs. Signed-off-by: Dario Faggioli --- Cc: Wei Liu Cc: Ian Jackson Cc: George Dunlap Cc: Stefano