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

Side by Side Diff: components/tracing/common/process_metrics_memory_dump_provider.h

Issue 2568313004: [memory-infra] Implement PollFastMemoryTotal in ProcessMetricsMemoryDumpProvider. (Closed)
Patch Set: nits. Created 4 years 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 | « no previous file | components/tracing/common/process_metrics_memory_dump_provider.cc » ('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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #ifndef COMPONENTS_TRACING_COMMON_PROCESS_MEMORY_METRICS_DUMP_PROVIDER_H_ 5 #ifndef COMPONENTS_TRACING_COMMON_PROCESS_MEMORY_METRICS_DUMP_PROVIDER_H_
6 #define COMPONENTS_TRACING_COMMON_PROCESS_MEMORY_METRICS_DUMP_PROVIDER_H_ 6 #define COMPONENTS_TRACING_COMMON_PROCESS_MEMORY_METRICS_DUMP_PROVIDER_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/files/scoped_file.h"
10 #include "base/gtest_prod_util.h" 11 #include "base/gtest_prod_util.h"
11 #include "base/macros.h" 12 #include "base/macros.h"
12 #include "base/process/process_handle.h" 13 #include "base/process/process_handle.h"
13 #include "base/trace_event/memory_dump_provider.h" 14 #include "base/trace_event/memory_dump_provider.h"
14 #include "build/build_config.h" 15 #include "build/build_config.h"
15 #include "components/tracing/tracing_export.h" 16 #include "components/tracing/tracing_export.h"
16 17
17 namespace base { 18 namespace base {
18 class ProcessMetrics; 19 class ProcessMetrics;
19 } 20 }
20 21
21 namespace tracing { 22 namespace tracing {
22 23
23 // Dump provider which collects process-wide memory stats. 24 // Dump provider which collects process-wide memory stats.
24 class TRACING_EXPORT ProcessMetricsMemoryDumpProvider 25 class TRACING_EXPORT ProcessMetricsMemoryDumpProvider
25 : public base::trace_event::MemoryDumpProvider { 26 : public base::trace_event::MemoryDumpProvider {
26 public: 27 public:
27 // Pass base::kNullProcessId to register for current process. 28 // Pass base::kNullProcessId to register for current process.
28 static void RegisterForProcess(base::ProcessId process); 29 static void RegisterForProcess(base::ProcessId process);
29 static void UnregisterForProcess(base::ProcessId process); 30 static void UnregisterForProcess(base::ProcessId process);
30 31
31 ~ProcessMetricsMemoryDumpProvider() override; 32 ~ProcessMetricsMemoryDumpProvider() override;
32 33
33 // MemoryDumpProvider implementation. 34 // MemoryDumpProvider implementation.
34 bool OnMemoryDump(const base::trace_event::MemoryDumpArgs& args, 35 bool OnMemoryDump(const base::trace_event::MemoryDumpArgs& args,
35 base::trace_event::ProcessMemoryDump* pmd) override; 36 base::trace_event::ProcessMemoryDump* pmd) override;
37 void PollFastMemoryTotal(uint64_t* memory_total) override;
38 void SuspendFastMemoryPolling() override;
36 39
37 private: 40 private:
38 FRIEND_TEST_ALL_PREFIXES(ProcessMetricsMemoryDumpProviderTest, 41 FRIEND_TEST_ALL_PREFIXES(ProcessMetricsMemoryDumpProviderTest,
39 ParseProcSmaps); 42 ParseProcSmaps);
40 FRIEND_TEST_ALL_PREFIXES(ProcessMetricsMemoryDumpProviderTest, DumpRSS); 43 FRIEND_TEST_ALL_PREFIXES(ProcessMetricsMemoryDumpProviderTest, DumpRSS);
44 FRIEND_TEST_ALL_PREFIXES(ProcessMetricsMemoryDumpProviderTest,
45 TestPollFastMemoryTotal);
41 46
42 ProcessMetricsMemoryDumpProvider(base::ProcessId process); 47 ProcessMetricsMemoryDumpProvider(base::ProcessId process);
43 48
44 bool DumpProcessTotals(const base::trace_event::MemoryDumpArgs& args, 49 bool DumpProcessTotals(const base::trace_event::MemoryDumpArgs& args,
45 base::trace_event::ProcessMemoryDump* pmd); 50 base::trace_event::ProcessMemoryDump* pmd);
46 bool DumpProcessMemoryMaps(const base::trace_event::MemoryDumpArgs& args, 51 bool DumpProcessMemoryMaps(const base::trace_event::MemoryDumpArgs& args,
47 base::trace_event::ProcessMemoryDump* pmd); 52 base::trace_event::ProcessMemoryDump* pmd);
48 53
49 static uint64_t rss_bytes_for_testing; 54 static uint64_t rss_bytes_for_testing;
50 55
51 #if defined(OS_LINUX) || defined(OS_ANDROID) 56 #if defined(OS_LINUX) || defined(OS_ANDROID)
52 static FILE* proc_smaps_for_testing; 57 static FILE* proc_smaps_for_testing;
58 static int fast_polling_statm_fd_for_testing;
59
60 base::ScopedFD fast_polling_statm_fd_;
53 #endif 61 #endif
54 62
55 base::ProcessId process_; 63 base::ProcessId process_;
56 std::unique_ptr<base::ProcessMetrics> process_metrics_; 64 std::unique_ptr<base::ProcessMetrics> process_metrics_;
57 65
58 // The peak may not be resettable on all the processes if the linux kernel is 66 // The peak may not be resettable on all the processes if the linux kernel is
59 // older than http://bit.ly/reset_rss or only on child processes if yama LSM 67 // older than http://bit.ly/reset_rss or only on child processes if yama LSM
60 // sandbox is enabled. 68 // sandbox is enabled.
61 bool is_rss_peak_resettable_; 69 bool is_rss_peak_resettable_;
62 70
63 DISALLOW_COPY_AND_ASSIGN(ProcessMetricsMemoryDumpProvider); 71 DISALLOW_COPY_AND_ASSIGN(ProcessMetricsMemoryDumpProvider);
64 }; 72 };
65 73
66 } // namespace tracing 74 } // namespace tracing
67 75
68 #endif // COMPONENTS_TRACING_COMMON_PROCESS_MEMORY_METRICS_DUMP_PROVIDER_H_ 76 #endif // COMPONENTS_TRACING_COMMON_PROCESS_MEMORY_METRICS_DUMP_PROVIDER_H_
OLDNEW
« no previous file with comments | « no previous file | components/tracing/common/process_metrics_memory_dump_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698