celix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [celix] Oipo commented on pull request #265: Bugfix/pstm deadlock work outside of locks
Date Wed, 01 Jul 2020 08:58:23 GMT

Oipo commented on pull request #265:
URL: https://github.com/apache/celix/pull/265#issuecomment-652289921


   It seems that the pstm_deadlock_tcp_test still uses zmq somehow:
   
   ```
   WARNING: ThreadSanitizer: data race (pid=23761)
     Write of size 8 at 0x7b04000014b0 by thread T2 (mutexes: read M280, write M147, write
M148, write M149):
       #0 socket <null> (libtsan.so.0+0x37f34)
       #1 <null> <null> (libzmq.so.5+0x6b5a8)
       #2 pubsub_zmqAdmin_setupTopicSender /home/oipo-unencrypted/Programming/celix-apache/bundles/pubsub/pubsub_admin_zmq/src/pubsub_zmq_admin.c:477
(libcelix_pubsub_admin_zmqd.so.1+0x2011a)
       #3 pstm_setupTopicSenderCallback /home/oipo-unencrypted/Programming/celix-apache/bundles/pubsub/pubsub_topology_manager/src/pubsub_topology_manager.c:886
(libcelix_pubsub_topology_managerd.so.1+0x1fd06)
       #4 serviceTracker_useHighestRankingServiceInternal /home/oipo-unencrypted/Programming/celix-apache/libs/framework/src/service_tracker.c:821
(libcelix_frameworkd.so.2+0xa2fb9)
       #5 celix_serviceTracker_useHighestRankingService /home/oipo-unencrypted/Programming/celix-apache/libs/framework/src/service_tracker.c:852
(libcelix_frameworkd.so.2+0xa33ac)
       #6 celix_bundleContext_useServiceWithOptions /home/oipo-unencrypted/Programming/celix-apache/libs/framework/src/bundle_context.c:784
(libcelix_frameworkd.so.2+0x6679d)
       #7 celix_bundleContext_useServiceWithId /home/oipo-unencrypted/Programming/celix-apache/libs/framework/src/bundle_context.c:731
(libcelix_frameworkd.so.2+0x65eca)
       #8 pstm_setupTopicSenders /home/oipo-unencrypted/Programming/celix-apache/bundles/pubsub/pubsub_topology_manager/src/pubsub_topology_manager.c:954
(libcelix_pubsub_topology_managerd.so.1+0x20bcf)
       #9 pstm_psaHandlingThread /home/oipo-unencrypted/Programming/celix-apache/bundles/pubsub/pubsub_topology_manager/src/pubsub_topology_manager.c:1104
(libcelix_pubsub_topology_managerd.so.1+0x23bbe)
       #10 <null> <null> (libtsan.so.0+0x2d1af)
   
     Previous write of size 8 at 0x7b04000014b0 by main thread:
       #0 pipe <null> (libtsan.so.0+0x3b8fc)
       #1 <null> <null> (libubsan.so.1+0x1d9f3)
       #2 void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test,
void>(testing::Test*, void (testing::Test::*)(), char const*) /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/src/gtest.cc:2433
(pstm_deadlock_tcp_testd+0x195a81)
       #3 void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*,
void (testing::Test::*)(), char const*) /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/src/gtest.cc:2469
(pstm_deadlock_tcp_testd+0x179d9e)
       #4 testing::TestInfo::Run() /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/src/gtest.cc:2690
(pstm_deadlock_tcp_testd+0xfcbd2)
       #5 testing::TestSuite::Run() /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/src/gtest.cc:2816
(pstm_deadlock_tcp_testd+0xffb55)
       #6 testing::internal::UnitTestImpl::RunAllTests() /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/src/gtest.cc:5338
(pstm_deadlock_tcp_testd+0x12ab32)
       #7 bool testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,
bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char
const*) /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/src/gtest.cc:2433
(pstm_deadlock_tcp_testd+0x19a7a8)
       #8 bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,
bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char
const*) /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/src/gtest.cc:2469
(pstm_deadlock_tcp_testd+0x17e989)
       #9 testing::UnitTest::Run() /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/src/gtest.cc:4925
(pstm_deadlock_tcp_testd+0x12026f)
       #10 RUN_ALL_TESTS() /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/include/gtest/gtest.h:2473
(pstm_deadlock_tcp_testd+0x1d2221)
       #11 main /home/oipo-unencrypted/Programming/celix-apache/build/_deps/googletest-src/googletest/src/gtest_main.cc:45
(pstm_deadlock_tcp_testd+0x1d2040)
   ```
   
   Ignore the "data race", as the race is introduced by the printf call that the sanitizer
introduces. The important thing here is that zmq is used.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



Mime
View raw message