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

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/logging.h" 7 #include "base/logging.h"
8 8
9 namespace content { 9 namespace content {
10 10
11 // Default sampling period, as recommended by Intel Power Gadget.
12 // Section 3.1 of
13 // http://software.intel.com/en-us/blogs/2013/10/03/using-the-intel-power-gadget -api-on-windows
14 const int kDefaultSamplePeriodMs = 50;
15
11 scoped_ptr<PowerDataProvider> PowerDataProvider::Create() { 16 scoped_ptr<PowerDataProvider> PowerDataProvider::Create() {
12 scoped_ptr<PowerDataProviderIA> provider(new PowerDataProviderIA()); 17 scoped_ptr<PowerDataProviderIA> provider(new PowerDataProviderIA());
13 if (provider->Initialize()) 18 if (provider->Initialize())
14 return make_scoped_ptr<PowerDataProvider>(provider.release()); 19 return make_scoped_ptr<PowerDataProvider>(provider.release());
15 20
16 return make_scoped_ptr<PowerDataProvider>(NULL); 21 return make_scoped_ptr<PowerDataProvider>(NULL);
17 } 22 }
18 23
19 PowerDataProviderIA::PowerDataProviderIA() 24 PowerDataProviderIA::PowerDataProviderIA()
20 : sockets_number_(0), 25 : sockets_number_(0),
(...skipping 26 matching lines...) Expand all
47 } 52 }
48 53
49 event.type = PowerEvent::SOC_PACKAGE; 54 event.type = PowerEvent::SOC_PACKAGE;
50 event.value = package_power; 55 event.value = package_power;
51 event.time = base::TimeTicks::Now(); 56 event.time = base::TimeTicks::Now();
52 events.push_back(event); 57 events.push_back(event);
53 58
54 return events; 59 return events;
55 } 60 }
56 61
62 int64 PowerDataProviderIA::SamplingRate() {
63 return kDefaultSamplePeriodMs;
64 }
65
57 bool PowerDataProviderIA::Initialize() { 66 bool PowerDataProviderIA::Initialize() {
58 if (is_open_) 67 if (is_open_)
59 return true; 68 return true;
60 69
61 if (!energy_lib_.IntelEnergyLibInitialize()) { 70 if (!energy_lib_.IntelEnergyLibInitialize()) {
62 LOG(ERROR) << "Power Data Provider initialize failed!"; 71 LOG(ERROR) << "Power Data Provider initialize failed!";
63 return false; 72 return false;
64 } 73 }
65 74
66 energy_lib_.GetNumNodes(&sockets_number_); 75 energy_lib_.GetNumNodes(&sockets_number_);
(...skipping 14 matching lines...) Expand all
81 90
82 if (package_msr_name.compare(name) == 0) 91 if (package_msr_name.compare(name) == 0)
83 power_msr_ids_[PowerEvent::SOC_PACKAGE] = i; 92 power_msr_ids_[PowerEvent::SOC_PACKAGE] = i;
84 } 93 }
85 94
86 is_open_ = true; 95 is_open_ = true;
87 return true; 96 return true;
88 } 97 }
89 98
90 } // namespace content 99 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698