Re: [Xen-devel] [RFC PATCH] xen-netback: making the bandwidth limiter runtime settable

2015-03-18 Thread Wei Liu
On Wed, Mar 18, 2015 at 05:21:08PM +0100, Imre Palik wrote: > On 03/17/15 12:17, Wei Liu wrote: > > On Fri, Mar 13, 2015 at 01:51:05PM +0100, Imre Palik wrote: > >> From: "Palik, Imre" > >> > >> With the current netback, the bandwidth limiter's parameters are only > >> settable during vif setup ti

Re: [Xen-devel] [RFC PATCH] xen-netback: making the bandwidth limiter runtime settable

2015-03-18 Thread Imre Palik
On 03/17/15 12:17, Wei Liu wrote: > On Fri, Mar 13, 2015 at 01:51:05PM +0100, Imre Palik wrote: >> From: "Palik, Imre" >> >> With the current netback, the bandwidth limiter's parameters are only >> settable during vif setup time. This patch register a watch on them, and >> thus makes them runtime

Re: [Xen-devel] [RFC PATCH] xen-netback: making the bandwidth limiter runtime settable

2015-03-17 Thread Ian Campbell
On Fri, 2015-03-13 at 13:51 +0100, Imre Palik wrote: [...] > diff --git a/drivers/net/xen-netback/interface.c > b/drivers/net/xen-netback/interface.c > index 3aa8648..34d8038 100644 > --- a/drivers/net/xen-netback/interface.c > +++ b/drivers/net/xen-netback/interface.c > @@ -463,6 +463,7 @@ int xe

Re: [Xen-devel] [RFC PATCH] xen-netback: making the bandwidth limiter runtime settable

2015-03-17 Thread Wei Liu
On Fri, Mar 13, 2015 at 01:51:05PM +0100, Imre Palik wrote: > From: "Palik, Imre" > > With the current netback, the bandwidth limiter's parameters are only > settable during vif setup time. This patch register a watch on them, and > thus makes them runtime changeable. > > When the watch fires,

[Xen-devel] [RFC PATCH] xen-netback: making the bandwidth limiter runtime settable

2015-03-13 Thread Imre Palik
From: "Palik, Imre" With the current netback, the bandwidth limiter's parameters are only settable during vif setup time. This patch register a watch on them, and thus makes them runtime changeable. When the watch fires, the timer is reset. The timer's mutex is used for fencing the change. Cc