Hi Ben,

these patches are based on your of1.1 git branch
(21602cd6bdf056d24b2c6191d636bebcbb336ad9 "ofp-util: Work on decoding OF1.1
flow_mods.")

They are all compile tested only. I have also, using the last patch
which adds some hacks to allow ovs-vswtichd and ovs-controller to netogiate
an Open Flow 1.2 session, tested limited communication using a subset
of the Open Flow 1.2 messages that are supported as of this series.

To be clear, that testing does not cover all messages. And it uses
OVS for both encoding and decoding so there is ample scope for the
same incorrect assumptions to be made on both send and receive.

There are numerous changes since my previous posting (v2),
they are all noted inline in the changelogs of individual patches.

Some changes of note are:

* Take into account the size of ofp11_match_header when calculating
  the pad for OXM encoding.
  - [PATCH 02/34] nx-match: Take into account leading header when

* Do not add NXPIF_OPENFLOW12, there seems to be no need
  for an Open Flow 1.2 flow format
  - [PATCH 03/34] ofp-util: Add OFPUTIL_P_OF12 and NXFF_OPENFLOW12

* Correct ofputil_put_match() implementation (numerous times!)
  -  [PATCH 05/34] ofp-util: Add ofputil_put_match()

* Enhance ofpacts_to_openflow11 to encode instructions
  This isn't quite as exciting as it may sound as only
  OFPIT11_APPLY_ACTIONS is supported, but it does seem to
  be useful as is.
  - [PATCH 06/34] ofp-actions: Enhance ofpacts_to_openflow11 to encode

* Correct encoding of Flow Mod messages
  - [PATCH 10/34] ofp-util: Allow encoding of Open Flow 1.2 Flow Mo

* Correct encoding of Packet Out messages
  - PATCH 24/34] ofp-util: Allow encoding of Open Flow 1.1 and 1.2 Packet Out 
Message

* Add encoding/decoding of some statistics messages:
  - [PATCH 26/34] ofp-util: Allow encoding of Open Flow 1.1 and 1.2 messages
  - [PATCH 28/34] ofp-util: Allow decoding of Open Flow 1.1 and 1.2 Flow 
Statistics Request Messages
  - [PATCH 29/34]  ofp-util: Allow encoding of Open Flow 1.2 Flow Statistics 
Response Messageso
  - [PATCH 30/34] ofp-util: Allow decoding of Open Flow 1.2 Flow Statistics 
Response
    Messages
  - [PATCH 31/34] openflow: Rename ofp_aggregate_stats_reply as 
ofp10_aggregate_stats_reply
  - [PATCH 32/34] ofp-util: Allow decoding of Open Flow 1.1 & 1.2 Flow 
Aggregate Statistics Response Messages

I have made my patches available in the devel/of12 branch of
git://github.com/horms/openvswitch.git. The current head is
6214ddc9d58b6d41ad9d296255f1c8b735651d99.


_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to