Re: [ovs-dev] [PATCH] ovs-ofctl: Handle any number of buckets in group statistics

2013-09-04 Thread Ben Pfaff
On Thu, Sep 05, 2013 at 09:13:42AM +0900, Simon Horman wrote: > On Wed, Sep 04, 2013 at 04:31:35PM -0700, Ben Pfaff wrote: > > On Mon, Sep 02, 2013 at 11:28:10AM +0900, Simon Horman wrote: > > > struct ofputil_group_stats has an arbitrary limit > > > of 16 buckets for which it can record statistics

Re: [ovs-dev] [PATCH] ovs-ofctl: Handle any number of buckets in group statistics

2013-09-04 Thread Simon Horman
On Wed, Sep 04, 2013 at 04:31:35PM -0700, Ben Pfaff wrote: > On Mon, Sep 02, 2013 at 11:28:10AM +0900, Simon Horman wrote: > > struct ofputil_group_stats has an arbitrary limit > > of 16 buckets for which it can record statistics. > > However the code does not appear to enforce this > > limit and i

Re: [ovs-dev] [PATCH] ovs-ofctl: Handle any number of buckets in group statistics

2013-09-04 Thread Ben Pfaff
On Mon, Sep 02, 2013 at 11:28:10AM +0900, Simon Horman wrote: > struct ofputil_group_stats has an arbitrary limit > of 16 buckets for which it can record statistics. > However the code does not appear to enforce this > limit and it seems to me that the code could overflow. > > This patch aims to r

[ovs-dev] [PATCH] ovs-ofctl: Handle any number of buckets in group statistics

2013-09-01 Thread Simon Horman
struct ofputil_group_stats has an arbitrary limit of 16 buckets for which it can record statistics. However the code does not appear to enforce this limit and it seems to me that the code could overflow. This patch aims to remove the arbitrary limit by removing the 'bucket_stats' field from struct