Changeset 95d45482 in mainline for kernel/generic/include/lib/refcount.h

Timestamp:

2018-11-09T22:29:12Z (5 years ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

be6e37a

Parents:

436a0a5

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-11-08 23:08:28)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-11-09 22:29:12)

Message:

XXX to NOTE

File:

• kernel/generic/include/lib/refcount.h (modified) (3 diffs)

kernel/generic/include/lib/refcount.h

@@ -64 +64 @@
 static inline void refcount_up(atomic_refcount_t *rc)
 {
-        // XXX: We can use relaxed operation because acquiring a reference
-        //      implies no ordering relationships. A reference-counted object
-        //      still needs to be synchronized independently of the refcount.
+        // NOTE: We can use relaxed operation because acquiring a reference
+        //       implies no ordering relationships. A reference-counted object
+        //       still needs to be synchronized independently of the refcount.
 
         int old = atomic_fetch_add_explicit(&rc->__cnt, 1,
@@ -96 +96 @@
 static inline bool refcount_down(atomic_refcount_t *rc)
 {
-        // XXX: The decrementers don't need to synchronize with each other,
-        //      but they do need to synchronize with the one doing deallocation.
+        // NOTE: The decrementers don't need to synchronize with each other,
+        //       but they do need to synchronize with the one doing deallocation.
         int old = atomic_fetch_sub_explicit(&rc->__cnt, 1,
             memory_order_release);
@@ -104 +104 @@
 
         if (old == 0) {
-                // XXX: We are holding the last reference, so we must now
-                //      synchronize with all the other decrementers.
+                // NOTE: We are holding the last reference, so we must now
+                //       synchronize with all the other decrementers.
 
                 int val = atomic_load_explicit(&rc->__cnt,