[ovs-dev] [PATCH v2 10/13] ofp-util: Provide bucket list helper functions

Simon Horman simon.horman at netronome.com
Wed Nov 12 04:48:57 UTC 2014


On Tue, Nov 11, 2014 at 09:24:10AM -0800, Ben Pfaff wrote:
> On Tue, Nov 11, 2014 at 12:39:26PM +0900, Simon Horman wrote:
> > This is in preparation for supporting the bucket commands of
> > (draft) Open Flow 1.5 group mod messages.
> > 
> > ONF-JIRA: EXT-350
> > Signed-off-by: Simon Horman <simon.horman at netronome.com>
> > 
> > ---
> > v2
> > * As suggested by Ben Pfaff
> >   - Use ONF-JIRA: EXT-350 annotation in changelog
> 
> I'd add a few function-level comments.  In particular it looks like
> ofputil_bucket_clone_list() relies on the caller to have already
> initialized 'dest', which is worth calling out (or maybe changing).

Thanks, will do.

I will include documentation of the 'dest' requirement for
ofputil_bucket_clone_list(). I prefer not to change it as callers
(in a subsequent patch) rely on it appending to an existing list
rather than (re)initialising the list.

I'm sure there are other ways to handle this and I'm happy to explore
options if you wish.



More information about the dev mailing list