Chromium Code Reviews| Index: include/core/SkDynamicAnnotations.h |
| diff --git a/include/core/SkDynamicAnnotations.h b/include/core/SkDynamicAnnotations.h |
| index 6d21cddb940b363d16a614d3b9f54f59af3e7dcf..2812fb4069d191bca1de730160bd8182e72ee83d 100644 |
| --- a/include/core/SkDynamicAnnotations.h |
| +++ b/include/core/SkDynamicAnnotations.h |
| @@ -19,6 +19,8 @@ extern "C" { |
| // TSAN provides these hooks. |
| void AnnotateIgnoreReadsBegin(const char* file, int line); |
| void AnnotateIgnoreReadsEnd(const char* file, int line); |
| +void AnnotateBenignRaceSized(const char* file, int line, |
| + const void* addr, long size, const char* desc); |
| } // extern "C" |
| // SK_ANNOTATE_UNPROTECTED_READ can wrap any variable read to tell TSAN to ignore that it appears to |
| @@ -37,10 +39,46 @@ inline T SK_ANNOTATE_UNPROTECTED_READ(const volatile T& x) { |
| return read; |
| } |
| +// Ignore racy reads and racy writes to this pointer, indefinitely. |
| +// If at all possible, use the more precise SK_ANNOTATE_UNPROTECTED_READ. |
| +template <typename T> |
| +void SK_ANNOTATE_BENIGN_RACE(T* ptr) { |
| + AnnotateBenignRaceSized(__FILE__, __LINE__, ptr, sizeof(*ptr), "SK_ANNOTATE_BENIGN_RACE"); |
| +} |
| + |
| #else // !DYNAMIC_ANNOTATIONS_ENABLED |
| #define SK_ANNOTATE_UNPROTECTED_READ(x) (x) |
| +#define SK_ANNOTATE_BENIGN_RACE(ptr) |
| #endif |
| +// Can be used to wrap values that are intentionally racy, usually small mutable cached values, e.g. |
| +// - SkMatrix type mask |
| +// - SkPathRef bounds |
| +// - SkPixelRef genIDs |
| +template <typename T> |
| +class SkRacy { |
|
reed1
2014/07/08 19:08:54
nit: SkTRacy
mtklein
2014/07/08 19:14:22
Done.
|
| +public: |
| + SkRacy() { SK_ANNOTATE_BENIGN_RACE(&fVal); } |
| + |
| + operator const T&() const { |
| + return fVal; |
| + } |
| + |
| + SkRacy& operator=(const T& val) { |
| + fVal = val; |
| + return *this; |
| + } |
| + |
| + const T* get() const { return &fVal; } |
| + T* get() { return &fVal; } |
| + |
| + const T* operator->() const { return &fVal; } |
| + T* operator->() { return &fVal; } |
| + |
| +private: |
| + T fVal; |
| +}; |
| + |
| #endif//SkDynamicAnnotations_DEFINED |