[ovs-git] Open vSwitch: openflow: New Nicira Extended PACKET_IN format. (master)

dev at openvswitch.org dev at openvswitch.org
Tue Jan 10 22:31:48 UTC 2012


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, master has been updated
       via  548349603423bea41ec2989583231c1a117f6a9e (commit)
       via  999fba59afd9c8eef30d30a6fd2f490b85c24665 (commit)
       via  102ce76655b4f99f7e3697011ca983c617a9f5ee (commit)
       via  5d6c3af0fbff6c67f36c34f6e4c885f54de1bfe3 (commit)
       via  c97664b30f65d3a087076cb33242ffe2cc52a814 (commit)
       via  65120a8a4ec658f4318ca3ac6159e27f584bca8e (commit)
       via  3e3252fa1d7e057463ddbd66dab84f0d02ef0f1d (commit)
       via  29ebe8803c3a1ef03cbc92fbc352a2cdce5a8f55 (commit)
       via  15056dc8aa9e62a96c8f9b9a6e2c90d086e3a01d (commit)
       via  1eb85ef5407e27b35dcec8a34eda23d3e39ddaa6 (commit)
       via  e50abca5be5f0682012f6c0ef9b010708e56be07 (commit)
       via  897a8e0791171c8cb71e27c31d159a08205e5741 (commit)
       via  2f4ca41b9c771dc09e6130e0ff5ecf64acd79c0f (commit)
       via  ec14ac26857f49968e48fa7b1ad85a07c1bfcc8d (commit)
       via  c499c75db61f296041d7c39878c4f8cfea7671d5 (commit)
       via  2bcf7df6f7a87d8940deff6fbbc359b0a310a74a (commit)
       via  6aa728e0efd1ed300c707af8fb54f9f3acacda10 (commit)
       via  17f5d263877b25b42cda73b2fc96a8825ec81b78 (commit)
      from  715a77b74caf22e38d1f232d1cc45036b9b83e62 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 548349603423bea41ec2989583231c1a117f6a9e
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=548349603423bea41ec2989583231c1a117f6a9e
Author: Ethan Jackson <ethan at nicira.com>
		
openflow: New Nicira Extended PACKET_IN format.
		
The new PACKET_IN format implemented in this patch includes flow
metadata such as the cookie, table_id, and registers.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 999fba59afd9c8eef30d30a6fd2f490b85c24665
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=999fba59afd9c8eef30d30a6fd2f490b85c24665
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto-dpif: Implement PACKET_IN in userspace.
		
In future patches, PACKET_IN messages will include meta-data which
is only available in userspace during action translation.  Either,
this data needs to be stored until it's required by a userspace
datapath action, or the PACKET_IN messages must be sent at the time
the data is available.  This patch implements the latter.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 102ce76655b4f99f7e3697011ca983c617a9f5ee
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=102ce76655b4f99f7e3697011ca983c617a9f5ee
Author: Ethan Jackson <ethan at nicira.com>
		
nx-match: New function nx_pull_match_loose().
		
Future patches will need the ability to skip over unsupported NXM
headers.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 5d6c3af0fbff6c67f36c34f6e4c885f54de1bfe3
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=5d6c3af0fbff6c67f36c34f6e4c885f54de1bfe3
Author: Ethan Jackson <ethan at nicira.com>
		
flow: Create new flow_metadata structure for packet_in messages.
		
This will ease the implementation of future patches.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit c97664b30f65d3a087076cb33242ffe2cc52a814
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=c97664b30f65d3a087076cb33242ffe2cc52a814
Author: Ethan Jackson <ethan at nicira.com>
		
packets: New packet_set_*() helper functions.
		
This commit pulls code used to modify L3 and L4 header fields
from dp_netdev into the packet library.  An additional user will
be added in a future commit.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 65120a8a4ec658f4318ca3ac6159e27f584bca8e
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=65120a8a4ec658f4318ca3ac6159e27f584bca8e
Author: Ethan Jackson <ethan at nicira.com>
		
ofputil: New function ofputil_decode_packet_in().
		
Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 3e3252fa1d7e057463ddbd66dab84f0d02ef0f1d
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=3e3252fa1d7e057463ddbd66dab84f0d02ef0f1d
Author: Ethan Jackson <ethan at nicira.com>
		
ofp-util: Don't use ofpbuf in ofputil_packet_in struct.
		
This will make the memory ownership clearer in future patches which
make more extensive use of ofputil_packet_in.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 29ebe8803c3a1ef03cbc92fbc352a2cdce5a8f55
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=29ebe8803c3a1ef03cbc92fbc352a2cdce5a8f55
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto: Always clone packets in PACKET_IN message.
		
This patch removes an optimization which significantly complicates
the code in ways which would get worse in future patches if not
removed.  Furthermore, future patches will have fewer cases which
can take advantage of the optimization further mitigating its
justification.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 15056dc8aa9e62a96c8f9b9a6e2c90d086e3a01d
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=15056dc8aa9e62a96c8f9b9a6e2c90d086e3a01d
Author: Ethan Jackson <ethan at nicira.com>
		
tests: Unit test PACKET_IN.
		
Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 1eb85ef5407e27b35dcec8a34eda23d3e39ddaa6
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=1eb85ef5407e27b35dcec8a34eda23d3e39ddaa6
Author: Ethan Jackson <ethan at nicira.com>
		
ovs-ofctl: Support daemonization for monitor and snoop.
		
This will ease implementation of future unit tests.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit e50abca5be5f0682012f6c0ef9b010708e56be07
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e50abca5be5f0682012f6c0ef9b010708e56be07
Author: Ethan Jackson <ethan at nicira.com>
		
ofp-print: Print checksum in ofp_packet_to_string().
		
This will improve the unit tests of future patches.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 897a8e0791171c8cb71e27c31d159a08205e5741
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=897a8e0791171c8cb71e27c31d159a08205e5741
Author: Ethan Jackson <ethan at nicira.com>
		
ofp-print: Remove tcpdump from ofp_packet_to_string().
		
Instead this patch uses flow_format() which gives very similar
output.  This patch will improve the reliability of unit tests in
future patches which rely on the results of ofp_packet_to_string().

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 2f4ca41b9c771dc09e6130e0ff5ecf64acd79c0f
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=2f4ca41b9c771dc09e6130e0ff5ecf64acd79c0f
Author: Ethan Jackson <ethan at nicira.com>
		
packets: Mask out CFI bit in eth_push_vlan().
		
We should never push a VLAN tag with the CFI bit set.  This patch
defensively enforces this invariant.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit ec14ac26857f49968e48fa7b1ad85a07c1bfcc8d
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=ec14ac26857f49968e48fa7b1ad85a07c1bfcc8d
Author: Ethan Jackson <ethan at nicira.com>
		
pktbuf: Directly use pointers in pktbuf_save().
		
In future patches, directly using a void * pointer in the
pktbuf_save() definition will simplify the code.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit c499c75db61f296041d7c39878c4f8cfea7671d5
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=c499c75db61f296041d7c39878c4f8cfea7671d5
Author: Ethan Jackson <ethan at nicira.com>
		
ofp-print: Remove vestigial 'total_len' argument.
		
ofp_print_packet() and ofp_packet_to_string() don't use the
'total_len' argument which they require callers to supply.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 2bcf7df6f7a87d8940deff6fbbc359b0a310a74a
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=2bcf7df6f7a87d8940deff6fbbc359b0a310a74a
Author: Ethan Jackson <ethan at nicira.com>
		
flow: Improve flow_format() output.
		
I find this significantly easier to read.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 6aa728e0efd1ed300c707af8fb54f9f3acacda10
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=6aa728e0efd1ed300c707af8fb54f9f3acacda10
Author: Ethan Jackson <ethan at nicira.com>
		
ofpbuf: Maintain header pointers in clone functions.
		
Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 17f5d263877b25b42cda73b2fc96a8825ec81b78
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=17f5d263877b25b42cda73b2fc96a8825ec81b78
Author: Ethan Jackson <ethan at nicira.com>
		
classifier: Update cls_rule_set_in_port() prototype.
		
Signed-off-by: Ethan Jackson <ethan at nicira.com>


-----------------------------------------------------------------------

Summary of changes:
 INSTALL.Linux                 |    5 -
 NEWS                          |    2 +
 include/openflow/nicira-ext.h |   67 +++++++++++-
 lib/classifier.h              |    2 +-
 lib/dpif-netdev.c             |  122 ++------------------
 lib/dpif.c                    |    3 +-
 lib/flow.c                    |   47 +++++---
 lib/flow.h                    |   14 +++
 lib/learning-switch.c         |    2 +
 lib/nx-match.c                |   59 +++++++---
 lib/nx-match.h                |    3 +
 lib/odp-util.c                |   15 +---
 lib/odp-util.h                |    1 -
 lib/ofp-print.c               |  166 ++++++++++++++-------------
 lib/ofp-print.h               |    4 +-
 lib/ofp-util.c                |  214 +++++++++++++++++++++++++++++------
 lib/ofp-util.h                |   24 ++++-
 lib/ofpbuf.c                  |   25 ++++-
 lib/packets.c                 |  105 +++++++++++++++++-
 lib/packets.h                 |    4 +
 manpages.mk                   |    2 +
 ofproto/connmgr.c             |   60 ++++++-----
 ofproto/connmgr.h             |    5 +-
 ofproto/ofproto-dpif.c        |  254 +++++++++++++++++++----------------------
 ofproto/ofproto.c             |   27 +++++
 ofproto/pktbuf.c              |   15 ++-
 ofproto/pktbuf.h              |    4 +-
 tests/ofp-print.at            |   39 +++++-
 tests/ofproto-dpif.at         |  169 +++++++++++++++++++++++++++
 tests/ovs-ofctl.at            |   17 +++-
 tests/test-flows.c            |    2 +-
 utilities/ovs-ofctl.8.in      |   33 ++++++
 utilities/ovs-ofctl.c         |  120 ++++++++++++++++++-
 33 files changed, 1155 insertions(+), 476 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list