Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(57)

Side by Side Diff: base/debug/activity_tracker_unittest.cc

Issue 1980743002: Track thread activities in order to diagnose hangs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@readwrite-mmf
Patch Set: added support for storing to disk Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "base/debug/activity_tracker.h"
6
7 #include <memory>
8
9 #include "base/bind.h"
10 #include "base/files/file.h"
11 #include "base/files/file_util.h"
12 #include "base/files/memory_mapped_file.h"
13 #include "base/files/scoped_temp_dir.h"
14 #include "base/memory/ptr_util.h"
15 #include "base/pending_task.h"
16 #include "base/synchronization/condition_variable.h"
17 #include "base/synchronization/lock.h"
18 #include "base/synchronization/spin_wait.h"
19 #include "base/threading/simple_thread.h"
20 #include "base/time/time.h"
21 #include "testing/gtest/include/gtest/gtest.h"
22
23 namespace base {
24 namespace debug {
25
26 namespace {
27
28 class TestActivityTracker : public ThreadActivityTracker {
29 public:
30 TestActivityTracker(std::unique_ptr<char[]> memory, size_t mem_size)
31 : ThreadActivityTracker(memset(memory.get(), 0, mem_size), mem_size),
32 mem_segment_(std::move(memory)) {}
33
34 ~TestActivityTracker() override {}
35
36 private:
37 std::unique_ptr<char[]> mem_segment_;
38 };
39
40 } // namespace
41
42
43 class ActivityTrackerTest : public testing::Test {
44 public:
45 const int kMemorySize = 1 << 10; // 1KiB
46 const int kStackSize = 256;
47
48 using StackEntry = ThreadActivityTracker::StackEntry;
49 using StackEntryData = ThreadActivityTracker::StackEntryData;
50
51 ActivityTrackerTest() {}
52
53 ~ActivityTrackerTest() override {
54 GlobalActivityTracker* global_tracker = GlobalActivityTracker::Get();
55 if (global_tracker) {
56 global_tracker->ReleaseTrackerForCurrentThreadForTesting();
57 delete global_tracker;
58 }
59 }
60
61 std::unique_ptr<ThreadActivityTracker> CreateActivityTracker() {
62 std::unique_ptr<char[]> memory(new char[kStackSize]);
63 return WrapUnique(new TestActivityTracker(std::move(memory), kStackSize));
64 }
65
66 size_t GetGlobalActiveTrackerCount() {
67 GlobalActivityTracker* global_tracker = GlobalActivityTracker::Get();
68 if (!global_tracker)
69 return 0;
70 return global_tracker->thread_trackers_.size();
71 }
72
73 size_t GetGlobalInactiveTrackerCount() {
74 GlobalActivityTracker* global_tracker = GlobalActivityTracker::Get();
75 if (!global_tracker)
76 return 0;
77 return global_tracker->available_memories_.size();
78 }
79
80 static void DoNothing() {}
81 };
82
83 TEST_F(ActivityTrackerTest, PushPopTest) {
84 std::unique_ptr<ThreadActivityTracker> tracker = CreateActivityTracker();
85 std::vector<ThreadActivityTracker::StackEntry> stack;
86 ThreadActivityAnalyzer analyzer(tracker.get());
87
88 ASSERT_EQ(0U, analyzer.SnapshotStack(&stack));
89 ASSERT_EQ(0U, stack.size());
90
91 char source1;
92 tracker->PushActivity(&source1, ThreadActivityTracker::ACT_TASK,
93 StackEntryData::ForTask(11));
94 ASSERT_EQ(1U, analyzer.SnapshotStack(&stack));
95 ASSERT_EQ(1U, stack.size());
96 EXPECT_NE(0, stack[0].time_ticks);
97 EXPECT_EQ(ThreadActivityTracker::ACT_TASK, stack[0].activity_type);
98 EXPECT_EQ(reinterpret_cast<intptr_t>(&source1), stack[0].source_address);
99 EXPECT_EQ(11U, stack[0].data.task.sequence_id);
100
101 char source2;
102 tracker->PushActivity(&source2, ThreadActivityTracker::ACT_LOCK,
103 StackEntryData::ForLock(22));
104 ASSERT_EQ(2U, analyzer.SnapshotStack(&stack));
105 ASSERT_EQ(2U, stack.size());
106 EXPECT_LE(stack[0].time_ticks, stack[1].time_ticks);
107 EXPECT_EQ(ThreadActivityTracker::ACT_LOCK, stack[1].activity_type);
108 EXPECT_EQ(reinterpret_cast<intptr_t>(&source2), stack[1].source_address);
109 EXPECT_EQ(11U, stack[0].data.lock.lock_id);
110
111 tracker->PopActivity(&source2);
112 ASSERT_EQ(1U, analyzer.SnapshotStack(&stack));
113 ASSERT_EQ(1U, stack.size());
114 EXPECT_EQ(ThreadActivityTracker::ACT_TASK, stack[0].activity_type);
115 EXPECT_EQ(reinterpret_cast<intptr_t>(&source1), stack[0].source_address);
116 EXPECT_EQ(11U, stack[0].data.task.sequence_id);
117
118 tracker->PopActivity(&source1);
119 ASSERT_EQ(0U, analyzer.SnapshotStack(&stack));
120 ASSERT_EQ(0U, stack.size());
121 }
122
123 TEST_F(ActivityTrackerTest, ScopedTaskTest) {
124 GlobalActivityTracker::CreateWithLocalMemory(kMemorySize, 0, "", 3);
125
126 ThreadActivityTracker* tracker =
127 GlobalActivityTracker::Get()->GetOrCreateTrackerForCurrentThread();
128 std::vector<ThreadActivityTracker::StackEntry> stack;
129 ThreadActivityAnalyzer analyzer(tracker);
130
131 ASSERT_EQ(0U, analyzer.SnapshotStack(&stack));
132 ASSERT_EQ(0U, stack.size());
133
134 {
135 PendingTask task1(FROM_HERE, base::Bind(&DoNothing));
136 ScopedTaskActivity activity1(task1);
137
138 ASSERT_EQ(1U, analyzer.SnapshotStack(&stack));
139 ASSERT_EQ(1U, stack.size());
140 EXPECT_EQ(ThreadActivityTracker::ACT_TASK, stack[0].activity_type);
141
142 {
143 PendingTask task2(FROM_HERE, base::Bind(&DoNothing));
144 ScopedTaskActivity activity2(task2);
145
146 ASSERT_EQ(2U, analyzer.SnapshotStack(&stack));
147 ASSERT_EQ(2U, stack.size());
148 EXPECT_EQ(ThreadActivityTracker::ACT_TASK, stack[1].activity_type);
149 }
150
151 ASSERT_EQ(1U, analyzer.SnapshotStack(&stack));
152 ASSERT_EQ(1U, stack.size());
153 EXPECT_EQ(ThreadActivityTracker::ACT_TASK, stack[0].activity_type);
154 }
155
156 ASSERT_EQ(0U, analyzer.SnapshotStack(&stack));
157 ASSERT_EQ(0U, stack.size());
158 }
159
160 class SimpleActivityThread : public SimpleThread {
161 public:
162 SimpleActivityThread(const std::string& name,
163 const void* source,
164 ThreadActivityTracker::ActivityType activity,
165 const ThreadActivityTracker::StackEntryData& data)
166 : SimpleThread(name, Options()),
167 source_(source),
168 activity_(activity),
169 data_(data),
170 exit_condition_(&lock_) {}
171
172 ~SimpleActivityThread() override {}
173
174 void Run() override {
175 GlobalActivityTracker::Get()
176 ->GetOrCreateTrackerForCurrentThread()
177 ->PushActivity(source_, activity_, data_);
178
179 {
180 AutoLock auto_lock(lock_);
181 ready_ = true;
182 while (!exit_)
183 exit_condition_.Wait();
184 }
185
186 GlobalActivityTracker::Get()
187 ->GetOrCreateTrackerForCurrentThread()
188 ->PopActivity(source_);
189 }
190
191 void Exit() {
192 AutoLock auto_lock(lock_);
193 exit_ = true;
194 exit_condition_.Signal();
195 }
196
197 void WaitReady() {
198 SPIN_FOR_1_SECOND_OR_UNTIL_TRUE(ready_);
199 }
200
201 private:
202 const void* source_;
203 ThreadActivityTracker::ActivityType activity_;
204 ThreadActivityTracker::StackEntryData data_;
205
206 bool ready_ = false;
207 bool exit_ = false;
208 Lock lock_;
209 ConditionVariable exit_condition_;
210
211 DISALLOW_COPY_AND_ASSIGN(SimpleActivityThread);
212 };
213
214 TEST_F(ActivityTrackerTest, ThreadDeathTest) {
215 GlobalActivityTracker::CreateWithLocalMemory(kMemorySize, 0, "", 3);
216 GlobalActivityTracker::Get()->GetOrCreateTrackerForCurrentThread();
217 const size_t starting_active = GetGlobalActiveTrackerCount();
218 const size_t starting_inactive = GetGlobalInactiveTrackerCount();
219
220 SimpleActivityThread t1("t1", nullptr, ThreadActivityTracker::ACT_TASK,
221 StackEntryData::ForTask(11));
222 t1.Start();
223 t1.WaitReady();
224 EXPECT_EQ(starting_active + 1, GetGlobalActiveTrackerCount());
225 EXPECT_EQ(starting_inactive, GetGlobalInactiveTrackerCount());
226
227 t1.Exit();
228 t1.Join();
229 EXPECT_EQ(starting_active, GetGlobalActiveTrackerCount());
230 EXPECT_EQ(starting_inactive + 1, GetGlobalInactiveTrackerCount());
231 }
232
233 } // namespace debug
234 } // namespace base
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698