[ovs-git] Open vSwitch: datapath-protocol: Rename to <linux/openvswitch.h>. (master)

dev at openvswitch.org dev at openvswitch.org
Wed Oct 12 23:42:37 UTC 2011


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  077257b83c68a36ea86f2d21c8395f60df710c21 (commit)
       via  9ea0bccc83a84458d934966eae15c78f3ee2bac8 (commit)
       via  09ded0ad489630236210a02f2d315816e558d701 (commit)
       via  13c0dbe79a50c412721a5df697c6a6303b22d2da (commit)
       via  5fa270099ab418ed0063144777ec76716dd3a207 (commit)
       via  ea36840fa4d6e9ce1e29fa8ce148edd3caef0072 (commit)
       via  98403001ece61cbf783297c467a06032b200b8d0 (commit)
       via  69ebca1e35a39d75032f3d8092da563f60673110 (commit)
       via  0e70cdcb8ddeb20417e682fd63769f9d16601b07 (commit)
      from  89a1c6d0a5aaa879769a022a71c7cee522013be2 (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 077257b83c68a36ea86f2d21c8395f60df710c21
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=077257b83c68a36ea86f2d21c8395f60df710c21
Author: Ben Pfaff <blp at nicira.com>
		
datapath-protocol: Rename to <linux/openvswitch.h>.
		
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Jesse Gross <jesse at nicira.com>
Bug #7559.


commit 9ea0bccc83a84458d934966eae15c78f3ee2bac8
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=9ea0bccc83a84458d934966eae15c78f3ee2bac8
Author: Ben Pfaff <blp at nicira.com>
		
datapath-protocol: Use Linux kernel types directly.
		
We want datapath-protocol.h to be acceptable as a Linux kernel header, so
it must use Linux kernel types and must not have references to Open vSwitch
symbols or header files.  This commit primarily makes that change to
datapath-protocol.h.

At the same time, at least for now we also want datapath-protocol.h to be
usable on non-Linux platforms, so we need some kind of compatiblity.  Thus,
this commit also introduces a <linux/types.h> header file that defines the
necessary Linux kernel types on non-Linux platforms.

In turn, this requires openvswitch/types.h to use the Linux types directly
for ovs_be<N>; otherwise, sparse complains because now __be<N> and
ovs_be<N> are incompatible from its perspective, so this commit makes that
change too.

I don't have a non-Linux kernel platform readily available, so I only
tested the non-Linux part of the linux/types.h substitute by forcing that
case to be triggered with #if 0.  It worked, except for errors in actual
Linux kernel headers included explicitly from OVS source files, so I think
it's likely to work in practice.

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


commit 09ded0ad489630236210a02f2d315816e558d701
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=09ded0ad489630236210a02f2d315816e558d701
Author: Ben Pfaff <blp at nicira.com>
		
datapath-protocol: Rename enums for consistency.
		
Most of the enum tags in this file are lowercased versions of the uppercase
enum prefixes (or slightly less abbreviated versions, e.g. "dp" becomes
"datapath").  This commit fixes up the others for consistency.

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


commit 13c0dbe79a50c412721a5df697c6a6303b22d2da
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=13c0dbe79a50c412721a5df697c6a6303b22d2da
Author: Ben Pfaff <blp at nicira.com>
		
datapath-protocol: Remove socket header #include.
		
It's not needed.

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


commit 5fa270099ab418ed0063144777ec76716dd3a207
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=5fa270099ab418ed0063144777ec76716dd3a207
Author: Ben Pfaff <blp at nicira.com>
		
datapath-protocol: Remove #include <linux/netlink.h>.
		
Bug #7559.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Jesse Gross <jesse at nicira.com>


commit ea36840fa4d6e9ce1e29fa8ce148edd3caef0072
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=ea36840fa4d6e9ce1e29fa8ce148edd3caef0072
Author: Ben Pfaff <blp at nicira.com>
		
datapath: Require explicit upcall_pid for new datapaths and vports.
		
This increases consistency with the OVS_ACTION_ATTR_USERSPACE action, which
also requires an explicit pid.

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


commit 98403001ece61cbf783297c467a06032b200b8d0
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=98403001ece61cbf783297c467a06032b200b8d0
Author: Ben Pfaff <blp at nicira.com>
		
datapath: Move Netlink PID for userspace actions from flows to actions.
		
Commit b063d9f06 "datapath: Use unicast Netlink sockets for upcalls" that
switched from multicast to unicast Netlink for sending upcalls added a
Netlink PID to each kernel flow, used by OVS_ACTION_ATTR_USERSPACE actions
within the flow as target.

This commit drops this per-flow PID in favor of a per-action PID, because
that is more flexible.  It does not yet make use of this additional
flexibility, so behavior should not change.

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


commit 69ebca1e35a39d75032f3d8092da563f60673110
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=69ebca1e35a39d75032f3d8092da563f60673110
Author: Ben Pfaff <blp at nicira.com>
		
dpif-linux: Avoid unaligned accesses to vport stats sent by the datapath.
		
Reported-by: Jesse Gross <jesse at nicira.com>


commit 0e70cdcb8ddeb20417e682fd63769f9d16601b07
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=0e70cdcb8ddeb20417e682fd63769f9d16601b07
Author: Ben Pfaff <blp at nicira.com>
		
dpif-linux: Use get_32aligned_u64() in an appropriate place.
		


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

Summary of changes:
 configure.ac                                       |    2 +-
 datapath/actions.c                                 |   26 +++-
 datapath/datapath.c                                |   95 ++++++------
 datapath/datapath.h                                |   10 +-
 datapath/flow.h                                    |    3 +-
 datapath/linux/compat/genetlink.inc                |    2 +-
 datapath/vport.h                                   |    2 +-
 include/automake.mk                                |    1 +
 include/linux/automake.mk                          |    3 +
 .../datapath-protocol.h => linux/openvswitch.h}    |  159 ++++++++++----------
 include/linux/types.h                              |   47 ++++++
 include/openvswitch/automake.mk                    |    1 -
 include/openvswitch/tunnel.h                       |    2 +-
 include/openvswitch/types.h                        |   12 +-
 lib/dpif-linux.c                                   |  144 ++++++------------
 lib/dpif-linux.h                                   |    9 +-
 lib/dpif-netdev.c                                  |   19 ++-
 lib/dpif-provider.h                                |   18 ++-
 lib/dpif.c                                         |   24 +++-
 lib/dpif.h                                         |    3 +-
 lib/netdev-vport.c                                 |   35 +++--
 lib/odp-util.c                                     |   84 +++++++---
 lib/odp-util.h                                     |    3 +-
 ofproto/ofproto-dpif.c                             |   61 +++++---
 24 files changed, 452 insertions(+), 313 deletions(-)
 create mode 100644 include/linux/automake.mk
 rename include/{openvswitch/datapath-protocol.h => linux/openvswitch.h} (81%)
 create mode 100644 include/linux/types.h


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list