From: Behdad Esfahbod <behdad@behdad.org>
Date: Thu, 9 Aug 2018 05:01:04 +0000 (-0700)
Subject: Fix previous commit to use atomic operations
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=51ffc3e65aacbece63995be99e2bc20538e3eb75;p=platform%2Fupstream%2FlibHarfBuzzSharp.git

Fix previous commit to use atomic operations
---

diff --git a/src/hb-machinery-private.hh b/src/hb-machinery-private.hh
index 39a7fd1..ec41e2b 100644
--- a/src/hb-machinery-private.hh
+++ b/src/hb-machinery-private.hh
@@ -639,9 +639,16 @@ struct hb_lazy_loader_t
 
   inline void set_stored (Stored *instance_)
   {
-    if (instance)
-      thiz ()->destroy (instance);
-    instance = instance_;
+    /* This *must* be called when there are no other threads accessing.
+     * However, to make TSan, etc, happy, we using cmpexch. */
+  retry:
+    Stored *p = (Stored *) hb_atomic_ptr_get (&this->instance);
+    if (p)
+    {
+      if (unlikely (!hb_atomic_ptr_cmpexch (const_cast<Stored **>(&this->instance), p, instance_)))
+        goto retry;
+      thiz ()->destroy (p);
+    }
   }
 
   inline const Returned * get (void) const