[ovs-git] Open vSwitch: connmgr: Remove now-unused function connmgr_broadcast(). (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  076f4c1ea3f551a6a9bf383ddc689f9cbc07b25b (commit)
       via  0b0517ef142805bf7691091c1eaafa10a669b06b (commit)
       via  2ed1202f141ecbda901c6531829273ea2ad8f1c8 (commit)
       via  d8653c386aa8bfaed60863710da6d785797d530f (commit)
       via  2a6dab2f80f0f4734142cf5a5c11643f1577b31f (commit)
      from  d8965755e707d97a515359a3d3732c46930d5350 (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 076f4c1ea3f551a6a9bf383ddc689f9cbc07b25b
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=076f4c1ea3f551a6a9bf383ddc689f9cbc07b25b
Author: Ben Pfaff <blp at nicira.com>
		
connmgr: Remove now-unused function connmgr_broadcast().
		
Signed-off-by: Ben Pfaff <blp at nicira.com>


commit 0b0517ef142805bf7691091c1eaafa10a669b06b
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=0b0517ef142805bf7691091c1eaafa10a669b06b
Author: Ben Pfaff <blp at nicira.com>
		
fail-open: Use connmgr_send_packet_in() instead of connmgr_broadcast().
		
Otherwise even controllers that should not receive any packet-ins (via
enable-async-messages=false) still receive the packet-ins that probe for
a controller being up when we're in fail-open.

Bug #9964.
Reported-by: James Schmidt <jschmidt at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


commit 2ed1202f141ecbda901c6531829273ea2ad8f1c8
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=2ed1202f141ecbda901c6531829273ea2ad8f1c8
Author: Ben Pfaff <blp at nicira.com>
		
ofproto: connmgr_send_packet_in() doesn't need buffer_id and total_len.
		
Trying to add a new caller for connmgr_send_packet_in(), I wasn't sure
what to put in these members.  Investigating, I saw that the function
didn't really need them, so this commit clears that up.

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


commit d8653c386aa8bfaed60863710da6d785797d530f
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=d8653c386aa8bfaed60863710da6d785797d530f
Author: Ben Pfaff <blp at nicira.com>
		
connmgr: Drop 'flow' parameter from connmgr_send_packet_in().
		
Only 'flow->in_port' was used, which was redundant with pin->fmd.in_port.

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


commit 2a6dab2f80f0f4734142cf5a5c11643f1577b31f
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=2a6dab2f80f0f4734142cf5a5c11643f1577b31f
Author: Ben Pfaff <blp at nicira.com>
		
ofp-util: Fix typo in comment.
		
The ofp_packet_in reasons are OFPR_*, not OFPRR_*.  (Duh.)

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


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

Summary of changes:
 lib/ofp-util.h         |    2 +-
 ofproto/connmgr.c      |   49 ++++++++++++-----------------------------------
 ofproto/connmgr.h      |    5 +--
 ofproto/fail-open.c    |   16 +++++++++-----
 ofproto/ofproto-dpif.c |    8 +-----
 5 files changed, 28 insertions(+), 52 deletions(-)


hooks/post-receive
-- 
Open vSwitch