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

Side by Side Diff: base/task_scheduler/task_scheduler.cc

Issue 2749303002: [reference - do not submit] Always create four pools in TaskSchedulerImpl. (Closed)
Patch Set: rebase Created 3 years, 9 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
« no previous file with comments | « base/task_scheduler/task_scheduler.h ('k') | base/task_scheduler/task_scheduler_impl.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/task_scheduler/task_scheduler.h" 5 #include "base/task_scheduler/task_scheduler.h"
6 6
7 #include <algorithm>
8
9 #include "base/bind.h"
10 #include "base/logging.h" 7 #include "base/logging.h"
11 #include "base/sys_info.h" 8 #include "base/task_scheduler/initialization_util.h"
12 #include "base/task_scheduler/scheduler_worker_pool_params.h" 9 #include "base/task_scheduler/scheduler_worker_pool_params.h"
13 #include "base/task_scheduler/task_scheduler_impl.h" 10 #include "base/task_scheduler/task_scheduler_impl.h"
14 #include "base/threading/platform_thread.h" 11 #include "base/threading/platform_thread.h"
15 #include "base/time/time.h" 12 #include "base/time/time.h"
16 13
17 namespace base { 14 namespace base {
18 15
19 namespace { 16 namespace {
20 17
21 // |g_task_scheduler| is intentionally leaked on shutdown. 18 // |g_task_scheduler| is intentionally leaked on shutdown.
22 TaskScheduler* g_task_scheduler = nullptr; 19 TaskScheduler* g_task_scheduler = nullptr;
23 20
24 } // namespace 21 } // namespace
25 22
23 TaskSchedulerInitParams::TaskSchedulerInitParams(
24 const SchedulerWorkerPoolParams& background_worker_pool_params_in,
25 const SchedulerWorkerPoolParams& background_blocking_worker_pool_params_in,
26 const SchedulerWorkerPoolParams& foreground_worker_pool_params_in,
27 const SchedulerWorkerPoolParams& foreground_blocking_worker_pool_params_in)
28 : background_worker_pool_params(background_worker_pool_params_in),
29 background_blocking_worker_pool_params(
30 background_blocking_worker_pool_params_in),
31 foreground_worker_pool_params(foreground_worker_pool_params_in),
32 foreground_blocking_worker_pool_params(
33 foreground_blocking_worker_pool_params_in) {}
34
35 bool TaskSchedulerInitParams::IsValid() const {
36 return background_worker_pool_params.IsValid() &&
37 background_blocking_worker_pool_params.IsValid() &&
38 foreground_worker_pool_params.IsValid() &&
39 foreground_blocking_worker_pool_params.IsValid();
40 }
41
26 #if !defined(OS_NACL) 42 #if !defined(OS_NACL)
27 // static 43 // static
28 void TaskScheduler::CreateAndSetSimpleTaskScheduler(const std::string& name) { 44 void TaskScheduler::CreateAndSetSimpleTaskScheduler(const std::string& name) {
29 constexpr int kMinNumThreads = 1; 45 using StandbyThreadPolicy = SchedulerWorkerPoolParams::StandbyThreadPolicy;
30 std::vector<SchedulerWorkerPoolParams> worker_pool_params_vector; 46
31 worker_pool_params_vector.emplace_back( 47 constexpr TimeDelta kSuggestedReclaimTime = TimeDelta::FromSeconds(30);
32 name, ThreadPriority::NORMAL, 48 constexpr int kMaxNumBackgroundThreads = 1;
33 SchedulerWorkerPoolParams::StandbyThreadPolicy::LAZY, 49 constexpr int kMaxNumBackgroundBlockingThreads = 2;
34 std::max(kMinNumThreads, base::SysInfo::NumberOfProcessors()), 50 constexpr int kMaxNumForegroundThreadsLowerBound = 2;
35 TimeDelta::FromSeconds(30)); 51 constexpr int kMaxNumForegroundThreadsUpperBound = 32;
52 constexpr double kMaxNumForegroundThreadsCoresMultiplier = 1;
53 constexpr int kMaxNumForegroundThreadsOffset = 0;
54 constexpr int kMaxNumForegroundBlockingThreadsLowerBound = 2;
55 constexpr int kMaxNumForegroundBlockingThreadsUpperBound = 64;
56 constexpr double kMaxNumForegroundBlockingThreadsCoresMultiplier = 2;
57 constexpr int kMaxNumForegroundBlockingThreadsOffset = 0;
58
36 CreateAndSetDefaultTaskScheduler( 59 CreateAndSetDefaultTaskScheduler(
37 worker_pool_params_vector, 60 name, {{StandbyThreadPolicy::LAZY, kMaxNumBackgroundThreads,
38 Bind([](const TaskTraits&) -> size_t { return 0; })); 61 kSuggestedReclaimTime},
62 {StandbyThreadPolicy::LAZY, kMaxNumBackgroundBlockingThreads,
63 kSuggestedReclaimTime},
64 {StandbyThreadPolicy::LAZY,
65 RecommendedMaxNumberOfThreadsInPool(
66 kMaxNumForegroundThreadsLowerBound,
67 kMaxNumForegroundThreadsUpperBound,
68 kMaxNumForegroundThreadsCoresMultiplier,
69 kMaxNumForegroundThreadsOffset),
70 kSuggestedReclaimTime},
71 {StandbyThreadPolicy::LAZY,
72 RecommendedMaxNumberOfThreadsInPool(
73 kMaxNumForegroundBlockingThreadsLowerBound,
74 kMaxNumForegroundBlockingThreadsUpperBound,
75 kMaxNumForegroundBlockingThreadsCoresMultiplier,
76 kMaxNumForegroundBlockingThreadsOffset),
77 kSuggestedReclaimTime}});
39 } 78 }
40 #endif // !defined(OS_NACL) 79 #endif // !defined(OS_NACL)
41 80
42 // static
43 void TaskScheduler::CreateAndSetDefaultTaskScheduler( 81 void TaskScheduler::CreateAndSetDefaultTaskScheduler(
44 const std::vector<SchedulerWorkerPoolParams>& worker_pool_params_vector, 82 const std::string& name,
45 const WorkerPoolIndexForTraitsCallback& 83 const TaskSchedulerInitParams& init_params) {
46 worker_pool_index_for_traits_callback) { 84 SetInstance(internal::TaskSchedulerImpl::Create(name, init_params));
47 SetInstance(internal::TaskSchedulerImpl::Create(
48 worker_pool_params_vector, worker_pool_index_for_traits_callback));
49 } 85 }
50 86
51 // static 87 // static
52 void TaskScheduler::SetInstance(std::unique_ptr<TaskScheduler> task_scheduler) { 88 void TaskScheduler::SetInstance(std::unique_ptr<TaskScheduler> task_scheduler) {
53 delete g_task_scheduler; 89 delete g_task_scheduler;
54 g_task_scheduler = task_scheduler.release(); 90 g_task_scheduler = task_scheduler.release();
55 } 91 }
56 92
57 // static 93 // static
58 TaskScheduler* TaskScheduler::GetInstance() { 94 TaskScheduler* TaskScheduler::GetInstance() {
59 return g_task_scheduler; 95 return g_task_scheduler;
60 } 96 }
61 97
62 } // namespace base 98 } // namespace base
OLDNEW
« no previous file with comments | « base/task_scheduler/task_scheduler.h ('k') | base/task_scheduler/task_scheduler_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698