[ovs-git] Open vSwitch: datapath: Harmonize [get|set]_skb_csum_pointers(). (master)

dev at openvswitch.org dev at openvswitch.org
Thu Dec 16 17:42:27 UTC 2010


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  e0ce13c4dc34f44d81e0001882a0be88321b5c87 (commit)
       via  3d8981230bb0c9c11db70f1444e4940ac02d3888 (commit)
      from  49ad0403a79ed166fad156dd649f8f9d63fc9dc3 (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 e0ce13c4dc34f44d81e0001882a0be88321b5c87
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e0ce13c4dc34f44d81e0001882a0be88321b5c87
Author: Jesse Gross <jesse at nicira.com>
		
datapath: Harmonize [get|set]_skb_csum_pointers().
		
The functions to get and set the checksum pointers consistently across
different kernel versions had different interpretations of what the
csum_offset pointer was relative to, which is confusing, to say the least.
This makes the meaning be the same as skb->csum_offset in modern kernels
and updates the caller.  For a given function the results were consistent
across kernel versions and the callers knew what the meaning should be, so
this doesn't actually fix any bugs.

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


commit 3d8981230bb0c9c11db70f1444e4940ac02d3888
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=3d8981230bb0c9c11db70f1444e4940ac02d3888
Author: Jesse Gross <jesse at nicira.com>
		
tunneling: Refresh IP header pointer after update_header().
		
We were assuming that the call to update_header() to finalize tunnel
headers wouldn't cause the skb linear data area to be reallocated.
So far this hasn't been a problem but it's not, generally speaking,
a good assumption to make.  Therefore, refetch the pointer to the IP
header instead of carrying it across the call.

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


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

Summary of changes:
 datapath/checksum.h |    4 ++--
 datapath/datapath.c |    2 ++
 datapath/tunnel.c   |    1 +
 3 files changed, 5 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Open vSwitch




More information about the git mailing list