[ovs-git] Open vSwitch: ofproto-dpif: Simplify ofproto_receive(). (master)

dev at openvswitch.org dev at openvswitch.org
Thu May 9 16:55:13 UTC 2013

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  3abe1835dbf92fb770d63a9754c77285e3994bd9 (commit)
       via  2301f7ebc15e7f46a73d5c4638eec221cc9d57d4 (commit)
       via  e3d8644df32fc4664970ec5e02cffe9fe742f681 (commit)
      from  2db15dc5eb0ed2137fdb47a08a00c9f42da8d87e (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 3abe1835dbf92fb770d63a9754c77285e3994bd9
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=3abe1835dbf92fb770d63a9754c77285e3994bd9
Author: Ben Pfaff <blp at nicira.com>
ofproto-dpif: Simplify ofproto_receive().
The tunnel and non-tunnel paths were pretty much the same anyway, so this
commit simplifies by merging them.

Signed-off-by: Ben Pfaff <blp at nicira.com>

commit 2301f7ebc15e7f46a73d5c4638eec221cc9d57d4
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=2301f7ebc15e7f46a73d5c4638eec221cc9d57d4
Author: Ben Pfaff <blp at nicira.com>
tunnel: Make tnl_port_receive() parameter 'const'.
This function no longer has much need to modify its argument, because the
caller can now easily do the modification itself, so this commit makes
that change.

Signed-off-by: Ben Pfaff <blp at nicira.com>

commit e3d8644df32fc4664970ec5e02cffe9fe742f681
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e3d8644df32fc4664970ec5e02cffe9fe742f681
Author: Ben Pfaff <blp at nicira.com>
ofproto-dpif: Make ofport_dpif_cast() internally consistent.
This function's assertion would dereference a null pointer given a null
'ofport' argument, but its return statement checked for a null pointer
argument.  This commit fixes the inconsistency in favor of supporting
null pointer arguments.  (I discovered this problem while writing a piece
of code that wanted support for a null pointer argument, otherwise I would
resolve the inconsistency in the other direction.)

Signed-off-by: Ben Pfaff <blp at nicira.com>


Summary of changes:
 ofproto/ofproto-dpif.c |   78 +++++++++++++++++++++---------------------------
 ofproto/tunnel.c       |   16 +++-------
 ofproto/tunnel.h       |    2 +-
 3 files changed, 40 insertions(+), 56 deletions(-)

Open vSwitch

More information about the git mailing list