[ovs-git] Open vSwitch: tunneling: Rely on protocol handles to parse ToS. (master)

[dev at openvswitch.org]

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  aae369c706cd56886f8560c43960110f79e062dc (commit)
      from  132b0dccf4a86614b468790dcc26d49fde4e08d9 (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 aae369c706cd56886f8560c43960110f79e062dc
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=aae369c706cd56886f8560c43960110f79e062dc
Author: Jesse Gross <jesse at nicira.com>
		
tunneling: Rely on protocol handles to parse ToS.
		
Currently the generic tunneling code assumes that on receive the
outer IP header is present on the packet and the protocol pointers
are initialized to it.  This knowledge is used in only one place,
which is to copy ECN bits from outer to inner IP packets.  These
assumptions are difficult to keep track of because the receive code
generally uses the protocol pointers for the inner packet.  A
number of bugs have resulted from incorrect assumptions, including
one fixed here about the location of a vlan header.  This drops the
convention and instead uses the protocol handles to supply the ToS,
which is less error-prone.

Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


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

Summary of changes:
 datapath/tunnel.c       |   40 +++++++++++++++++++++++-----------------
 datapath/tunnel.h       |    2 +-
 datapath/vport-capwap.c |    2 +-
 datapath/vport-gre.c    |    2 +-
 4 files changed, 26 insertions(+), 20 deletions(-)


hooks/post-receive
-- 
Open vSwitch