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

Side by Side Diff: webrtc/rtc_base/thread_annotations.h

Issue 3011973002: Add thread annotation macros with RTC_ prefix. (Closed)
Patch Set: Created 3 years, 3 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 | webrtc/rtc_base/thread_annotations_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 // 1 //
2 // Copyright (c) 2013 The WebRTC project authors. All Rights Reserved. 2 // Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
3 // 3 //
4 // Use of this source code is governed by a BSD-style license 4 // Use of this source code is governed by a BSD-style license
5 // that can be found in the LICENSE file in the root of the source 5 // that can be found in the LICENSE file in the root of the source
6 // tree. An additional intellectual property rights grant can be found 6 // tree. An additional intellectual property rights grant can be found
7 // in the file PATENTS. All contributing project authors may 7 // in the file PATENTS. All contributing project authors may
8 // be found in the AUTHORS file in the root of the source tree. 8 // be found in the AUTHORS file in the root of the source tree.
9 // 9 //
10 // Borrowed from 10 // Borrowed from
11 // https://code.google.com/p/gperftools/source/browse/src/base/thread_annotation s.h 11 // https://code.google.com/p/gperftools/source/browse/src/base/thread_annotation s.h
12 // but adapted for clang attributes instead of the gcc. 12 // but adapted for clang attributes instead of the gcc.
13 // 13 //
14 // This header file contains the macro definitions for thread safety 14 // This header file contains the macro definitions for thread safety
15 // annotations that allow the developers to document the locking policies 15 // annotations that allow the developers to document the locking policies
16 // of their multi-threaded code. The annotations can also help program 16 // of their multi-threaded code. The annotations can also help program
17 // analysis tools to identify potential thread safety issues. 17 // analysis tools to identify potential thread safety issues.
18 18
19 #ifndef WEBRTC_RTC_BASE_THREAD_ANNOTATIONS_H_ 19 #ifndef WEBRTC_RTC_BASE_THREAD_ANNOTATIONS_H_
20 #define WEBRTC_RTC_BASE_THREAD_ANNOTATIONS_H_ 20 #define WEBRTC_RTC_BASE_THREAD_ANNOTATIONS_H_
21 21
22 #if defined(__clang__) && (!defined(SWIG)) 22 #if defined(__clang__) && (!defined(SWIG))
23 #define THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x)) 23 #define RTC_THREAD_ANNOTATION_ATTRIBUTE__(x) __attribute__((x))
24 #else 24 #else
25 #define THREAD_ANNOTATION_ATTRIBUTE__(x) // no-op 25 #define RTC_THREAD_ANNOTATION_ATTRIBUTE__(x) // no-op
26 #endif 26 #endif
27 27
28 // Document if a shared variable/field needs to be protected by a lock. 28 // Document if a shared variable/field needs to be protected by a lock.
29 // GUARDED_BY allows the user to specify a particular lock that should be 29 // GUARDED_BY allows the user to specify a particular lock that should be
30 // held when accessing the annotated variable, while GUARDED_VAR only 30 // held when accessing the annotated variable, while GUARDED_VAR only
31 // indicates a shared variable should be guarded (by any lock). GUARDED_VAR 31 // indicates a shared variable should be guarded (by any lock). GUARDED_VAR
32 // is primarily used when the client cannot express the name of the lock. 32 // is primarily used when the client cannot express the name of the lock.
33 #if !defined(GUARDED_BY) 33 #if !defined(GUARDED_BY)
34 #define GUARDED_BY(x) THREAD_ANNOTATION_ATTRIBUTE__(guarded_by(x)) 34 #define GUARDED_BY(x) RTC_THREAD_ANNOTATION_ATTRIBUTE__(guarded_by(x))
35 #endif 35 #endif
36 #define RTC_GUARDED_BY(x) RTC_THREAD_ANNOTATION_ATTRIBUTE__(guarded_by(x))
36 #if !defined(GUARDED_VAR) 37 #if !defined(GUARDED_VAR)
37 #define GUARDED_VAR THREAD_ANNOTATION_ATTRIBUTE__(guarded_var) 38 #define GUARDED_VAR RTC_THREAD_ANNOTATION_ATTRIBUTE__(guarded_var)
38 #endif 39 #endif
40 #define RTC_GUARDED_VAR RTC_THREAD_ANNOTATION_ATTRIBUTE__(guarded_var)
kwiberg-webrtc 2017/09/06 10:31:55 Instead of duplicating the definitions, you could
39 41
40 // Document if the memory location pointed to by a pointer should be guarded 42 // Document if the memory location pointed to by a pointer should be guarded
41 // by a lock when dereferencing the pointer. Similar to GUARDED_VAR, 43 // by a lock when dereferencing the pointer. Similar to GUARDED_VAR,
42 // PT_GUARDED_VAR is primarily used when the client cannot express the name 44 // PT_GUARDED_VAR is primarily used when the client cannot express the name
43 // of the lock. Note that a pointer variable to a shared memory location 45 // of the lock. Note that a pointer variable to a shared memory location
44 // could itself be a shared variable. For example, if a shared global pointer 46 // could itself be a shared variable. For example, if a shared global pointer
45 // q, which is guarded by mu1, points to a shared memory location that is 47 // q, which is guarded by mu1, points to a shared memory location that is
46 // guarded by mu2, q should be annotated as follows: 48 // guarded by mu2, q should be annotated as follows:
47 // int *q GUARDED_BY(mu1) PT_GUARDED_BY(mu2); 49 // int *q GUARDED_BY(mu1) PT_GUARDED_BY(mu2);
48 #if !defined(PT_GUARDED_BY) 50 #if !defined(PT_GUARDED_BY)
49 #define PT_GUARDED_BY(x) THREAD_ANNOTATION_ATTRIBUTE__(pt_guarded_by(x)) 51 #define PT_GUARDED_BY(x) RTC_THREAD_ANNOTATION_ATTRIBUTE__(pt_guarded_by(x))
50 #endif 52 #endif
53 #define RTC_PT_GUARDED_BY(x) RTC_THREAD_ANNOTATION_ATTRIBUTE__(pt_guarded_by(x))
51 #if !defined(PT_GUARDED_VAR) 54 #if !defined(PT_GUARDED_VAR)
52 #define PT_GUARDED_VAR THREAD_ANNOTATION_ATTRIBUTE__(pt_guarded_var) 55 #define PT_GUARDED_VAR RTC_THREAD_ANNOTATION_ATTRIBUTE__(pt_guarded_var)
53 #endif 56 #endif
57 #define RTC_PT_GUARDED_VAR RTC_THREAD_ANNOTATION_ATTRIBUTE__(pt_guarded_var)
54 58
55 // Document the acquisition order between locks that can be held 59 // Document the acquisition order between locks that can be held
56 // simultaneously by a thread. For any two locks that need to be annotated 60 // simultaneously by a thread. For any two locks that need to be annotated
57 // to establish an acquisition order, only one of them needs the annotation. 61 // to establish an acquisition order, only one of them needs the annotation.
58 // (i.e. You don't have to annotate both locks with both ACQUIRED_AFTER 62 // (i.e. You don't have to annotate both locks with both ACQUIRED_AFTER
59 // and ACQUIRED_BEFORE.) 63 // and ACQUIRED_BEFORE.)
60 #if !defined(ACQUIRED_AFTER) 64 #if !defined(ACQUIRED_AFTER)
61 #define ACQUIRED_AFTER(x) THREAD_ANNOTATION_ATTRIBUTE__(acquired_after(x)) 65 #define ACQUIRED_AFTER(x) RTC_THREAD_ANNOTATION_ATTRIBUTE__(acquired_after(x))
62 #endif 66 #endif
67 #define RTC_ACQUIRED_AFTER(x) \
68 RTC_THREAD_ANNOTATION_ATTRIBUTE__(acquired_after(x))
63 #if !defined(ACQUIRED_BEFORE) 69 #if !defined(ACQUIRED_BEFORE)
64 #define ACQUIRED_BEFORE(x) THREAD_ANNOTATION_ATTRIBUTE__(acquired_before(x)) 70 #define ACQUIRED_BEFORE(x) RTC_THREAD_ANNOTATION_ATTRIBUTE__(acquired_before(x))
65 #endif 71 #endif
72 #define RTC_ACQUIRED_BEFORE(x) \
73 RTC_THREAD_ANNOTATION_ATTRIBUTE__(acquired_before(x))
66 74
67 // The following three annotations document the lock requirements for 75 // The following three annotations document the lock requirements for
68 // functions/methods. 76 // functions/methods.
69 77
70 // Document if a function expects certain locks to be held before it is called 78 // Document if a function expects certain locks to be held before it is called
71 #if !defined(EXCLUSIVE_LOCKS_REQUIRED) 79 #if !defined(EXCLUSIVE_LOCKS_REQUIRED)
72 #define EXCLUSIVE_LOCKS_REQUIRED(...) \ 80 #define EXCLUSIVE_LOCKS_REQUIRED(...) \
73 THREAD_ANNOTATION_ATTRIBUTE__(exclusive_locks_required(__VA_ARGS__)) 81 RTC_THREAD_ANNOTATION_ATTRIBUTE__(exclusive_locks_required(__VA_ARGS__))
74 #endif 82 #endif
75 83 #define RTC_EXCLUSIVE_LOCKS_REQUIRED(...) \
84 RTC_THREAD_ANNOTATION_ATTRIBUTE__(exclusive_locks_required(__VA_ARGS__))
76 #if !defined(SHARED_LOCKS_REQUIRED) 85 #if !defined(SHARED_LOCKS_REQUIRED)
77 #define SHARED_LOCKS_REQUIRED(...) \ 86 #define SHARED_LOCKS_REQUIRED(...) \
78 THREAD_ANNOTATION_ATTRIBUTE__(shared_locks_required(__VA_ARGS__)) 87 RTC_THREAD_ANNOTATION_ATTRIBUTE__(shared_locks_required(__VA_ARGS__))
79 #endif 88 #endif
89 #define RTC_SHARED_LOCKS_REQUIRED(...) \
90 RTC_THREAD_ANNOTATION_ATTRIBUTE__(shared_locks_required(__VA_ARGS__))
80 91
81 // Document the locks acquired in the body of the function. These locks 92 // Document the locks acquired in the body of the function. These locks
82 // cannot be held when calling this function (as google3's Mutex locks are 93 // cannot be held when calling this function (as google3's Mutex locks are
83 // non-reentrant). 94 // non-reentrant).
84 #if !defined(LOCKS_EXCLUDED) 95 #if !defined(LOCKS_EXCLUDED)
85 #define LOCKS_EXCLUDED(...) \ 96 #define LOCKS_EXCLUDED(...) \
86 THREAD_ANNOTATION_ATTRIBUTE__(locks_excluded(__VA_ARGS__)) 97 RTC_THREAD_ANNOTATION_ATTRIBUTE__(locks_excluded(__VA_ARGS__))
87 #endif 98 #endif
99 #define RTC_LOCKS_EXCLUDED(...) \
100 RTC_THREAD_ANNOTATION_ATTRIBUTE__(locks_excluded(__VA_ARGS__))
88 101
89 // Document the lock the annotated function returns without acquiring it. 102 // Document the lock the annotated function returns without acquiring it.
90 #if !defined(LOCK_RETURNED) 103 #if !defined(LOCK_RETURNED)
91 #define LOCK_RETURNED(x) THREAD_ANNOTATION_ATTRIBUTE__(lock_returned(x)) 104 #define LOCK_RETURNED(x) RTC_THREAD_ANNOTATION_ATTRIBUTE__(lock_returned(x))
92 #endif 105 #endif
106 #define RTC_LOCK_RETURNED(x) RTC_THREAD_ANNOTATION_ATTRIBUTE__(lock_returned(x))
93 107
94 // Document if a class/type is a lockable type (such as the Mutex class). 108 // Document if a class/type is a lockable type (such as the Mutex class).
95 #if !defined(LOCKABLE) 109 #if !defined(LOCKABLE)
96 #define LOCKABLE THREAD_ANNOTATION_ATTRIBUTE__(lockable) 110 #define LOCKABLE RTC_THREAD_ANNOTATION_ATTRIBUTE__(lockable)
97 #endif 111 #endif
112 #define RTC_LOCKABLE RTC_THREAD_ANNOTATION_ATTRIBUTE__(lockable)
98 113
99 // Document if a class is a scoped lockable type (such as the MutexLock class). 114 // Document if a class is a scoped lockable type (such as the MutexLock class).
100 #if !defined(SCOPED_LOCKABLE) 115 #if !defined(SCOPED_LOCKABLE)
101 #define SCOPED_LOCKABLE THREAD_ANNOTATION_ATTRIBUTE__(scoped_lockable) 116 #define SCOPED_LOCKABLE RTC_THREAD_ANNOTATION_ATTRIBUTE__(scoped_lockable)
102 #endif 117 #endif
118 #define RTC_SCOPED_LOCKABLE RTC_THREAD_ANNOTATION_ATTRIBUTE__(scoped_lockable)
103 119
104 // The following annotations specify lock and unlock primitives. 120 // The following annotations specify lock and unlock primitives.
105 #if !defined(EXCLUSIVE_LOCK_FUNCTION) 121 #if !defined(EXCLUSIVE_LOCK_FUNCTION)
106 #define EXCLUSIVE_LOCK_FUNCTION(...) \ 122 #define EXCLUSIVE_LOCK_FUNCTION(...) \
107 THREAD_ANNOTATION_ATTRIBUTE__(exclusive_lock_function(__VA_ARGS__)) 123 RTC_THREAD_ANNOTATION_ATTRIBUTE__(exclusive_lock_function(__VA_ARGS__))
108 #endif 124 #endif
125 #define RTC_EXCLUSIVE_LOCK_FUNCTION(...) \
126 RTC_THREAD_ANNOTATION_ATTRIBUTE__(exclusive_lock_function(__VA_ARGS__))
109 127
110 #if !defined(SHARED_LOCK_FUNCTION) 128 #if !defined(SHARED_LOCK_FUNCTION)
111 #define SHARED_LOCK_FUNCTION(...) \ 129 #define SHARED_LOCK_FUNCTION(...) \
112 THREAD_ANNOTATION_ATTRIBUTE__(shared_lock_function(__VA_ARGS__)) 130 RTC_THREAD_ANNOTATION_ATTRIBUTE__(shared_lock_function(__VA_ARGS__))
113 #endif 131 #endif
132 #define RTC_SHARED_LOCK_FUNCTION(...) \
133 RTC_THREAD_ANNOTATION_ATTRIBUTE__(shared_lock_function(__VA_ARGS__))
114 134
115 #if !defined(EXCLUSIVE_TRYLOCK_FUNCTION) 135 #if !defined(EXCLUSIVE_TRYLOCK_FUNCTION)
116 #define EXCLUSIVE_TRYLOCK_FUNCTION(...) \ 136 #define EXCLUSIVE_TRYLOCK_FUNCTION(...) \
117 THREAD_ANNOTATION_ATTRIBUTE__(exclusive_trylock_function(__VA_ARGS__)) 137 RTC_THREAD_ANNOTATION_ATTRIBUTE__(exclusive_trylock_function(__VA_ARGS__))
118 #endif 138 #endif
139 #define RTC_EXCLUSIVE_TRYLOCK_FUNCTION(...) \
140 RTC_THREAD_ANNOTATION_ATTRIBUTE__(exclusive_trylock_function(__VA_ARGS__))
119 141
120 #if !defined(SHARED_TRYLOCK_FUNCTION) 142 #if !defined(SHARED_TRYLOCK_FUNCTION)
121 #define SHARED_TRYLOCK_FUNCTION(...) \ 143 #define SHARED_TRYLOCK_FUNCTION(...) \
122 THREAD_ANNOTATION_ATTRIBUTE__(shared_trylock_function(__VA_ARGS__)) 144 RTC_THREAD_ANNOTATION_ATTRIBUTE__(shared_trylock_function(__VA_ARGS__))
123 #endif 145 #endif
146 #define RTC_SHARED_TRYLOCK_FUNCTION(...) \
147 RTC_THREAD_ANNOTATION_ATTRIBUTE__(shared_trylock_function(__VA_ARGS__))
124 148
125 #if !defined(UNLOCK_FUNCTION) 149 #if !defined(UNLOCK_FUNCTION)
126 #define UNLOCK_FUNCTION(...) \ 150 #define UNLOCK_FUNCTION(...) \
127 THREAD_ANNOTATION_ATTRIBUTE__(unlock_function(__VA_ARGS__)) 151 RTC_THREAD_ANNOTATION_ATTRIBUTE__(unlock_function(__VA_ARGS__))
128 #endif 152 #endif
153 #define RTC_UNLOCK_FUNCTION(...) \
154 RTC_THREAD_ANNOTATION_ATTRIBUTE__(unlock_function(__VA_ARGS__))
129 155
130 // An escape hatch for thread safety analysis to ignore the annotated function. 156 // An escape hatch for thread safety analysis to ignore the annotated function.
131 #if !defined(NO_THREAD_SAFETY_ANALYSIS) 157 #if !defined(NO_THREAD_SAFETY_ANALYSIS)
132 #define NO_THREAD_SAFETY_ANALYSIS \ 158 #define NO_THREAD_SAFETY_ANALYSIS \
133 THREAD_ANNOTATION_ATTRIBUTE__(no_thread_safety_analysis) 159 RTC_THREAD_ANNOTATION_ATTRIBUTE__(no_thread_safety_analysis)
134 #endif 160 #endif
161 #define RTC_NO_THREAD_SAFETY_ANALYSIS \
162 RTC_THREAD_ANNOTATION_ATTRIBUTE__(no_thread_safety_analysis)
135 163
136 #endif // WEBRTC_RTC_BASE_THREAD_ANNOTATIONS_H_ 164 #endif // WEBRTC_RTC_BASE_THREAD_ANNOTATIONS_H_
OLDNEW
« no previous file with comments | « no previous file | webrtc/rtc_base/thread_annotations_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698