diff options
| author | Andy Zhou <azhou@ovn.org> | 2016-06-17 15:41:26 -0700 |
|---|---|---|
| committer | Andy Zhou <azhou@ovn.org> | 2016-07-05 16:32:23 -0700 |
| commit | 1b870ac0ca7d9e7a4de307fce480d915e69d3c6e (patch) | |
| tree | c4088c311ab5b4734d1ab817f12434b1a419e5a9 /tests/test-rcu.c | |
| parent | c2715e8d6354e4b1c7159fe69faa9ff2f31ee7db (diff) | |
| download | openvswitch-1b870ac0ca7d9e7a4de307fce480d915e69d3c6e.tar.gz | |
lib: Remove extra API dependency for ovs_thread_create()
When calling ovs_thread_create() without calling fatal_signal_init()
first, ovs_thread_create() some times asserts. This dependency is
subtle and not very obvious.
The root cause seems to be that, within ovs_thread_create(), the
multi-threaded state is declared before all initializations are done.
Signed-off-by: Andy Zhou <azhou@ovn.org>
Acked-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'tests/test-rcu.c')
| -rw-r--r-- | tests/test-rcu.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tests/test-rcu.c b/tests/test-rcu.c index e66367e83..965f3c49f 100644 --- a/tests/test-rcu.c +++ b/tests/test-rcu.c @@ -39,7 +39,6 @@ test_rcu_quiesce(int argc OVS_UNUSED, char *argv[] OVS_UNUSED) { pthread_t quiescer; - fatal_signal_init(); quiescer = ovs_thread_create("quiescer", quiescer_main, NULL); /* This is the main thread of the process. After spawning its first |