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

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

Issue 159523003: More instrumentation for GDI allocation failures (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 10 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/debug/gdi_debug_util_win.h ('k') | skia/ext/bitmap_platform_device_win.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright (c) 2014 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 #include "base/debug/gdi_debug_util_win.h"
5
6 #include <cmath>
7
8 #include <psapi.h>
9 #include <TlHelp32.h>
10
11 #include "base/debug/alias.h"
12 #include "base/logging.h"
13 #include "base/win/scoped_handle.h"
14
15 namespace {
16
17 void CollectChildGDIUsageAndDie(DWORD parent_pid) {
18 HANDLE snapshot = CreateToolhelp32Snapshot(TH32CS_SNAPPROCESS, 0) ;
19 if(snapshot == INVALID_HANDLE_VALUE)
20 CHECK(false);
21
22 int child_count = 0;
23 base::debug::Alias(&child_count);
24 int peak_gdi_count = 0;
25 base::debug::Alias(&peak_gdi_count);
26 int sum_gdi_count = 0;
27 base::debug::Alias(&sum_gdi_count);
28 int sum_user_count = 0;
29 base::debug::Alias(&sum_user_count);
30
31 PROCESSENTRY32 proc_entry = {0};
32 proc_entry.dwSize = sizeof(PROCESSENTRY32) ;
33 if(!Process32First(snapshot, &proc_entry))
34 CHECK(false);
35
36 do {
37 if (parent_pid != proc_entry.th32ParentProcessID)
38 continue;
39 // Got a child process. Compute GDI usage.
40 base::win::ScopedHandle process(
41 ::OpenProcess(PROCESS_QUERY_INFORMATION,
42 FALSE,
43 proc_entry.th32ParentProcessID));
44 if (!process)
45 continue;
46
47 int num_gdi_handles = ::GetGuiResources(process.Get(), GR_GDIOBJECTS);
48 int num_user_handles = ::GetGuiResources(process.Get(), GR_USEROBJECTS);
49
50 // Compute sum and peak counts.
51 ++child_count;
52 sum_user_count += num_user_handles;
53 sum_gdi_count += num_gdi_handles;
54 if (peak_gdi_count < num_gdi_handles)
55 peak_gdi_count = num_gdi_handles;
56
57 } while(Process32Next(snapshot, &proc_entry));
58
59 ::CloseHandle(snapshot) ;
60 CHECK(false);
61 }
62
63 } // namespace
64
65 namespace base {
66 namespace debug {
67
68 void GDIBitmapAllocFailure(BITMAPINFOHEADER* header, HANDLE shared_section) {
69 // Make sure parameters are saved in the minidump.
70 DWORD last_error = ::GetLastError();
71
72 LONG width = header->biWidth;
73 LONG heigth = header->biHeight;
74
75 base::debug::Alias(&last_error);
76 base::debug::Alias(&width);
77 base::debug::Alias(&heigth);
78 base::debug::Alias(&shared_section);
79
80 int num_user_handles = GetGuiResources(GetCurrentProcess(),
81 GR_USEROBJECTS);
82
83 int num_gdi_handles = GetGuiResources(GetCurrentProcess(),
84 GR_GDIOBJECTS);
85 if (num_gdi_handles == 0) {
86 DWORD get_gui_resources_error = GetLastError();
87 base::debug::Alias(&get_gui_resources_error);
88 CHECK(false);
89 }
90
91 base::debug::Alias(&num_gdi_handles);
92 base::debug::Alias(&num_user_handles);
93
94 const DWORD kLotsOfHandles = 9990;
95 if (num_gdi_handles > kLotsOfHandles)
96 CHECK(false);
97
98 PROCESS_MEMORY_COUNTERS_EX pmc;
99 pmc.cb = sizeof(pmc);
100 if (!GetProcessMemoryInfo(GetCurrentProcess(),
101 reinterpret_cast<PROCESS_MEMORY_COUNTERS*>(&pmc),
102 sizeof(pmc))) {
103 CHECK(false);
104 }
105 const size_t kLotsOfMemory = 1500 * 1024 * 1024; // 1.5GB
106 if (pmc.PagefileUsage > kLotsOfMemory)
107 CHECK(false);
108 if (pmc.PrivateUsage > kLotsOfMemory)
109 CHECK(false);
110
111 void* small_data = NULL;
112 base::debug::Alias(&small_data);
113
114 if (std::abs(heigth) * width > 100) {
115 // Huh, that's weird. We don't have crazy handle count, we don't have
116 // ridiculous memory usage. Try to allocate a small bitmap and see if that
117 // fails too.
118 header->biWidth = 5;
119 header->biHeight = -5;
120 HBITMAP small_bitmap = CreateDIBSection(
121 NULL, reinterpret_cast<BITMAPINFO*>(&header),
122 0, &small_data, shared_section, 0);
123 }
124 // Maybe the child processes are the ones leaking GDI or USER resouces.
125 CollectChildGDIUsageAndDie(::GetCurrentProcessId());
126 }
127
128 } // namespace debug
129 } // namespace base
OLDNEW
« no previous file with comments | « base/debug/gdi_debug_util_win.h ('k') | skia/ext/bitmap_platform_device_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698