Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15557 )

Change subject: gutil: bump up linux-syscall-support.h
......................................................................


Patch Set 3:

(1 comment)

> Patch Set 2:
>
> (1 comment)
>
> It looks like the other patch has an older version of 
> linux-syscall-support.h. It seems better to bring it up to date with upstream.
>
> TSAN error that caused the test failure is below. I couldn't find a related 
> JIRA but couldn't reproduce locally or see how it might've been caused by 
> this patch (maybe messing with futex did something?). Not sure how to proceed 
> with debugging this.
>
> WARNING: ThreadSanitizer: data race (pid=12896)
>   Write of size 8 at 0x7b2c000672b8 by main thread:
>     #0 operator delete(void*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/llvm-9.0.0.src/projects/compiler-rt/lib/tsan/rtl/tsan_new_delete.cc:126
>  (tablet_server_quiescing-itest+0x4f5119)
>     #1 
> std::__1::default_delete<kudu::ThreadPoolToken>::operator()(kudu::ThreadPoolToken*)
>  const 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2338:5
>  (libkudu_util.so+0x2f48b6)
>     #2 std::__1::unique_ptr<kudu::ThreadPoolToken, 
> std::__1::default_delete<kudu::ThreadPoolToken> 
> >::reset(kudu::ThreadPoolToken*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2593:7
>  (libkudu_util.so+0x2f481d)
>     #3 std::__1::unique_ptr<kudu::ThreadPoolToken, 
> std::__1::default_delete<kudu::ThreadPoolToken> >::~unique_ptr() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2547:19
>  (libkudu_util.so+0x2eeccb)
>     #4 kudu::ThreadPool::~ThreadPool() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/threadpool.cc:300:1 
> (libkudu_util.so+0x2eb29e)
>     #5 
> std::__1::default_delete<kudu::ThreadPool>::operator()(kudu::ThreadPool*) 
> const 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2338:5
>  (libkudu_util.so+0x23638e)
>     #6 std::__1::unique_ptr<kudu::ThreadPool, 
> std::__1::default_delete<kudu::ThreadPool> >::reset(kudu::ThreadPool*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2593:7
>  (libkudu_util.so+0x2362fd)
>     #7 std::__1::unique_ptr<kudu::ThreadPool, 
> std::__1::default_delete<kudu::ThreadPool> >::~unique_ptr() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2547:19
>  (libkudu_util.so+0x230d6b)
>     #8 kudu::DnsResolver::~DnsResolver() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/net/dns_resolver.cc:72:1
>  (libkudu_util.so+0x27ae63)
>     #9 
> std::__1::default_delete<kudu::DnsResolver>::operator()(kudu::DnsResolver*) 
> const 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2338:5
>  (libkudu_client.so+0x1eb22e)
>     #10 std::__1::unique_ptr<kudu::DnsResolver, 
> std::__1::default_delete<kudu::DnsResolver> >::reset(kudu::DnsResolver*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2593:7
>  (libkudu_client.so+0x1e20fd)
>     #11 kudu::client::KuduClient::Data::~Data() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/client/client-internal.cc:184:17
>  (libkudu_client.so+0x1dd71d)
>     #12 kudu::client::KuduClient::~KuduClient() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/client/client.cc:396:3 
> (libkudu_client.so+0x1981dc)
>     #13 
> std::__1::default_delete<kudu::client::KuduClient>::operator()(kudu::client::KuduClient*)
>  const 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2338:5
>  (libkudu_client.so+0x1bd11e)
>     #14 std::__1::__shared_ptr_pointer<kudu::client::KuduClient*, 
> std::__1::default_delete<kudu::client::KuduClient>, 
> std::__1::allocator<kudu::client::KuduClient> >::__on_zero_shared() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:3511:5
>  (libkudu_client.so+0x1bce62)
>     #15 std::__1::__shared_count::__release_shared() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:3415:9
>  (tablet_server_quiescing-itest+0x520b55)
>     #16 std::__1::__shared_weak_count::__release_shared() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:3457:27
>  (tablet_server_quiescing-itest+0x520af9)
>     #17 std::__1::shared_ptr<kudu::client::KuduClient>::~shared_ptr() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:4393:19
>  (tablet_server_quiescing-itest+0x520688)
>     #18 kudu::TestWorkload::~TestWorkload() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/integration-tests/test_workload.cc:96:1
>  (libitest_util.so+0xc1922)
>     #19 
> std::__1::default_delete<kudu::TestWorkload>::operator()(kudu::TestWorkload*) 
> const 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2338:5
>  (tablet_server_quiescing-itest+0x52edde)
>     #20 std::__1::unique_ptr<kudu::TestWorkload, 
> std::__1::default_delete<kudu::TestWorkload> >::reset(kudu::TestWorkload*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2593:7
>  (tablet_server_quiescing-itest+0x52ed4d)
>     #21 std::__1::unique_ptr<kudu::TestWorkload, 
> std::__1::default_delete<kudu::TestWorkload> >::~unique_ptr() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/memory:2547:19
>  (tablet_server_quiescing-itest+0x51d27b)
>     #22 
> kudu::itest::TServerQuiescingITest_TestDoesntAllowNewScansLeadersOnly_Test::TestBody()
>  
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/integration-tests/tablet_server_quiescing-itest.cc:305:1
>  (tablet_server_quiescing-itest+0x4f94b7)
>     #23 void 
> testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, 
> void>(testing::Test*, void (testing::Test::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2402:10
>  (libgmock.so+0x54f9f)
>     #24 void 
> testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, 
> void>(testing::Test*, void (testing::Test::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2438:14
>  (libgmock.so+0x54f9f)
>     #25 testing::Test::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2474:5
>  (libgmock.so+0x33f38)
>     #26 testing::TestInfo::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2656:11
>  (libgmock.so+0x351dc)
>     #27 testing::TestCase::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2774:28
>  (libgmock.so+0x35ef6)
>     #28 testing::internal::UnitTestImpl::RunAllTests() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:4649:43
>  (libgmock.so+0x423a6)
>     #29 bool 
> testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,
>  bool>(testing::internal::UnitTestImpl*, bool 
> (testing::internal::UnitTestImpl::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2402:10
>  (libgmock.so+0x55e4f)
>     #30 bool 
> testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,
>  bool>(testing::internal::UnitTestImpl*, bool 
> (testing::internal::UnitTestImpl::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2438:14
>  (libgmock.so+0x55e4f)
>     #31 testing::UnitTest::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:4257:10
>  (libgmock.so+0x41c61)
>     #32 RUN_ALL_TESTS() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/gtest/gtest.h:2233:46
>  (libkudu_test_main.so+0x47bb)
>     #33 main 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/test_main.cc:106:13 
> (libkudu_test_main.so+0x3636)
>
>   Previous read of size 8 at 0x7b2c000672b8 by thread T15:
>     #0 scoped_refptr<kudu::Histogram>::operator kudu::Histogram* 
> scoped_refptr<kudu::Histogram>::*() const 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/gutil/ref_counted.h:280:38
>  (tablet_server_quiescing-itest+0x527eb9)
>     #1 kudu::ThreadPool::DoSubmit(std::__1::function<void ()>, 
> kudu::ThreadPoolToken*) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/threadpool.cc:501:7 
> (libkudu_util.so+0x2ea20a)
>     #2 kudu::ThreadPool::Submit(std::__1::function<void ()>) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/threadpool.cc:408:10
>  (libkudu_util.so+0x2ebb12)
>     #3 kudu::DnsResolver::ResolveAddressesAsync(kudu::HostPort const&, 
> std::__1::vector<kudu::Sockaddr, std::__1::allocator<kudu::Sockaddr> >*, 
> kudu::Callback<void (kudu::Status const&)> const&) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/net/dns_resolver.cc:88:25
>  (libkudu_util.so+0x27b3a7)
>     #4 
> kudu::client::internal::RemoteTabletServer::InitProxy(kudu::client::KuduClient*,
>  kudu::Callback<void (kudu::Status const&)> const&) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/client/meta_cache.cc:135:33
>  (libkudu_client.so+0x235074)
>     #5 
> kudu::client::internal::MetaCacheServerPicker::PickLeader(kudu::Callback<void 
> (kudu::Status const&, kudu::client::internal::RemoteTabletServer*)> const&, 
> kudu::MonoTime const&) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/client/meta_cache.cc:465:11
>  (libkudu_client.so+0x23713d)
>     #6 kudu::rpc::RetriableRpc<kudu::client::internal::RemoteTabletServer, 
> kudu::tserver::WriteRequestPB, kudu::tserver::WriteResponsePB>::SendRpc() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/retriable_rpc.h:165:19
>  (libkudu_client.so+0x1870e5)
>     #7 kudu::rpc::RpcRetrier::DelayedRetryCb(kudu::rpc::Rpc*, kudu::Status 
> const&) /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/rpc.cc:81:10 
> (libkrpc.so+0x12d5b5)
>     #8 boost::_mfi::mf2<void, kudu::rpc::RpcRetrier, kudu::rpc::Rpc*, 
> kudu::Status const&>::operator()(kudu::rpc::RpcRetrier*, kudu::rpc::Rpc*, 
> kudu::Status const&) const 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/boost/bind/mem_fn_template.hpp:280:29
>  (libkrpc.so+0x12e03a)
>     #9 void boost::_bi::list3<boost::_bi::value<kudu::rpc::RpcRetrier*>, 
> boost::_bi::value<kudu::rpc::Rpc*>, boost::arg<1> 
> >::operator()<boost::_mfi::mf2<void, kudu::rpc::RpcRetrier, kudu::rpc::Rpc*, 
> kudu::Status const&>, boost::_bi::rrlist1<kudu::Status const&> 
> >(boost::_bi::type<void>, boost::_mfi::mf2<void, kudu::rpc::RpcRetrier, 
> kudu::rpc::Rpc*, kudu::Status const&>&, boost::_bi::rrlist1<kudu::Status 
> const&>&, int) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/boost/bind/bind.hpp:398:9
>  (libkrpc.so+0x12df72)
>     #10 void boost::_bi::bind_t<void, boost::_mfi::mf2<void, 
> kudu::rpc::RpcRetrier, kudu::rpc::Rpc*, kudu::Status const&>, 
> boost::_bi::list3<boost::_bi::value<kudu::rpc::RpcRetrier*>, 
> boost::_bi::value<kudu::rpc::Rpc*>, boost::arg<1> > 
> >::operator()<kudu::Status const&>(kudu::Status const&) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/boost/bind/bind.hpp:1234:16
>  (libkrpc.so+0x12de98)
>     #11 
> boost::detail::function::void_function_obj_invoker1<boost::_bi::bind_t<void, 
> boost::_mfi::mf2<void, kudu::rpc::RpcRetrier, kudu::rpc::Rpc*, kudu::Status 
> const&>, boost::_bi::list3<boost::_bi::value<kudu::rpc::RpcRetrier*>, 
> boost::_bi::value<kudu::rpc::Rpc*>, boost::arg<1> > >, void, kudu::Status 
> const&>::invoke(boost::detail::function::function_buffer&, kudu::Status 
> const&) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/boost/function/function_template.hpp:159:11
>  (libkrpc.so+0x12dc33)
>     #12 boost::function1<void, kudu::Status const&>::operator()(kudu::Status 
> const&) const 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/boost/function/function_template.hpp:770:14
>  (libkrpc.so+0x10ce54)
>     #13 kudu::rpc::DelayedTask::TimerHandler(ev::timer&, int) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/reactor.cc:747:5 
> (libkrpc.so+0x1075b4)
>     #14 void ev::base<ev_timer, 
> ev::timer>::method_thunk<kudu::rpc::DelayedTask, 
> &(kudu::rpc::DelayedTask::TimerHandler(ev::timer&, int))>(ev_loop*, 
> ev_timer*, int) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/ev++.h:479:7
>  (libkrpc.so+0x114cba)
>     #15 ev_invoke_pending 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/libev-4.20/ev.c:3155:11
>  (libev.so.4+0x9a52)
>     #16 kudu::rpc::ReactorThread::InvokePendingCb(ev_loop*) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/reactor.cc:197:3 
> (libkrpc.so+0x103695)
>     #17 ev_run 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/libev-4.20/ev.c:3555:7
>  (libev.so.4+0xabfc)
>     #18 ev::loop_ref::run(int) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/ev++.h:211:7
>  (libkrpc.so+0x10c4e8)
>     #19 kudu::rpc::ReactorThread::RunThread() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/reactor.cc:498:9 
> (libkrpc.so+0x10685b)
>     #20 kudu::rpc::ReactorThread::Init()::$_0::operator()() const 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/reactor.cc:189:48 
> (libkrpc.so+0x1097f1)
>     #21 
> decltype(std::__1::forward<kudu::rpc::ReactorThread::Init()::$_0&>(fp)()) 
> std::__1::__invoke<kudu::rpc::ReactorThread::Init()::$_0&>(kudu::rpc::ReactorThread::Init()::$_0&)
>  
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/type_traits:3530:1
>  (libkrpc.so+0x109781)
>     #22 void 
> std::__1::__invoke_void_return_wrapper<void>::__call<kudu::rpc::ReactorThread::Init()::$_0&>(kudu::rpc::ReactorThread::Init()::$_0&)
>  
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/__functional_base:348:9
>  (libkrpc.so+0x109711)
>     #23 
> std::__1::__function::__alloc_func<kudu::rpc::ReactorThread::Init()::$_0, 
> std::__1::allocator<kudu::rpc::ReactorThread::Init()::$_0>, void 
> ()>::operator()() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/functional:1533:16
>  (libkrpc.so+0x1096d1)
>     #24 std::__1::__function::__func<kudu::rpc::ReactorThread::Init()::$_0, 
> std::__1::allocator<kudu::rpc::ReactorThread::Init()::$_0>, void 
> ()>::operator()() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/functional:1707:12
>  (libkrpc.so+0x10876d)
>     #25 std::__1::__function::__value_func<void ()>::operator()() const 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/functional:1860:16
>  (libkrpc.so+0x100744)
>     #26 std::__1::function<void ()>::operator()() const 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/c++/v1/functional:2419:12
>  (libkrpc.so+0xfe469)
>     #27 kudu::Thread::SuperviseThread(void*) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/thread.cc:674:3 
> (libkudu_util.so+0x2d473a)
>
>   As if synchronized via sleep:
>     #0 usleep 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/llvm-9.0.0.src/projects/compiler-rt/lib/tsan/rtl/tsan_interceptors.cc:354
>  (tablet_server_quiescing-itest+0x45a342)
>     #1 sq_stop 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/squeasel-030ccce87359d892e22fb368c5fc5b75d9a2a5f7/squeasel.c:4960:12
>  (libserver_process.so+0x111229)
>     #2 kudu::Webserver::Stop() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/server/webserver.cc:392:5
>  (libserver_process.so+0xe4db2)
>     #3 kudu::server::ServerBase::ShutdownImpl() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/server/server_base.cc:719:18
>  (libserver_process.so+0xb9782)
>     #4 kudu::server::ServerBase::Shutdown() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/server/server_base.h:145:5
>  (libkserver.so+0xb389)
>     #5 kudu::kserver::KuduServer::Shutdown() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/kserver/kserver.cc:199:15
>  (libkserver.so+0xa6b3)
>     #6 kudu::tserver::TabletServer::ShutdownImpl() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/tserver/tablet_server.cc:167:17
>  (libtserver.so+0x16e6ce)
>     #7 kudu::tserver::TabletServer::Shutdown() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/tserver/tablet_server.h:64:5
>  (libtserver.so+0x1702b9)
>     #8 kudu::tserver::MiniTabletServer::Shutdown() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/tserver/mini_tablet_server.cc:116:14
>  (libtserver.so+0x10d843)
>     #9 
> kudu::itest::TServerQuiescingITest_TestDoesntAllowNewScansLeadersOnly_Test::TestBody()
>  
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/integration-tests/tablet_server_quiescing-itest.cc:301:7
>  (tablet_server_quiescing-itest+0x4f941f)
>     #10 void 
> testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, 
> void>(testing::Test*, void (testing::Test::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2402:10
>  (libgmock.so+0x54f9f)
>     #11 void 
> testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, 
> void>(testing::Test*, void (testing::Test::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2438:14
>  (libgmock.so+0x54f9f)
>     #12 testing::Test::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2474:5
>  (libgmock.so+0x33f38)
>     #13 testing::TestInfo::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2656:11
>  (libgmock.so+0x351dc)
>     #14 testing::TestCase::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2774:28
>  (libgmock.so+0x35ef6)
>     #15 testing::internal::UnitTestImpl::RunAllTests() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:4649:43
>  (libgmock.so+0x423a6)
>     #16 bool 
> testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,
>  bool>(testing::internal::UnitTestImpl*, bool 
> (testing::internal::UnitTestImpl::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2402:10
>  (libgmock.so+0x55e4f)
>     #17 bool 
> testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,
>  bool>(testing::internal::UnitTestImpl*, bool 
> (testing::internal::UnitTestImpl::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2438:14
>  (libgmock.so+0x55e4f)
>     #18 testing::UnitTest::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:4257:10
>  (libgmock.so+0x41c61)
>     #19 RUN_ALL_TESTS() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/gtest/gtest.h:2233:46
>  (libkudu_test_main.so+0x47bb)
>     #20 main 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/test_main.cc:106:13 
> (libkudu_test_main.so+0x3636)
>
>   Thread T15 'rpc reactor-145' (tid=14548, running) created by main thread at:
>     #0 pthread_create 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/llvm-9.0.0.src/projects/compiler-rt/lib/tsan/rtl/tsan_interceptors.cc:967
>  (tablet_server_quiescing-itest+0x451e29)
>     #1 kudu::Thread::StartThread(std::__1::basic_string<char, 
> std::__1::char_traits<char>, std::__1::allocator<char> >, 
> std::__1::basic_string<char, std::__1::char_traits<char>, 
> std::__1::allocator<char> >, std::__1::function<void ()>, unsigned long, 
> scoped_refptr<kudu::Thread>*) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/thread.cc:618:15 
> (libkudu_util.so+0x2d3faa)
>     #2 kudu::Thread::Create(std::__1::basic_string<char, 
> std::__1::char_traits<char>, std::__1::allocator<char> >, 
> std::__1::basic_string<char, std::__1::char_traits<char>, 
> std::__1::allocator<char> >, std::__1::function<void ()>, 
> scoped_refptr<kudu::Thread>*) 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/thread.h:146:12 
> (libmaster.so+0x18e829)
>     #3 kudu::rpc::ReactorThread::Init() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/reactor.cc:188:10 
> (libkrpc.so+0x1033e6)
>     #4 kudu::rpc::Reactor::Init() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/reactor.cc:764:18 
> (libkrpc.so+0x107931)
>     #5 kudu::rpc::Messenger::Init() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/messenger.cc:447:5 
> (libkrpc.so+0xe21dd)
>     #6 
> kudu::rpc::MessengerBuilder::Build(std::__1::shared_ptr<kudu::rpc::Messenger>*)
>  /home/jenkins-slave/workspace/kudu-master/1/src/kudu/rpc/messenger.cc:203:3 
> (libkrpc.so+0xe1d07)
>     #7 
> kudu::client::KuduClientBuilder::Build(std::__1::shared_ptr<kudu::client::KuduClient>*)
>  /home/jenkins-slave/workspace/kudu-master/1/src/kudu/client/client.cc:351:3 
> (libkudu_client.so+0x197767)
>     #8 
> kudu::cluster::InternalMiniCluster::CreateClient(kudu::client::KuduClientBuilder*,
>  std::__1::shared_ptr<kudu::client::KuduClient>*) const 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/mini-cluster/internal_mini_cluster.cc:350:19
>  (libmini_cluster.so+0x75d76)
>     #9 kudu::TestWorkload::Setup() 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/integration-tests/test_workload.cc:290:5
>  (libitest_util.so+0xc3712)
>     #10 
> kudu::itest::TServerQuiescingITest_TestDoesntAllowNewScansLeadersOnly_Test::TestBody()
>  
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/integration-tests/tablet_server_quiescing-itest.cc:279:16
>  (tablet_server_quiescing-itest+0x4f9337)
>     #11 void 
> testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, 
> void>(testing::Test*, void (testing::Test::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2402:10
>  (libgmock.so+0x54f9f)
>     #12 void 
> testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, 
> void>(testing::Test*, void (testing::Test::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2438:14
>  (libgmock.so+0x54f9f)
>     #13 testing::Test::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2474:5
>  (libgmock.so+0x33f38)
>     #14 testing::TestInfo::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2656:11
>  (libgmock.so+0x351dc)
>     #15 testing::TestCase::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2774:28
>  (libgmock.so+0x35ef6)
>     #16 testing::internal::UnitTestImpl::RunAllTests() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:4649:43
>  (libgmock.so+0x423a6)
>     #17 bool 
> testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,
>  bool>(testing::internal::UnitTestImpl*, bool 
> (testing::internal::UnitTestImpl::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2402:10
>  (libgmock.so+0x55e4f)
>     #18 bool 
> testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl,
>  bool>(testing::internal::UnitTestImpl*, bool 
> (testing::internal::UnitTestImpl::*)(), char const*) 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:2438:14
>  (libgmock.so+0x55e4f)
>     #19 testing::UnitTest::Run() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/src/googletest-release-1.8.0/googletest/src/gtest.cc:4257:10
>  (libgmock.so+0x41c61)
>     #20 RUN_ALL_TESTS() 
> /home/jenkins-slave/workspace/kudu-master/1/thirdparty/installed/tsan/include/gtest/gtest.h:2233:46
>  (libkudu_test_main.so+0x47bb)
>     #21 main 
> /home/jenkins-slave/workspace/kudu-master/1/src/kudu/util/test_main.cc:106:13 
> (libkudu_test_main.so+0x3636)

That looks like KUDU-2059. Unrelated to this patch.

http://gerrit.cloudera.org:8080/#/c/15557/2/src/kudu/gutil/spinlock_linux-inl.h
File src/kudu/gutil/spinlock_linux-inl.h:

PS2:
> Updated all the files except linux_syscall_support.h, where it seemed bette
Nah, let's leave linux_syscall_support.h pristine, for the reasons you 
described.



--
To view, visit http://gerrit.cloudera.org:8080/15557
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6c46acb17f048890a3f93fc6b910b2df3c1a7058
Gerrit-Change-Number: 15557
Gerrit-PatchSet: 3
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward <zhaoren...@hotmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Comment-Date: Wed, 25 Mar 2020 22:59:15 +0000
Gerrit-HasComments: Yes

Reply via email to