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

Side by Side Diff: base/process/memory.h

Issue 2130293003: Change OOMs to raise custom exception rather than breakpoint on Windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments Created 4 years, 5 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 | « no previous file | base/process/memory_unittest.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 (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 BASE_PROCESS_MEMORY_H_ 5 #ifndef BASE_PROCESS_MEMORY_H_
6 #define BASE_PROCESS_MEMORY_H_ 6 #define BASE_PROCESS_MEMORY_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include "base/base_export.h" 10 #include "base/base_export.h"
(...skipping 30 matching lines...) Expand all
41 // This adjusts /proc/<pid>/oom_score_adj so the Linux OOM killer will 41 // This adjusts /proc/<pid>/oom_score_adj so the Linux OOM killer will
42 // prefer to kill certain process types over others. The range for the 42 // prefer to kill certain process types over others. The range for the
43 // adjustment is [-1000, 1000], with [0, 1000] being user accessible. 43 // adjustment is [-1000, 1000], with [0, 1000] being user accessible.
44 // If the Linux system doesn't support the newer oom_score_adj range 44 // If the Linux system doesn't support the newer oom_score_adj range
45 // of [0, 1000], then we revert to using the older oom_adj, and 45 // of [0, 1000], then we revert to using the older oom_adj, and
46 // translate the given value into [0, 15]. Some aliasing of values 46 // translate the given value into [0, 15]. Some aliasing of values
47 // may occur in that case, of course. 47 // may occur in that case, of course.
48 BASE_EXPORT bool AdjustOOMScore(ProcessId process, int score); 48 BASE_EXPORT bool AdjustOOMScore(ProcessId process, int score);
49 #endif 49 #endif
50 50
51 #if defined(OS_WIN)
52 namespace win {
53
54 // Custom exception code chosen to indicate an out of memory error.
55 // See https://msdn.microsoft.com/en-us/library/het71c37.aspx.
56 // "To make sure that you do not define a code that conflicts with an existing
57 // exception code" ... "The resulting error code should therefore have the
58 // highest four bits set to hexadecimal E."
59 // 0xe0000008 was chosen arbitrarily, as 0x00000008 is ERROR_NOT_ENOUGH_MEMORY.
60 const DWORD kOomExceptionCode = 0xe0000008;
61
62 } // namespace win
63 #endif
64
51 // Special allocator functions for callers that want to check for OOM. 65 // Special allocator functions for callers that want to check for OOM.
52 // These will not abort if the allocation fails even if 66 // These will not abort if the allocation fails even if
53 // EnableTerminationOnOutOfMemory has been called. 67 // EnableTerminationOnOutOfMemory has been called.
54 // This can be useful for huge and/or unpredictable size memory allocations. 68 // This can be useful for huge and/or unpredictable size memory allocations.
55 // Please only use this if you really handle the case when the allocation 69 // Please only use this if you really handle the case when the allocation
56 // fails. Doing otherwise would risk security. 70 // fails. Doing otherwise would risk security.
57 // These functions may still crash on OOM when running under memory tools, 71 // These functions may still crash on OOM when running under memory tools,
58 // specifically ASan and other sanitizers. 72 // specifically ASan and other sanitizers.
59 // Return value tells whether the allocation succeeded. If it fails |result| is 73 // Return value tells whether the allocation succeeded. If it fails |result| is
60 // set to NULL, otherwise it holds the memory address. 74 // set to NULL, otherwise it holds the memory address.
61 BASE_EXPORT WARN_UNUSED_RESULT bool UncheckedMalloc(size_t size, 75 BASE_EXPORT WARN_UNUSED_RESULT bool UncheckedMalloc(size_t size,
62 void** result); 76 void** result);
63 BASE_EXPORT WARN_UNUSED_RESULT bool UncheckedCalloc(size_t num_items, 77 BASE_EXPORT WARN_UNUSED_RESULT bool UncheckedCalloc(size_t num_items,
64 size_t size, 78 size_t size,
65 void** result); 79 void** result);
66 80
67 } // namespace base 81 } // namespace base
68 82
69 #endif // BASE_PROCESS_MEMORY_H_ 83 #endif // BASE_PROCESS_MEMORY_H_
OLDNEW
« no previous file with comments | « no previous file | base/process/memory_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698