ovs-rcu: Make ovsrcu_quiesce() flush the callback event set.
authorAlex Wang <alexw@nicira.com>
Tue, 9 Sep 2014 18:01:52 +0000 (11:01 -0700)
committerAlex Wang <alexw@nicira.com>
Fri, 12 Sep 2014 17:26:56 +0000 (10:26 -0700)
commit9c7e020fd60baa98528bd202caeb9e908b036216
tree9b1330a06f88963dcddd533fb4835ca2645c696d
parentb8f958eaf355c48aaa659b50d5f39d7f0f509436
ovs-rcu: Make ovsrcu_quiesce() flush the callback event set.

On current master, the per-thread callback event set is flushed
when ovsrcu_quiesce_start() is called or when the callback
event set is full.  For threads that only call 'ovsrcu_quiesce()'
to indicate quiescient state, their callback event set will not
be flushed for execution until the set is full.  And this could
take a very long time.

Theoretically, this should not be an issue, since rcu postponed
callback events should only free the old version of objects.
However, current ovs does not follow this rule, and some callback
events include other activities like unregistering the netdev
from global name-netdev map.  The delay of unregistering the netdev
(by threads that only calls ovsrcu_quiesce()) will prevent the
recreate of same netdev indefinitely.

As a short-term workaround, this commit makes every call to
ovsrcu_quiesce() flush the callback event set.  In the long run,
there will be a refactor of the use of ovs-rcu module, in which all
callback events only free the old version of objects.

Signed-off-by: Alex Wang <alexw@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
lib/ovs-rcu.c