On Thu, Sep 27, 2012 at 12:39:03PM -0700, Ben Pfaff wrote:
> On Thu, Sep 27, 2012 at 03:38:36PM +0900, Simon Horman wrote:
> > On Wed, Sep 26, 2012 at 09:54:34AM -0700, Ben Pfaff wrote:
> > > On Wed, Sep 26, 2012 at 11:32:35AM +0900, Simon Horman wrote:
> > > > Based heavily on work by Isaku Yamahata <[email protected]>
> > > > 
> > > > Cc: Isaku Yamahata <[email protected]>
> > > > Signed-off-by: Simon Horman <[email protected]>
> > > 
> > > It looks good to me except that I don't see any documentation in
> > > ovs-ofctl(8).
> > 
> > Sure I can add that, but I think the feature can only be used in
> > conjunction with OpenFlow 1.1+ and there is currently no way to enable that
> > Open Flow version in the code in git.
> 
> I think that it will still work by translating each set_field into one
> or more NXAST_REG_LOAD actions.

Good point. I will send a revised patch with some documentation in ovs-ofctl(8).
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to