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

Side by Side Diff: content/browser/power_profiler/power_data_provider_ia_win.cc

Issue 309803002: [PowerProfiler] Power Profiler service should detect the sampling rate from the data provider. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "content/browser/power_profiler/power_data_provider_ia_win.h" 5 #include "content/browser/power_profiler/power_data_provider_ia_win.h"
6 6
7 #include "base/basictypes.h"
7 #include "base/logging.h" 8 #include "base/logging.h"
8 9
9 namespace content { 10 namespace content {
10 11
12 // Default sampling period, as recommended by Intel Power Gadget.
13 // Section 3.1 of
14 // http://software.intel.com/en-us/blogs/2013/10/03/using-the-intel-power-gadget -api-on-windows
15 const int kDefaultSamplePeriodMs = 50;
16
11 scoped_ptr<PowerDataProvider> PowerDataProvider::Create() { 17 scoped_ptr<PowerDataProvider> PowerDataProvider::Create() {
12 scoped_ptr<PowerDataProviderIA> provider(new PowerDataProviderIA()); 18 scoped_ptr<PowerDataProviderIA> provider(new PowerDataProviderIA());
13 if (provider->Initialize()) 19 if (provider->Initialize())
14 return make_scoped_ptr<PowerDataProvider>(provider.release()); 20 return make_scoped_ptr<PowerDataProvider>(provider.release());
15 21
16 return make_scoped_ptr<PowerDataProvider>(NULL); 22 return make_scoped_ptr<PowerDataProvider>(NULL);
17 } 23 }
18 24
19 PowerDataProviderIA::PowerDataProviderIA() 25 PowerDataProviderIA::PowerDataProviderIA()
20 : sockets_number_(0), 26 : sockets_number_(0),
(...skipping 26 matching lines...) Expand all
47 } 53 }
48 54
49 event.type = PowerEvent::SOC_PACKAGE; 55 event.type = PowerEvent::SOC_PACKAGE;
50 event.value = package_power; 56 event.value = package_power;
51 event.time = base::TimeTicks::Now(); 57 event.time = base::TimeTicks::Now();
52 events.push_back(event); 58 events.push_back(event);
53 59
54 return events; 60 return events;
55 } 61 }
56 62
63 base::TimeDelta PowerDataProviderIA::GetSamplingRate() {
64 return base::TimeDelta::FromMilliseconds(kDefaultSamplePeriodMs);
65 }
66
57 bool PowerDataProviderIA::Initialize() { 67 bool PowerDataProviderIA::Initialize() {
58 if (is_open_) 68 if (is_open_)
59 return true; 69 return true;
60 70
61 if (!energy_lib_.IntelEnergyLibInitialize()) { 71 if (!energy_lib_.IntelEnergyLibInitialize()) {
62 LOG(ERROR) << "Power Data Provider initialize failed!"; 72 LOG(ERROR) << "Power Data Provider initialize failed!";
63 return false; 73 return false;
64 } 74 }
65 75
66 energy_lib_.GetNumNodes(&sockets_number_); 76 energy_lib_.GetNumNodes(&sockets_number_);
(...skipping 14 matching lines...) Expand all
81 91
82 if (package_msr_name.compare(name) == 0) 92 if (package_msr_name.compare(name) == 0)
83 power_msr_ids_[PowerEvent::SOC_PACKAGE] = i; 93 power_msr_ids_[PowerEvent::SOC_PACKAGE] = i;
84 } 94 }
85 95
86 is_open_ = true; 96 is_open_ = true;
87 return true; 97 return true;
88 } 98 }
89 99
90 } // namespace content 100 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/power_profiler/power_data_provider_ia_win.h ('k') | content/browser/power_profiler/power_profiler_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698