On Fri, Sep 28, 2012 at 11:57:05AM +0900, Simon Horman wrote: > 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).
Thanks! _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
