Re: [Xen-devel] [PATCH v2 10/10] xl: allow to set the ratelimit value online for Credit2

2016-10-14 Thread George Dunlap
On 13/10/16 23:19, Jim Fehlig wrote: > On 09/29/2016 08:54 PM, Dario Faggioli wrote: >> Last part of the wiring necessary for allowing to >> change the value of the ratelimit_us parameter online, >> for Credit2 (like it is already for Credit1). >> >> Signed-off-by: Dario Faggioli

Re: [Xen-devel] [PATCH v2 10/10] xl: allow to set the ratelimit value online for Credit2

2016-10-13 Thread Jim Fehlig
On 09/29/2016 08:54 PM, Dario Faggioli wrote: Last part of the wiring necessary for allowing to change the value of the ratelimit_us parameter online, for Credit2 (like it is already for Credit1). Signed-off-by: Dario Faggioli Reviewed-by: George Dunlap

Re: [Xen-devel] [PATCH v2 10/10] xl: allow to set the ratelimit value online for Credit2

2016-09-30 Thread Ian Jackson
Dario Faggioli writes ("Re: [Xen-devel] [PATCH v2 10/10] xl: allow to set the ratelimit value online for Credit2"): > I mean, AFAICT, there's xl calling a libxl function and printing an > error if it fails. The libxl function also logs an error --true-- but > this seems rather common to me, and

Re: [Xen-devel] [PATCH v2 10/10] xl: allow to set the ratelimit value online for Credit2

2016-09-30 Thread Dario Faggioli
On Fri, 2016-09-30 at 11:34 +0100, Ian Jackson wrote: > Dario Faggioli writes ("[PATCH v2 10/10] xl: allow to set > theratelimit value online for Credit2"): > > > > +static int sched_credit2_params_set(int poolid, > > +libxl_sched_credit2_params > > *scinfo) >

Re: [Xen-devel] [PATCH v2 10/10] xl: allow to set the ratelimit value online for Credit2

2016-09-30 Thread Ian Jackson
Dario Faggioli writes ("[PATCH v2 10/10] xl: allow to set the ratelimit value online for Credit2"): > Last part of the wiring necessary for allowing to > change the value of the ratelimit_us parameter online, > for Credit2 (like it is already for Credit1). > > Signed-off-by: Dario Faggioli

[Xen-devel] [PATCH v2 10/10] xl: allow to set the ratelimit value online for Credit2

2016-09-29 Thread Dario Faggioli
Last part of the wiring necessary for allowing to change the value of the ratelimit_us parameter online, for Credit2 (like it is already for Credit1). Signed-off-by: Dario Faggioli Reviewed-by: George Dunlap --- Cc: Ian Jackson