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

Side by Side Diff: include/core/SkSpinlock.h

Issue 1229003004: Fix componene debug build failure in chromium (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: alternative way Created 5 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 | no next file » | 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 2015 Google Inc. 2 * Copyright 2015 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 // This file is not part of the public Skia API. 8 // This file is not part of the public Skia API.
9 9
10 #ifndef SkSpinlock_DEFINED 10 #ifndef SkSpinlock_DEFINED
11 #define SkSpinlock_DEFINED 11 #define SkSpinlock_DEFINED
12 12
13 #include "SkAtomics.h" 13 #include "SkAtomics.h"
14 14
15 #define SK_DECLARE_STATIC_SPINLOCK(name) namespace {} static SkPODSpinlock name 15 #define SK_DECLARE_STATIC_SPINLOCK(name) namespace {} static SkPODSpinlock name
16 16
17 // This class has no constructor and must be zero-initialized (the macro above d oes this). 17 // This class has no constructor and must be zero-initialized (the macro above d oes this).
18 class SkPODSpinlock { 18 class SK_API SkPODSpinlock {
19 public: 19 public:
20 void acquire() { 20 void acquire() {
21 // To act as a mutex, we need an acquire barrier if we take the lock. 21 // To act as a mutex, we need an acquire barrier if we take the lock.
22 if (sk_atomic_exchange(&fLocked, true, sk_memory_order_acquire)) { 22 if (sk_atomic_exchange(&fLocked, true, sk_memory_order_acquire)) {
23 // Lock was contended. Fall back to an out-of-line spin loop. 23 // Lock was contended. Fall back to an out-of-line spin loop.
24 this->contendedAcquire(); 24 this->contendedAcquire();
25 } 25 }
26 } 26 }
27 27
28 void release() { 28 void release() {
29 // To act as a mutex, we need a release barrier. 29 // To act as a mutex, we need a release barrier.
30 sk_atomic_store(&fLocked, false, sk_memory_order_release); 30 sk_atomic_store(&fLocked, false, sk_memory_order_release);
31 } 31 }
32 32
33 private: 33 private:
34 void contendedAcquire(); 34 void contendedAcquire();
35 bool fLocked; 35 bool fLocked;
36 }; 36 };
37 37
38 // For non-global-static use cases, this is normally what you want. 38 // For non-global-static use cases, this is normally what you want.
39 class SkSpinlock : public SkPODSpinlock { 39 class SkSpinlock : public SkPODSpinlock {
40 public: 40 public:
41 SkSpinlock() { this->release(); } 41 SkSpinlock() { this->release(); }
42 }; 42 };
43 43
44 #endif//SkSpinlock_DEFINED 44 #endif//SkSpinlock_DEFINED
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698