On Fri, Oct 16, 2015 at 08:22:06AM -0700, Ben Pfaff wrote:
> On Fri, Oct 16, 2015 at 07:50:48PM +0900, Simon Horman wrote:
> > This corrects the parsing of 'fields' specified for groups on
> > the command line. 'fields' may be used in conjunction with the
> > Netronome selection method extension to describe which fields of
> > the flow should be used as by the selection method.
> >
> > This patch corrects two problems with the current implementation
> > as compared to the documentation in the ovs-ofctl man page.
> > * Allows parsing of more than one field
> > * Allows parsing of masks for fields
> >
> > Fixes: 18ac06d3546e ("ofp-util: Encoding and decoding of (draft) OpenFlow
> > 1.5 group messages.")
> > Signed-off-by: Simon Horman <[email protected]>
>
> I wrote a patch yesterday that did the same thing, but now I see that I
> failed to post it. I've posted it now. Can you take a look at it and
> compare the effects?
> http://openvswitch.org/pipermail/dev/2015-October/061318.html
Funny that we were both looking at that problem at about the same time.
Your approach looks good to me and I have Acked it accordingly.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
