On Tue, Nov 12, 2013 at 7:53 AM, Ben Pfaff <b...@nicira.com> wrote: > On Fri, Nov 08, 2013 at 10:54:38AM -0800, Jarno Rajahalme wrote: >> Add DPIF-level support for meters. Allow meter_set to modify the meter >> configuration (e.g. set the burst size if unspecified). >> Summary: > > I think we need to get Jesse's view on the meter stuff. I'll review > the userspace API, if you like, or if you think that a userspace-only > implementation would be useful, but we need his input to get it into > the kernel.
The concern that I have is that I think the actual meter implementation needs to be better integrated with the rest of the kernel in order to be upstreamable. I'm a little nervous about defining an API until we have at least one implementation that could be used in practice. The other thing that I think would be good to talk about is how this relates to the existing QoS features and other possible ways that we might want to extend meters in the future. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev