1*6777b538SAndroid Build Coastguard Worker // Copyright 2018 The Chromium Authors
2*6777b538SAndroid Build Coastguard Worker // Use of this source code is governed by a BSD-style license that can be
3*6777b538SAndroid Build Coastguard Worker // found in the LICENSE file.
4*6777b538SAndroid Build Coastguard Worker
5*6777b538SAndroid Build Coastguard Worker #include "base/no_destructor.h"
6*6777b538SAndroid Build Coastguard Worker
7*6777b538SAndroid Build Coastguard Worker #include <memory>
8*6777b538SAndroid Build Coastguard Worker #include <string>
9*6777b538SAndroid Build Coastguard Worker #include <utility>
10*6777b538SAndroid Build Coastguard Worker #include <vector>
11*6777b538SAndroid Build Coastguard Worker
12*6777b538SAndroid Build Coastguard Worker #include "base/atomicops.h"
13*6777b538SAndroid Build Coastguard Worker #include "base/barrier_closure.h"
14*6777b538SAndroid Build Coastguard Worker #include "base/check.h"
15*6777b538SAndroid Build Coastguard Worker #include "base/functional/bind.h"
16*6777b538SAndroid Build Coastguard Worker #include "base/system/sys_info.h"
17*6777b538SAndroid Build Coastguard Worker #include "base/threading/platform_thread.h"
18*6777b538SAndroid Build Coastguard Worker #include "base/threading/simple_thread.h"
19*6777b538SAndroid Build Coastguard Worker #include "build/build_config.h"
20*6777b538SAndroid Build Coastguard Worker #include "testing/gtest/include/gtest/gtest.h"
21*6777b538SAndroid Build Coastguard Worker
22*6777b538SAndroid Build Coastguard Worker namespace base {
23*6777b538SAndroid Build Coastguard Worker
24*6777b538SAndroid Build Coastguard Worker namespace {
25*6777b538SAndroid Build Coastguard Worker
26*6777b538SAndroid Build Coastguard Worker static_assert(!std::is_trivially_destructible_v<std::string>);
27*6777b538SAndroid Build Coastguard Worker static_assert(
28*6777b538SAndroid Build Coastguard Worker std::is_trivially_destructible_v<base::NoDestructor<std::string>>);
29*6777b538SAndroid Build Coastguard Worker
30*6777b538SAndroid Build Coastguard Worker struct CheckOnDestroy {
~CheckOnDestroybase::__anonc85363690111::CheckOnDestroy31*6777b538SAndroid Build Coastguard Worker ~CheckOnDestroy() { CHECK(false); }
32*6777b538SAndroid Build Coastguard Worker };
33*6777b538SAndroid Build Coastguard Worker
TEST(NoDestructorTest,SkipsDestructors)34*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, SkipsDestructors) {
35*6777b538SAndroid Build Coastguard Worker NoDestructor<CheckOnDestroy> destructor_should_not_run;
36*6777b538SAndroid Build Coastguard Worker }
37*6777b538SAndroid Build Coastguard Worker
38*6777b538SAndroid Build Coastguard Worker struct UncopyableUnmovable {
39*6777b538SAndroid Build Coastguard Worker UncopyableUnmovable() = default;
UncopyableUnmovablebase::__anonc85363690111::UncopyableUnmovable40*6777b538SAndroid Build Coastguard Worker explicit UncopyableUnmovable(int value) : value(value) {}
41*6777b538SAndroid Build Coastguard Worker
42*6777b538SAndroid Build Coastguard Worker UncopyableUnmovable(const UncopyableUnmovable&) = delete;
43*6777b538SAndroid Build Coastguard Worker UncopyableUnmovable& operator=(const UncopyableUnmovable&) = delete;
44*6777b538SAndroid Build Coastguard Worker
45*6777b538SAndroid Build Coastguard Worker int value = 1;
46*6777b538SAndroid Build Coastguard Worker std::string something_with_a_nontrivial_destructor;
47*6777b538SAndroid Build Coastguard Worker };
48*6777b538SAndroid Build Coastguard Worker
49*6777b538SAndroid Build Coastguard Worker struct CopyOnly {
50*6777b538SAndroid Build Coastguard Worker CopyOnly() = default;
51*6777b538SAndroid Build Coastguard Worker
52*6777b538SAndroid Build Coastguard Worker CopyOnly(const CopyOnly&) = default;
53*6777b538SAndroid Build Coastguard Worker CopyOnly& operator=(const CopyOnly&) = default;
54*6777b538SAndroid Build Coastguard Worker
55*6777b538SAndroid Build Coastguard Worker CopyOnly(CopyOnly&&) = delete;
56*6777b538SAndroid Build Coastguard Worker CopyOnly& operator=(CopyOnly&&) = delete;
57*6777b538SAndroid Build Coastguard Worker };
58*6777b538SAndroid Build Coastguard Worker
59*6777b538SAndroid Build Coastguard Worker struct MoveOnly {
60*6777b538SAndroid Build Coastguard Worker MoveOnly() = default;
61*6777b538SAndroid Build Coastguard Worker
62*6777b538SAndroid Build Coastguard Worker MoveOnly(const MoveOnly&) = delete;
63*6777b538SAndroid Build Coastguard Worker MoveOnly& operator=(const MoveOnly&) = delete;
64*6777b538SAndroid Build Coastguard Worker
65*6777b538SAndroid Build Coastguard Worker MoveOnly(MoveOnly&&) = default;
66*6777b538SAndroid Build Coastguard Worker MoveOnly& operator=(MoveOnly&&) = default;
67*6777b538SAndroid Build Coastguard Worker };
68*6777b538SAndroid Build Coastguard Worker
69*6777b538SAndroid Build Coastguard Worker struct ForwardingTestStruct {
ForwardingTestStructbase::__anonc85363690111::ForwardingTestStruct70*6777b538SAndroid Build Coastguard Worker ForwardingTestStruct(const CopyOnly&, MoveOnly&&) {}
71*6777b538SAndroid Build Coastguard Worker
72*6777b538SAndroid Build Coastguard Worker std::string something_with_a_nontrivial_destructor;
73*6777b538SAndroid Build Coastguard Worker };
74*6777b538SAndroid Build Coastguard Worker
TEST(NoDestructorTest,UncopyableUnmovable)75*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, UncopyableUnmovable) {
76*6777b538SAndroid Build Coastguard Worker static NoDestructor<UncopyableUnmovable> default_constructed;
77*6777b538SAndroid Build Coastguard Worker EXPECT_EQ(1, default_constructed->value);
78*6777b538SAndroid Build Coastguard Worker
79*6777b538SAndroid Build Coastguard Worker static NoDestructor<UncopyableUnmovable> constructed_with_arg(-1);
80*6777b538SAndroid Build Coastguard Worker EXPECT_EQ(-1, constructed_with_arg->value);
81*6777b538SAndroid Build Coastguard Worker }
82*6777b538SAndroid Build Coastguard Worker
TEST(NoDestructorTest,ForwardsArguments)83*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, ForwardsArguments) {
84*6777b538SAndroid Build Coastguard Worker CopyOnly copy_only;
85*6777b538SAndroid Build Coastguard Worker MoveOnly move_only;
86*6777b538SAndroid Build Coastguard Worker
87*6777b538SAndroid Build Coastguard Worker static NoDestructor<ForwardingTestStruct> test_forwarding(
88*6777b538SAndroid Build Coastguard Worker copy_only, std::move(move_only));
89*6777b538SAndroid Build Coastguard Worker }
90*6777b538SAndroid Build Coastguard Worker
TEST(NoDestructorTest,Accessors)91*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, Accessors) {
92*6777b538SAndroid Build Coastguard Worker static NoDestructor<std::string> awesome("awesome");
93*6777b538SAndroid Build Coastguard Worker
94*6777b538SAndroid Build Coastguard Worker EXPECT_EQ("awesome", *awesome);
95*6777b538SAndroid Build Coastguard Worker EXPECT_EQ(0, awesome->compare("awesome"));
96*6777b538SAndroid Build Coastguard Worker EXPECT_EQ(0, awesome.get()->compare("awesome"));
97*6777b538SAndroid Build Coastguard Worker }
98*6777b538SAndroid Build Coastguard Worker
99*6777b538SAndroid Build Coastguard Worker // Passing initializer list to a NoDestructor like in this test
100*6777b538SAndroid Build Coastguard Worker // is ambiguous in GCC.
101*6777b538SAndroid Build Coastguard Worker // https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84849
102*6777b538SAndroid Build Coastguard Worker #if !defined(COMPILER_GCC) && !defined(__clang__)
TEST(NoDestructorTest,InitializerList)103*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, InitializerList) {
104*6777b538SAndroid Build Coastguard Worker static NoDestructor<std::vector<std::string>> vector({"a", "b", "c"});
105*6777b538SAndroid Build Coastguard Worker }
106*6777b538SAndroid Build Coastguard Worker #endif
107*6777b538SAndroid Build Coastguard Worker } // namespace
108*6777b538SAndroid Build Coastguard Worker
109*6777b538SAndroid Build Coastguard Worker namespace {
110*6777b538SAndroid Build Coastguard Worker
111*6777b538SAndroid Build Coastguard Worker // A class whose constructor busy-loops until it is told to complete
112*6777b538SAndroid Build Coastguard Worker // construction.
113*6777b538SAndroid Build Coastguard Worker class BlockingConstructor {
114*6777b538SAndroid Build Coastguard Worker public:
BlockingConstructor()115*6777b538SAndroid Build Coastguard Worker BlockingConstructor() {
116*6777b538SAndroid Build Coastguard Worker EXPECT_FALSE(WasConstructorCalled());
117*6777b538SAndroid Build Coastguard Worker subtle::NoBarrier_Store(&constructor_called_, 1);
118*6777b538SAndroid Build Coastguard Worker EXPECT_TRUE(WasConstructorCalled());
119*6777b538SAndroid Build Coastguard Worker while (!subtle::NoBarrier_Load(&complete_construction_))
120*6777b538SAndroid Build Coastguard Worker PlatformThread::YieldCurrentThread();
121*6777b538SAndroid Build Coastguard Worker done_construction_ = true;
122*6777b538SAndroid Build Coastguard Worker }
123*6777b538SAndroid Build Coastguard Worker BlockingConstructor(const BlockingConstructor&) = delete;
124*6777b538SAndroid Build Coastguard Worker BlockingConstructor& operator=(const BlockingConstructor&) = delete;
125*6777b538SAndroid Build Coastguard Worker ~BlockingConstructor() = delete;
126*6777b538SAndroid Build Coastguard Worker
127*6777b538SAndroid Build Coastguard Worker // Returns true if BlockingConstructor() was entered.
WasConstructorCalled()128*6777b538SAndroid Build Coastguard Worker static bool WasConstructorCalled() {
129*6777b538SAndroid Build Coastguard Worker return subtle::NoBarrier_Load(&constructor_called_);
130*6777b538SAndroid Build Coastguard Worker }
131*6777b538SAndroid Build Coastguard Worker
132*6777b538SAndroid Build Coastguard Worker // Instructs BlockingConstructor() that it may now unblock its construction.
CompleteConstructionNow()133*6777b538SAndroid Build Coastguard Worker static void CompleteConstructionNow() {
134*6777b538SAndroid Build Coastguard Worker subtle::NoBarrier_Store(&complete_construction_, 1);
135*6777b538SAndroid Build Coastguard Worker }
136*6777b538SAndroid Build Coastguard Worker
done_construction() const137*6777b538SAndroid Build Coastguard Worker bool done_construction() const { return done_construction_; }
138*6777b538SAndroid Build Coastguard Worker
139*6777b538SAndroid Build Coastguard Worker private:
140*6777b538SAndroid Build Coastguard Worker // Use Atomic32 instead of AtomicFlag for them to be trivially initialized.
141*6777b538SAndroid Build Coastguard Worker static subtle::Atomic32 constructor_called_;
142*6777b538SAndroid Build Coastguard Worker static subtle::Atomic32 complete_construction_;
143*6777b538SAndroid Build Coastguard Worker
144*6777b538SAndroid Build Coastguard Worker bool done_construction_ = false;
145*6777b538SAndroid Build Coastguard Worker };
146*6777b538SAndroid Build Coastguard Worker
147*6777b538SAndroid Build Coastguard Worker // static
148*6777b538SAndroid Build Coastguard Worker subtle::Atomic32 BlockingConstructor::constructor_called_ = 0;
149*6777b538SAndroid Build Coastguard Worker // static
150*6777b538SAndroid Build Coastguard Worker subtle::Atomic32 BlockingConstructor::complete_construction_ = 0;
151*6777b538SAndroid Build Coastguard Worker
152*6777b538SAndroid Build Coastguard Worker // A SimpleThread running at |thread_type| which invokes |before_get| (optional)
153*6777b538SAndroid Build Coastguard Worker // and then invokes thread-safe scoped-static-initializationconstruction on its
154*6777b538SAndroid Build Coastguard Worker // NoDestructor instance.
155*6777b538SAndroid Build Coastguard Worker class BlockingConstructorThread : public SimpleThread {
156*6777b538SAndroid Build Coastguard Worker public:
BlockingConstructorThread(ThreadType thread_type,OnceClosure before_get)157*6777b538SAndroid Build Coastguard Worker BlockingConstructorThread(ThreadType thread_type, OnceClosure before_get)
158*6777b538SAndroid Build Coastguard Worker : SimpleThread("BlockingConstructorThread", Options(thread_type)),
159*6777b538SAndroid Build Coastguard Worker before_get_(std::move(before_get)) {}
160*6777b538SAndroid Build Coastguard Worker BlockingConstructorThread(const BlockingConstructorThread&) = delete;
161*6777b538SAndroid Build Coastguard Worker BlockingConstructorThread& operator=(const BlockingConstructorThread&) =
162*6777b538SAndroid Build Coastguard Worker delete;
163*6777b538SAndroid Build Coastguard Worker
Run()164*6777b538SAndroid Build Coastguard Worker void Run() override {
165*6777b538SAndroid Build Coastguard Worker if (before_get_)
166*6777b538SAndroid Build Coastguard Worker std::move(before_get_).Run();
167*6777b538SAndroid Build Coastguard Worker
168*6777b538SAndroid Build Coastguard Worker static NoDestructor<BlockingConstructor> instance;
169*6777b538SAndroid Build Coastguard Worker EXPECT_TRUE(instance->done_construction());
170*6777b538SAndroid Build Coastguard Worker }
171*6777b538SAndroid Build Coastguard Worker
172*6777b538SAndroid Build Coastguard Worker private:
173*6777b538SAndroid Build Coastguard Worker OnceClosure before_get_;
174*6777b538SAndroid Build Coastguard Worker };
175*6777b538SAndroid Build Coastguard Worker
176*6777b538SAndroid Build Coastguard Worker } // namespace
177*6777b538SAndroid Build Coastguard Worker
178*6777b538SAndroid Build Coastguard Worker // Tests that if the thread assigned to construct the local-static
179*6777b538SAndroid Build Coastguard Worker // initialization of the NoDestructor runs at background priority : the
180*6777b538SAndroid Build Coastguard Worker // foreground threads will yield to it enough for it to eventually complete
181*6777b538SAndroid Build Coastguard Worker // construction. While local-static thread-safe initialization isn't specific to
182*6777b538SAndroid Build Coastguard Worker // NoDestructor, it is tested here as NoDestructor is set to replace
183*6777b538SAndroid Build Coastguard Worker // LazyInstance and this is an important regression test for it
184*6777b538SAndroid Build Coastguard Worker // (https://crbug.com/797129).
TEST(NoDestructorTest,PriorityInversionAtStaticInitializationResolves)185*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, PriorityInversionAtStaticInitializationResolves) {
186*6777b538SAndroid Build Coastguard Worker TimeTicks test_begin = TimeTicks::Now();
187*6777b538SAndroid Build Coastguard Worker
188*6777b538SAndroid Build Coastguard Worker // Construct BlockingConstructor from a thread that is lower priority than the
189*6777b538SAndroid Build Coastguard Worker // other threads that will be constructed. This thread used to be BACKGROUND
190*6777b538SAndroid Build Coastguard Worker // priority but that caused it to be starved by other simultaneously running
191*6777b538SAndroid Build Coastguard Worker // test processes, leading to false-positive failures.
192*6777b538SAndroid Build Coastguard Worker BlockingConstructorThread background_getter(ThreadType::kDefault,
193*6777b538SAndroid Build Coastguard Worker OnceClosure());
194*6777b538SAndroid Build Coastguard Worker background_getter.Start();
195*6777b538SAndroid Build Coastguard Worker
196*6777b538SAndroid Build Coastguard Worker while (!BlockingConstructor::WasConstructorCalled())
197*6777b538SAndroid Build Coastguard Worker PlatformThread::Sleep(Milliseconds(1));
198*6777b538SAndroid Build Coastguard Worker
199*6777b538SAndroid Build Coastguard Worker // Spin 4 foreground thread per core contending to get the already under
200*6777b538SAndroid Build Coastguard Worker // construction NoDestructor. When they are all running and poking at it :
201*6777b538SAndroid Build Coastguard Worker // allow the background thread to complete its work.
202*6777b538SAndroid Build Coastguard Worker const int kNumForegroundThreads = 4 * SysInfo::NumberOfProcessors();
203*6777b538SAndroid Build Coastguard Worker std::vector<std::unique_ptr<SimpleThread>> foreground_threads;
204*6777b538SAndroid Build Coastguard Worker RepeatingClosure foreground_thread_ready_callback =
205*6777b538SAndroid Build Coastguard Worker BarrierClosure(kNumForegroundThreads,
206*6777b538SAndroid Build Coastguard Worker BindOnce(&BlockingConstructor::CompleteConstructionNow));
207*6777b538SAndroid Build Coastguard Worker for (int i = 0; i < kNumForegroundThreads; ++i) {
208*6777b538SAndroid Build Coastguard Worker // Create threads that are higher priority than background_getter. See above
209*6777b538SAndroid Build Coastguard Worker // for why these particular priorities are chosen.
210*6777b538SAndroid Build Coastguard Worker foreground_threads.push_back(std::make_unique<BlockingConstructorThread>(
211*6777b538SAndroid Build Coastguard Worker ThreadType::kDisplayCritical, foreground_thread_ready_callback));
212*6777b538SAndroid Build Coastguard Worker foreground_threads.back()->Start();
213*6777b538SAndroid Build Coastguard Worker }
214*6777b538SAndroid Build Coastguard Worker
215*6777b538SAndroid Build Coastguard Worker // This test will hang if the foreground threads become stuck in
216*6777b538SAndroid Build Coastguard Worker // NoDestructor's construction per the background thread never being scheduled
217*6777b538SAndroid Build Coastguard Worker // to complete construction.
218*6777b538SAndroid Build Coastguard Worker for (auto& foreground_thread : foreground_threads)
219*6777b538SAndroid Build Coastguard Worker foreground_thread->Join();
220*6777b538SAndroid Build Coastguard Worker background_getter.Join();
221*6777b538SAndroid Build Coastguard Worker
222*6777b538SAndroid Build Coastguard Worker // Fail if this test takes more than 5 seconds (it takes 5-10 seconds on a
223*6777b538SAndroid Build Coastguard Worker // Z840 without https://crrev.com/527445 but is expected to be fast (~30ms)
224*6777b538SAndroid Build Coastguard Worker // with the fix).
225*6777b538SAndroid Build Coastguard Worker EXPECT_LT(TimeTicks::Now() - test_begin, Seconds(5));
226*6777b538SAndroid Build Coastguard Worker }
227*6777b538SAndroid Build Coastguard Worker
228*6777b538SAndroid Build Coastguard Worker } // namespace base
229