[ovs-dev] [PATCH v6 0/3] set-field action preliminary support

Simon Horman horms at verge.net.au
Thu Sep 20 04:17:17 UTC 2012


Hi Ben,

On Thu, Sep 13, 2012 at 09:45:09PM +0900, Simon Horman wrote:
> Hi,
> 
> This series adds preliminary support for set-field actions.
> I have picked up this series from Yamahata-san on his request.
> It is based on the current master branch.
> 
> Changes for each patch are noted inline.
> I believe this series addresses raised by Ben Pfaff in respect to
> v5 of this series.

Thanks for merging the first patch of this series.
Could you please consider the other two too?

> 
> Availability
> ------------
> 
> git://github.com/horms/openvswitch.git devel/set-field
> 
> Head commit, c6fc664af022cc50cb00ca7574cc8595675d5e64
> ("ofp-actions: Add parsing of set_field actions")
> 
> Patch List
> ----------
> 
> Patch one has been updated
> [PATCH 1/3] ofp-actions: helper functions for of12 set-field action
> 
> Patch two is unchanged
> [PATCH 2/3] ofp-actions: plumb of12 actions support
> 
> The third patch is new (though based on old code from Yamahata-san)
> [PATCH 3/3] ofp-actions: Add parsing of set_field actions
> 
> I have omitted the following patch which i consider to be still under
> discussion:
> 
> [RFC] ofp-actions: Check pre-requisists of set-fiel
> 
> Overall Diffstat
> ----------------
> 
>  include/openflow/openflow-1.2.h |    2 
>  lib/nx-match.c                  |  132 ++++++++++++++++++++++++++++--
>  lib/nx-match.h                  |    2 
>  lib/ofp-actions.c               |  101 ++++++++++++++++++++++
>  lib/ofp-actions.h               |    6 +
>  lib/ofp-parse.c                 |   47 ++++++++++
>  lib/ofp-util.c                  |    7 +
>  lib/ofp-util.def                |    5 -
>  tests/ofp-print.at              |   64 ++++++++++++++
>  utilities/ovs-ofctl.c           |    3 
>  10 files changed, 351 insertions(+), 18 deletions(-)
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
> 



More information about the dev mailing list