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