[ovs-git] Open vSwitch: ofproto-dpif-upcall: Fix a bug. (master)

dev at openvswitch.org dev at openvswitch.org
Fri Apr 25 20:03:06 UTC 2014

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  6f12bda359fb13fb2c0d6f958f56956bb76e47d7 (commit)
      from  5c09b672114fab2357c8ce71d608d486eb5ae1ae (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 6f12bda359fb13fb2c0d6f958f56956bb76e47d7
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=6f12bda359fb13fb2c0d6f958f56956bb76e47d7
Author: Alex Wang <alexw at nicira.com>
ofproto-dpif-upcall: Fix a bug.
Commit 7d170098 (ofproto-dpif-upcall: Remove the flow_dumper thread.)
initialized the memory barrier inside the udpif_start_threads() function.
However, the udpif_start_threads() function does not check the number of
revalidator threads specified in udpif.  So, when the number is zero, it
causes the error in barrier initialization.  This could happen when the
other_config:flow-restore-wait is set and the udpif_flush() is called.

This commit fixes the issue, by checking the specified number of threads
in udpif_start_threads().

Reported-by: Gurucharan Shetty <gshetty at nicira.com>
Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Gurucharan Shetty <gshetty at nicira.com>


Summary of changes:
 ofproto/ofproto-dpif-upcall.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Open vSwitch

More information about the git mailing list